ddanielr commented on PR #4163: URL: https://github.com/apache/accumulo/pull/4163#issuecomment-1894161318
> @EdColeman and @ddanielr for making the change in main, thinking of the following strategy. > > * Create just the new AccumuloTables enum in main and merge it forward > * Do the refactor in main to inline and merge that commit forward ignoring the changes in it > * Do the refactor in elasticity as its own commit > > If that strategy sounds good I can do that w/o any further PRs. I think that's an easier strategy and keeps the elasticity complexity fixes from being hidden in a merge commit. -- 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]
