EdColeman opened a new issue, #4094:
URL: https://github.com/apache/accumulo/issues/4094
In `ReferencedTabletFile` and inherited by `StoredTabletFile` there is a
method
```
public TableId getTableId() ...
```
The id is being derived by parsing the file path. If a table had been
cloned, the id in the path may not be the table id. At a minimum, the method
could be renamed, and possibly return an id that is not a TableId so that any
id derived from the path cannot be directly used as a TableId.
The issue would be if someone assumed that the TableId returned from a
ReferencedTabletFile was the same a the "real" table id and then made metadata
changes based on incorrect information. Just using the method signature, it
seems easy to make an incorrect assumption on what that id returned represents.
--
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]
