FrankChen021 commented on code in PR #20592:
URL: https://github.com/apache/druid/pull/20592#discussion_r4273095208
##########
processing/src/main/java/org/apache/druid/common/guava/GuavaUtils.java:
##########
@@ -71,7 +71,7 @@ public static <T extends Enum<T>> T getEnumIfPresent(final
Class<T> enumClass, f
/**
* If first argument is not null, return it, else return the other argument.
- * Sort of like {@link <T> com.google.common.base.Objects#firstNonNull(T, T)}
+ * Sort of like {@link com.google.common.base.Objects#firstNonNull(T, T)}
Review Comment:
[P3] Point the repaired link at MoreObjects
**Finding:** The link targets com.google.common.base.Objects#firstNonNull,
but the pinned Guava version provides this method on MoreObjects; generated
Javadoc reports the reference as unresolved.
**Suggestion:** Update the link target to MoreObjects#firstNonNull.
--
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]
