cshannon commented on code in PR #3502:
URL: https://github.com/apache/accumulo/pull/3502#discussion_r1232394767


##########
core/src/main/java/org/apache/accumulo/core/metadata/StoredTabletFile.java:
##########
@@ -46,7 +49,7 @@ public class StoredTabletFile extends 
AbstractTabletFile<StoredTabletFile> {
   public StoredTabletFile(String metadataEntry) {
     super(new Path(metadataEntry));
     this.metadataEntry = metadataEntry;

Review Comment:
   Of course there could be a mistake in the future but as of now it's not an 
issue when I search the code. I took out that validate call because it was 
pointed out as an issue in the original Issue description #3500. 
   
   Leaving the call the performance was 0.005 ms/op but with it removed I get 
0.004 ms/op so a little better.



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