imbajin commented on code in PR #685:
URL:
https://github.com/apache/incubator-hugegraph-toolchain/pull/685#discussion_r2465632514
##########
hugegraph-client/src/main/java/org/apache/hugegraph/client/RestClient.java:
##########
@@ -39,6 +40,10 @@
public class RestClient extends AbstractRestClient {
private static final int SECOND = 1000;
+ private String version;
Review Comment:
⚠️ **字段位置和访问控制**
`supportGs` 字段的位置和访问修饰符发生了变化(从原来的位置移到了靠前的位置,且使用了 Lombok 注解)。建议:
1. 保持字段声明的顺序一致(`supportGs` 和 `version` 相关字段应该放在一起)
2. 确认 `version` 字段是否需要添加 getter(当前只声明但未使用 `@Getter`)
**当前代码:**
```java
private String version;
@Getter
@Setter
private boolean supportGs;
```
**建议:**
```java
private String version; // 考虑添加 @Getter 如果需要对外暴露
@Getter
@Setter
private boolean supportGs;
```
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]