Remove dead "waitForHealth" code in ReplSetTest

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Works as Designed
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Replication
    • None
    • Replication
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      'waitForHealth' is an optional parameter to a few ReplSetTest methods, but it always defaults to false, and I don't see any tests using it:

      $ git grep waitForHealth
      src/mongo/shell/replsettest.js:    function _waitForHealth(node, state, timeout) {
      src/mongo/shell/replsettest.js:     * @param {boolean} [waitForHealth] If true, wait for the health indicator of the replica set
      src/mongo/shell/replsettest.js:    this._waitForInitialConnection = function(n, waitForHealth) {
      src/mongo/shell/replsettest.js:        waitForHealth = waitForHealth || false;
      src/mongo/shell/replsettest.js:        if (waitForHealth) {
      src/mongo/shell/replsettest.js:            _waitForHealth(this.nodes[n], Health.UP);
      src/mongo/shell/replsettest.js:     * @param {boolean} [waitForHealth] If true, wait for the health indicator of the replica set
      src/mongo/shell/replsettest.js:        n, options, restart, waitForHealth) {
      src/mongo/shell/replsettest.js:        return this._waitForInitialConnection(n, waitForHealth);
      

      If it's dead code, it would be good to remove it.

            Assignee:
            [DO NOT USE] Backlog - Replication Team
            Reporter:
            Esha Maharishi (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: