wangbo commented on code in PR #48187:
URL: https://github.com/apache/doris/pull/48187#discussion_r1971486336
##########
fe/fe-core/src/main/java/org/apache/doris/qe/ConnectContext.java:
##########
@@ -996,17 +1001,17 @@ public ThreadInfo toThreadInfo(boolean isFull) {
return threadInfo;
}
- public boolean isResourceTagsSet() {
- return isResourceTagsSet;
+ public boolean isSetComputeGroup() {
+ return isSetComputeGroup;
}
- public Set<Tag> getResourceTags() {
- return resourceTags;
+ private Set<Tag> getComputeGroupTags() {
+ return computeGroupTags;
}
- public void setResourceTags(Set<Tag> resourceTags) {
- this.resourceTags = resourceTags;
- this.isResourceTagsSet = !this.resourceTags.isEmpty();
+ public void setComputeGroupTags(Set<Tag> computeGroupTags) {
Review Comment:
这个主要是为了兼容旧版的resource
tag用法，是设置到连接里。每次登录时设置，之前跟youge讨论过要不要把这个删了，最后youge担心获取tag比较高频，每次从usermgr里拿性能可能有问题，于是保留了连接粒度的
--
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-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
