keith-turner commented on code in PR #5935:
URL: https://github.com/apache/accumulo/pull/5935#discussion_r2383291009


##########
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:
   > Are users expected to create CompactableFile implementations?
   
   I have found those to be useful when writing unit test for different 
plugins.  Your plugin will be passed those and will not need to create them 
when running in accumulo.



-- 
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]

Reply via email to