cshannon commented on PR #3432: URL: https://github.com/apache/accumulo/pull/3432#issuecomment-1564557832
Sunny IT tests passed and I kicked off a full IT build. Also, I just had a thought that maybe as part of this PR (which only applies in the 2.1 branch where there is still an inheritance relationship) we should actually override https://github.com/apache/accumulo/blob/969fc52f1c5e0c195a8cef800a87e0fab13cd89c/core/src/main/java/org/apache/accumulo/core/metadata/TabletFile.java#L109 and https://github.com/apache/accumulo/blob/969fc52f1c5e0c195a8cef800a87e0fab13cd89c/core/src/main/java/org/apache/accumulo/core/metadata/TabletFile.java#L109 and either throw a new `UnsupportedOperationException` or return the metadataEntry (instead of the normalized path) so we prevent any errors in the future so the normalized path is never used. In #3417 this of course wouldn't be necessary anymore. -- 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]
