dlmarion commented on code in PR #4104:
URL: https://github.com/apache/accumulo/pull/4104#discussion_r1434407845
##########
core/src/main/java/org/apache/accumulo/core/metadata/schema/Ample.java:
##########
@@ -557,7 +557,7 @@ ConditionalTabletMutator requireSame(TabletMetadata
tabletMetadata, ColumnType t
* second cause is ample resubmitting on unknown as mentioned above. Below
are a few examples
* that go over how Ample will handle these different situations.
*
- * <h3>Example 1</h3>
+ * <h4>Example 1</h4>
Review Comment:
That's fair, but I was just trying to fix the build. I used the PR mechanism
to test that all checks passed successfully. The commit to include the headings
was done long ago, I think a recent change in the build caused it to fail.
Update to JDK 17 maybe? If we want to remove all HTML headings from all
javadoc, then it might be useful to create a separate ticket as I'm assuming we
would want to add a check to prevent them from being reintroduced.
--
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]
