Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/921#discussion_r150978981
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRoot.java 
---
    @@ -315,7 +481,12 @@ public int compareTo(DrillbitInfo drillbitToCompare) {
     
           if (this.isVersionMatch() == drillbitToCompare.isVersionMatch()) {
             if (this.version.equals(drillbitToCompare.getVersion())) {
    -          return this.address.compareTo(drillbitToCompare.getAddress());
    +          {
    +            if(this.address.equals(drillbitToCompare.getAddress())) {
    +              return 
(this.controlPort.compareTo(drillbitToCompare.getControlPort()));
    --- End diff --
    
    Repeat of the `isSameDrillbit()` pattern noted earlier.


---

Reply via email to