Repository: hbase Updated Branches: refs/heads/branch-2 22e7ae031 -> 8c7293c51 refs/heads/branch-2.0 d8985e04b -> 0207ed03f
HBASE-20438 Add an HBase antipattern check for reintroducing commons-logging Signed-off-by: Sean Busbey <bus...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/8c7293c5 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/8c7293c5 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/8c7293c5 Branch: refs/heads/branch-2 Commit: 8c7293c5147f788a7e7fa150868df4d15f0e0f55 Parents: 22e7ae0 Author: Nihal Jain <nihaljain...@gmail.com> Authored: Wed Apr 18 23:03:07 2018 +0530 Committer: Sean Busbey <bus...@apache.org> Committed: Fri Apr 20 11:58:39 2018 -0500 ---------------------------------------------------------------------- dev-support/hbase-personality.sh | 6 ++++++ 1 file changed, 6 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/8c7293c5/dev-support/hbase-personality.sh ---------------------------------------------------------------------- diff --git a/dev-support/hbase-personality.sh b/dev-support/hbase-personality.sh index 2198913..b010503 100755 --- a/dev-support/hbase-personality.sh +++ b/dev-support/hbase-personality.sh @@ -642,6 +642,12 @@ function hbaseanti_patchfile ((result=result+1)) fi + warnings=$(${GREP} -cE 'org.apache.commons.logging.Log(Factory|;)' "${patchfile}") + if [[ ${warnings} -gt 0 ]]; then + add_vote_table -1 hbaseanti "" "The patch appears to use commons-logging instead of slf4j." + ((result=result+1)) + fi + if [[ ${result} -gt 0 ]]; then return 1 fi