cshannon commented on PR #3417: URL: https://github.com/apache/accumulo/pull/3417#issuecomment-1566204189
As I was looking at the next round of changes after this PR to start storing a range as part of the metadata I noticed that the getPathStr() method returns the normalized path. I decided to rename this to `getNormalizedPath()` in my newest commit so it is more clear what is actually being returned. The main reason I did this is the original path that is given to construct a TabletFile is stored and can be accessed with `getPath()`. So calling getPathStr() is a bit confusing as I would have expected this to return the original string given and not the normalized version. Returning the normalized version looks to be fine/correct (it's used in a lot of places, even by StoredTabletFile) for reading/writing but this just makes what is being used clearer in the code when reading it. -- 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]
