Uploaded image for project: 'Kudu'
  1. Kudu
  2. KUDU-822

[java client] All masters are good means none of them are

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • M5
    • None
    • client
    • None

    Description

      Saw this on jenkins:

      2015-06-16 17:59:33,738 (New I/O worker #1) [WARN - org.kududb.client.BaseKuduTest$3.call(BaseKuduTest.java:403)] Got exception
      org.kududb.client.NonRecoverableException: Couldn't find a valid master in (127.0.0.1:64030,127.0.0.1:64032,127.0.0.1:64034), exceptions: []
      	at org.kududb.client.GetMasterRegistrationReceived.incrementCountAndCheckExhausted(GetMasterRegistrationReceived.java:101)
      	at org.kududb.client.GetMasterRegistrationReceived.access$300(GetMasterRegistrationReceived.java:25)
      	at org.kududb.client.GetMasterRegistrationReceived$GetMasterRegistrationCB.call(GetMasterRegistrationReceived.java:157)
      	at org.kududb.client.GetMasterRegistrationReceived$GetMasterRegistrationCB.call(GetMasterRegistrationReceived.java:122)
      

      It lists 0 exceptions. This is the code:

        private void incrementCountAndCheckExhausted() {
          if (countResponsesReceived.incrementAndGet() == numMasters) {
            if (responseDCalled.compareAndSet(false, true)) {
              boolean allUnrecoverable = true;
              for (Exception ex : exceptionsReceived) {
                if (!(ex instanceof NonRecoverableException)) {
                  allUnrecoverable = false;
                  break;
                }
              }
              String allHosts = NetUtil.hostsAndPortsToString(masterAddrs);
              if (allUnrecoverable) {
                // This will stop retries.
                responseD.callback(new NonRecoverableException("Couldn't find a valid master in (" +
                    allHosts + "), exceptions: " + exceptionsReceived));
      

      allUnrecoverable stayed true because there were no exceptions. That's probably fine, we should continue and later we'll figure out the real leader.

      Attachments

        Activity

          People

            jdcryans Jean-Daniel Cryans
            jdcryans Jean-Daniel Cryans
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: