Copilot commented on code in PR #6205:
URL: https://github.com/apache/hive/pull/6205#discussion_r2597716770
##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -6551,6 +6551,19 @@ public HiveConf(Configuration other, Class<?> cls) {
initialize(cls);
}
+ /**
+ * For internal use only, assumed the "other" has loaded all properties that
intend to use
+ * and want to cast it to a HiveConf without extra re-loading the source
file.
+ * @param other
Review Comment:
The Javadoc parameter tag is missing a description. Consider adding a
description for the `other` parameter to document what configuration is being
wrapped.
```suggestion
* @param other The Configuration instance whose properties are to be
wrapped by this HiveConf.
```
--
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]
