Copilot commented on code in PR #16933:
URL: https://github.com/apache/iotdb/pull/16933#discussion_r2634394428
##########
iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/conf/CommonConfig.java:
##########
@@ -2728,4 +2729,12 @@ public String getAuditableOperationResult() {
public void setAuditableOperationResult(String auditableOperationResult) {
this.auditableOperationResult = auditableOperationResult;
}
+
+ public boolean getRestrictObjectLimit() {
+ return restrictObjectLimit;
+ }
+
Review Comment:
The getter method name should follow the standard Java Bean naming
convention for boolean properties. For a boolean field named
"restrictObjectLimit", the getter should be "isRestrictObjectLimit()" rather
than "getRestrictObjectLimit()". This is inconsistent with other boolean
getters in this class (e.g., isEnforceStrongPassword(),
isEnableBinaryAllocator()) and causes a mismatch with Thrift-generated code
that uses "isRestrictObjectLimit()".
```suggestion
public boolean isRestrictObjectLimit() {
return restrictObjectLimit;
}
@Deprecated
public boolean getRestrictObjectLimit() {
return isRestrictObjectLimit();
}
```
--
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]
