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]