[SERVER-6972] C++ driver should use isMaster, not replSetGetStatus, to monitor health of a replica set Created: 07/Sep/12 Updated: 11/Jul/16 Resolved: 02/Apr/13 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Internal Client, Security, Sharding |
| Affects Version/s: | 2.0.7, 2.2.0 |
| Fix Version/s: | 2.5.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Spencer Brody (Inactive) | Assignee: | Spencer Brody (Inactive) |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||
| Operating System: | ALL | ||||||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||||||
| Description |
|
It is a part of the driver spec that all drivers should use isMaster for monitoring the health of a replica set and for node discovery. The C++ driver is the only driver that uses replSetGetStatus. This causes several problems, the most major of which is that replSetGetStatus requires auth to run, so it is currently completely impossible to use the C++ driver on an authenticated cluster. Another potential problem is that replSetGetStatus is a heavier command that may take locks and puts more load on the server than isMaster does. Mongos currently isn't using any information from replSetGetStatus that isn't available in isMaster, so we should be able to use isMaster instead without losing any functionality. |
| Comments |
| Comment by auto [ 02/Apr/13 ] |
|
Author: {u'date': u'2013-04-02T15:04:52Z', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}Message: |
| Comment by auto [ 02/Apr/13 ] |
|
Author: {u'date': u'2013-03-15T18:46:45Z', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}Message: |
| Comment by auto [ 08/Feb/13 ] |
|
Author: {u'date': u'2013-02-08T20:22:22Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}Message: Skip jstests/sharding/read_pref.js when running test with auth |
| Comment by auto [ 24/Jan/13 ] |
|
Author: {u'date': u'2013-01-24T20:20:04Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}Message: |
| Comment by auto [ 07/Nov/12 ] |
|
Author: {u'date': u'2012-11-07T01:48:16Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}Message: |
| Comment by auto [ 10/Sep/12 ] |
|
Author: {u'date': u'2012-09-07T13:23:07-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}Message: |