CRZbulabula commented on code in PR #16432:
URL: https://github.com/apache/iotdb/pull/16432#discussion_r2362330438
##########
iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/auth/entity/User.java:
##########
@@ -73,7 +82,15 @@ public void addRole(String roleName) {
roleSet.add(roleName);
}
+ public void setUserId(long userId) {
+ this.userId = userId;
+ }
Review Comment:
Cuz we encounter some compatitive issues. Specifically, when loading the old
version of authority snapshot, where there are no userId records, we cannot set
userId accordingly. To set the userId correctly, we have to wait until all
authority entities are loaded completely. Thus we left this setter here.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
