I've filed HBASE-26617 a while ago and recently I implemented a PR for master branch.
https://github.com/apache/hbase/pull/4214 The PR is a bit large because it will also format the pom, the actual changes are here https://github.com/apache/hbase/pull/4214/files#diff-9c5fb3d1b7e3b0f54bc5c4182965c4fe1f9023d449017cece3005d3f90e8e4d8R2673 In the PR I make use of the eclipse formatter and import order file to format our java file. For pom, I just use most of the default formatter, the only exception is to expand empty element. The spotless plugin support setting a ratchetFrom option, which could be a commit or a tag. Only files changed after this commit will be formatted, so we can avoid generating a very big patch when running spotless:apply. So after we get this in, before submitting a PR, you can just type mvn spotless:apply, then most of the checkstyle issues will be solved automatically. And we could also add a spotless:check step in our pre commit job, to make sure we run spotless:apply before submitting a PR. Just tell me what you think about the above plan. Suggestions are always welcomed. Thanks.
