bbeaudreault opened a new pull request, #4580: URL: https://github.com/apache/hbase/pull/4580
This should have been a relatively simple backport, but ran into some issues: - in the original backport, i tested the change using `mvn clean verify -pl hbase-server -am`. I forgot that AggregationClient had moved to hbase-endpoint, so this erroneously made it seem like the backport was good (since it just didn't compile hbase-endpoint at all) - I fixed that with an addendum, but somehow my editor messed up the import grouping. Lesson learned: make sure to test if not doing a PR (i tried, but will be more careful next time), and make sure to run spotless:apply any time i make a non-cherrypick change. Submitting this as a PR so i can run the full pre-commit checks and be absolutely sure that this will be the last addendum for this issue. -- 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]
