[GitHub] [doris] compiletheworld commented on a diff in pull request #10901: [feature](resource-tag) support multi tag for a single Backend

[GitBox <gi...@apache.org>]

compiletheworld commented on code in PR #10901:
URL: https://github.com/apache/doris/pull/10901#discussion_r922689190


##########
fe/fe-core/src/main/java/org/apache/doris/system/Backend.java:
##########
@@ -116,8 +119,14 @@ public enum BackendState {
     // additional backendStatus information for BE, display in JSON format
     @SerializedName("backendStatus")
     private BackendStatus backendStatus = new BackendStatus();
-    @SerializedName("tag")
-    private Tag tag = Tag.DEFAULT_BACKEND_TAG;
+    // the locationTag is also saved in tagMap, use a single field here to avoid
+    // creating this everytime we get it.
+    @SerializedName(value = "locationTag", alternate = {"tag"})
+    private Tag locationTag = Tag.DEFAULT_BACKEND_TAG;

Review Comment:
   Will we delete this field `locationTag` in the future?
   It may be confusing if we keep a special `entry` out of the `tagMap`.



-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org