[ https://issues.apache.org/jira/browse/CALCITE-5917?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17753048#comment-17753048 ]
Benchao Li commented on CALCITE-5917: ------------------------------------- We would accept such minor changes, e.g., improving javadoc/method parameter names, and usually there is no need to link to a Jira. Take a example, [https://github.com/apache/calcite/pull/3308] > Comments in TimeString class are incorrect > ------------------------------------------ > > Key: CALCITE-5917 > URL: https://issues.apache.org/jira/browse/CALCITE-5917 > Project: Calcite > Issue Type: Bug > Components: core > Affects Versions: 1.35.0 > Reporter: Mihai Budiu > Priority: Trivial > > Here is an example of an incorrect comment: > {code:java} > /** Sets the fraction field of a {@code TimeString} to a given number > * of milliseconds. Nukes the value set via {@link #withNanos}. > * > * <p>For example, > * {@code new TimeString(1970, 1, 1, 2, 3, 4).withMillis(56)} > * yields {@code TIME '1970-01-01 02:03:04.056'}. */ > public TimeString withMillis(int millis) { > {code} > There are several such comments. -- This message was sent by Atlassian Jira (v8.20.10#820010)