Fix read-only mode for LoadBalancerSingle
authorLucas Werkmeister <lucas.werkmeister@wikimedia.de>
Wed, 6 Feb 2019 18:49:12 +0000 (19:49 +0100)
committerLucas Werkmeister <lucas.werkmeister@wikimedia.de>
Fri, 8 Feb 2019 13:06:35 +0000 (14:06 +0100)
commit0b583944e45b65be94312fc855b5366615fa290c
tree853bef14ed6196d32a8e264e4a73644149d4bb5e
parentd90508d3c2f646895c60406d242cb65d33c9306d
Fix read-only mode for LoadBalancerSingle

For master connections, LoadBalancer unconditionally replaces the
connection’s read-only mode with the load balancer’s, so if we don’t
include the read-only reason in the parameters we pass to the parent
constructor, the parameter we set on the connection has no effect.

I attempted to write tests for this, but eventually had to stop wasting
time on it, since I couldn’t get anywhere. Add a warning to the top of
the class to discourage others from making the same mistake.

Change-Id: I481553fac4f0f13a429ad4bbfdf15007c8299af9
includes/libs/rdbms/loadbalancer/LoadBalancerSingle.php