keith-turner commented on issue #5666:
URL: https://github.com/apache/accumulo/issues/5666#issuecomment-2997690231
The following is one way to approach these changes.
1. In 2.1.4 make the changes to TabletId and add a unit test for those new
TabletId methods..
2. In a follow on PR for main adapt existing tests to use the new methods
in TabletId.
This approach will introduce a minimal set of changes for 2.1 that could
easily be backed out and only made in main if there is opposition to the change
in 2.1. Changing existing test to use the new method in 2.1 could introduce
lots of merge conflicts when going from 2.1->main, so would be best to avoid
that.
--
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]
