dlmarion commented on code in PR #5935:
URL: https://github.com/apache/accumulo/pull/5935#discussion_r2383036596
##########
core/src/main/java/org/apache/accumulo/core/metadata/StoredTabletFile.java:
##########
@@ -149,17 +149,6 @@ public String toString() {
return metadataEntry;
}
- /**
- * Validates that the provided metadata string for the StoredTabletFile is
valid.
- */
- public static void validate(String metadataEntry) {
Review Comment:
Seems like this should be used somewhere.
##########
core/src/main/java/org/apache/accumulo/core/client/admin/compaction/CompactableFile.java:
##########
@@ -52,12 +52,4 @@ static CompactableFile create(URI uri, long estimatedSize,
long estimatedEntries
return new CompactableFileImpl(uri, estimatedSize, estimatedEntries);
}
- /**
- * Creates a new CompactableFile object that implements this interface.
- *
- * @since 4.0.0
- */
- static CompactableFile create(URI uri, Range range, long estimatedSize, long
estimatedEntries) {
Review Comment:
Added in 4.0, but not referenced. Is this a bug?
--
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]
