Revert "HBASE-20478 move hbaseanti import checks to checkstyle"
Need to update version of checkstyle used for new features.
This reverts commit 90d103d6d5
.
This commit is contained in:
parent
90d103d6d5
commit
e6220c8b60
|
@ -631,6 +631,24 @@ function hbaseanti_patchfile
|
||||||
((result=result+1))
|
((result=result+1))
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
warnings=$(${GREP} -c 'import org.apache.hadoop.classification' "${patchfile}")
|
||||||
|
if [[ ${warnings} -gt 0 ]]; then
|
||||||
|
add_vote_table -1 hbaseanti "" "The patch appears use Hadoop classification instead of HBase."
|
||||||
|
((result=result+1))
|
||||||
|
fi
|
||||||
|
|
||||||
|
warnings=$(${GREP} -c 'import org.codehaus.jackson' "${patchfile}")
|
||||||
|
if [[ ${warnings} -gt 0 ]]; then
|
||||||
|
add_vote_table -1 hbaseanti "" "The patch appears use Jackson 1 classes/annotations."
|
||||||
|
((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
|
if [[ ${result} -gt 0 ]]; then
|
||||||
return 1
|
return 1
|
||||||
fi
|
fi
|
||||||
|
|
|
@ -85,12 +85,7 @@
|
||||||
org.apache.commons.collections4,
|
org.apache.commons.collections4,
|
||||||
org.apache.commons.lang,
|
org.apache.commons.lang,
|
||||||
org.apache.curator.shaded,
|
org.apache.curator.shaded,
|
||||||
org.apache.hadoop.classification,
|
org.apache.htrace.shaded"/>
|
||||||
org.apache.htrace.shaded,
|
|
||||||
org.codehaus.jackson"/>
|
|
||||||
<property name="illegalClasses" value="
|
|
||||||
org.apache.commons.logging.Log,
|
|
||||||
org.apache.commons.logging.LogFactory"/>
|
|
||||||
</module>
|
</module>
|
||||||
<!-- Javadoc Checks
|
<!-- Javadoc Checks
|
||||||
http://checkstyle.sourceforge.net/config_javadoc.html -->
|
http://checkstyle.sourceforge.net/config_javadoc.html -->
|
||||||
|
|
Loading…
Reference in New Issue