QiuMM commented on a change in pull request #7066: Support assign tasks to run 
on different tiers of MiddleManagers
URL: https://github.com/apache/incubator-druid/pull/7066#discussion_r332027487
 
 

 ##########
 File path: 
server/src/main/java/org/apache/druid/discovery/WorkerNodeService.java
 ##########
 @@ -33,16 +33,19 @@
   private final String ip;
   private final int capacity;
   private final String version;
+  private final String tier;
 
   public WorkerNodeService(
       @JsonProperty("ip") String ip,
       @JsonProperty("capacity") int capacity,
-      @JsonProperty("version") String version
+      @JsonProperty("version") String version,
+      @JsonProperty("tier") String tier
   )
   {
     this.ip = ip;
     this.capacity = capacity;
     this.version = version;
+    this.tier = tier;
 
 Review comment:
   If users want to use this feature, they must upgrade overlord and all 
middleManager ( follow the rolling upgrade is ok). I have noted this in the 
document. An extra restart of overlord is not necessary.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to