nastra commented on a change in pull request #2826:
URL: https://github.com/apache/iceberg/pull/2826#discussion_r691001361
##########
File path:
hive3/src/main/java/org/apache/iceberg/mr/hive/vector/CompatibilityHiveVectorUtils.java
##########
@@ -59,7 +59,7 @@ private CompatibilityHiveVectorUtils() {
* Returns serialized mapwork instance from a job conf - ported from Hive
source code LlapHiveUtils#findMapWork
*
* @param job JobConf instance
- * @return
+ * @return A serialized {@link MapWork} based on the given job conf
Review comment:
I searched across the codebase and it seems that it's a bit inconsistent
in terms of upper/lowercase. I generally think that something like this would
be better if addressed by Checkstyle (but not sure if checkstyle supports such
a check). Otherwise stuff like this will essentially pop up in lots of PR
reviews
--
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]
