ctubbsii commented on code in PR #4104:
URL: https://github.com/apache/accumulo/pull/4104#discussion_r1434402159
##########
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:
I think it's better not to use headers in javadocs. Can just start a new
paragraph and have the `<ul>` immediately follow. Javadoc is really strict
about the use of headers because it uses them itself. I don't think we really
need to use them most of the time.
--
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]
