keith-turner commented on PR #3272:
URL: https://github.com/apache/accumulo/pull/3272#issuecomment-1497626850
> Should this start with the 2.1 branch?
I was wondering about that, its only a reorganization of the code. The pros
I can think of for going to 2.1 are :
* Makes it easier to maintain 2.1
* Makes it easier to merge from 2.1 to 3
The only con I can think of is that these changes may introduce a bug in 2.1.
Are there any other pros or cons?
If this change were pulled back to 2.1 then would also want to pull #3273
back. There is also another simple change I was thinking of making in main.
Wanted to rename the TabletLocator to TabletCache to reflect the changes that
will be made in the elasticity branch to generalize the responsibility of the
client side tablet cache. The reason I am making all of these changes in main
is to make merging to the elasticity branch easier. I plan to make the actual
changes to cache functionality to support ondemand tablets only in the
elasticity branch. All of these prereq changes for that cahce changes will
have been made in main. So would it make sense to make all of these prereq
changes in 2.1 instead?
--
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]
