FLUME-3280 Improve maven build to help code reviews by adding static …
authorFerenc Szabo <szaboferee@apache.org>
Thu, 8 Nov 2018 17:11:27 +0000 (18:11 +0100)
committerFerenc Szabo <szaboferee@apache.org>
Thu, 8 Nov 2018 17:11:27 +0000 (18:11 +0100)
commitfb16ffc076de9917be7585e5583344bf3fd1c62a
tree5dc22e1f02daa5044f0965084567bd05677238a9
parent493b53b6430c687f88c438e5883ef9120d7aedb2
FLUME-3280 Improve maven build to help code reviews by adding static …

…code analyzer

Adding spotbugs, pmd to the build.
moving flume-checkstyle to a new build-support module that contains
any future build tool configuration.

Fixed some trivial checkstyle issues.

Updated apache parent version and maven plugin versions.
Added maxAllowedViolations so this commit could contain only the new checks
and not any code change.

Did some cleanup in the pom files.

This closes #232

Reviewers: Peter Turcsanyi, Endre Major

(Ferenc Szabo via Ferenc Szabo)
101 files changed:
.gitignore
.mvn/wrapper/MavenWrapperDownloader.java [new file with mode: 0755]
.mvn/wrapper/maven-wrapper.properties [new file with mode: 0755]
.travis.yml
build-support/pom.xml [new file with mode: 0644]
build-support/src/main/resources/config/checkstyle/checkstyle-suppressions.xml [moved from flume-checkstyle/src/main/resources/flume/checkstyle-suppressions.xml with 100% similarity]
build-support/src/main/resources/config/checkstyle/checkstyle.xml [moved from flume-checkstyle/src/main/resources/flume/checkstyle.xml with 94% similarity]
flume-checkstyle/pom.xml [deleted file]
flume-ng-auth/pom.xml
flume-ng-channels/flume-file-channel/pom.xml
flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/Log.java
flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/LogFileV2.java
flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/instrumentation/FileChannelCounterMBean.java
flume-ng-channels/flume-jdbc-channel/pom.xml
flume-ng-channels/flume-kafka-channel/pom.xml
flume-ng-channels/flume-spillable-memory-channel/pom.xml
flume-ng-channels/pom.xml
flume-ng-clients/flume-ng-log4jappender/pom.xml
flume-ng-clients/flume-ng-log4jappender/src/test/java/org/apache/flume/clients/log4jappender/TestLog4jAppender.java
flume-ng-configfilters/flume-ng-environment-variable-config-filter/pom.xml
flume-ng-configfilters/flume-ng-external-process-config-filter/pom.xml
flume-ng-configfilters/flume-ng-external-process-config-filter/src/test/java/org/apache/flume/configfilter/TestExternalProcessConfigFilter.java
flume-ng-configfilters/flume-ng-hadoop-credential-store-config-filter/pom.xml
flume-ng-configuration/pom.xml
flume-ng-configuration/src/main/java/org/apache/flume/conf/channel/ChannelSelectorConfiguration.java
flume-ng-core/pom.xml
flume-ng-core/src/main/java/org/apache/flume/SinkProcessor.java
flume-ng-core/src/main/java/org/apache/flume/channel/AbstractChannel.java
flume-ng-core/src/main/java/org/apache/flume/channel/ReplicatingChannelSelector.java
flume-ng-core/src/main/java/org/apache/flume/client/avro/ReliableSpoolingFileEventReader.java
flume-ng-core/src/main/java/org/apache/flume/formatter/output/BucketPath.java
flume-ng-core/src/main/java/org/apache/flume/instrumentation/kafka/KafkaChannelCounter.java
flume-ng-core/src/main/java/org/apache/flume/serialization/AvroEventDeserializer.java
flume-ng-core/src/main/java/org/apache/flume/serialization/DurablePositionTracker.java
flume-ng-core/src/main/java/org/apache/flume/serialization/EventSerializer.java
flume-ng-core/src/main/java/org/apache/flume/serialization/FlumeEventAvroEventSerializer.java
flume-ng-core/src/main/java/org/apache/flume/sink/AbstractRpcSink.java
flume-ng-core/src/main/java/org/apache/flume/sink/AvroSink.java
flume-ng-core/src/main/java/org/apache/flume/sink/SinkGroup.java
flume-ng-core/src/main/java/org/apache/flume/source/SyslogParser.java
flume-ng-core/src/main/java/org/apache/flume/source/SyslogUtils.java
flume-ng-core/src/main/java/org/apache/flume/source/shaded/guava/RateLimiter.java
flume-ng-core/src/test/java/org/apache/flume/TestCounterGroup.java
flume-ng-core/src/test/java/org/apache/flume/client/avro/TestBufferedLineReader.java
flume-ng-core/src/test/java/org/apache/flume/serialization/TestDurablePositionTracker.java
flume-ng-core/src/test/java/org/apache/flume/serialization/TestResettableFileInputStream.java
flume-ng-core/src/test/java/org/apache/flume/sink/TestThriftSink.java
flume-ng-core/src/test/java/org/apache/flume/source/TestPollableSourceRunner.java
flume-ng-core/src/test/java/org/apache/flume/source/TestStressSource.java
flume-ng-core/src/test/java/org/apache/flume/source/TestSyslogUdpSource.java
flume-ng-core/src/test/java/org/apache/flume/source/TestThriftSource.java
flume-ng-core/src/test/java/org/apache/flume/tools/TestFlumeConfigurator.java
flume-ng-dist/pom.xml
flume-ng-dist/src/main/assembly/src.xml
flume-ng-doc/site.xml
flume-ng-doc/sphinx/FlumeUserGuide.rst
flume-ng-embedded-agent/pom.xml
flume-ng-legacy-sources/flume-avro-source/pom.xml
flume-ng-legacy-sources/flume-thrift-source/pom.xml
flume-ng-legacy-sources/flume-thrift-source/src/main/java/org/apache/flume/source/thriftLegacy/ThriftLegacySource.java
flume-ng-legacy-sources/pom.xml
flume-ng-node/pom.xml
flume-ng-sdk/pom.xml
flume-ng-sdk/src/main/java/org/apache/flume/api/FailoverRpcClient.java
flume-ng-sdk/src/main/java/org/apache/flume/api/RpcClient.java
flume-ng-sdk/src/main/java/org/apache/flume/api/RpcClientFactory.java
flume-ng-sinks/flume-dataset-sink/pom.xml
flume-ng-sinks/flume-hdfs-sink/pom.xml
flume-ng-sinks/flume-hive-sink/pom.xml
flume-ng-sinks/flume-http-sink/pom.xml
flume-ng-sinks/flume-http-sink/src/test/java/org/apache/flume/sink/http/TestHttpSink.java
flume-ng-sinks/flume-irc-sink/pom.xml
flume-ng-sinks/flume-ng-elasticsearch-sink/pom.xml
flume-ng-sinks/flume-ng-elasticsearch-sink/src/main/java/org/apache/flume/sink/elasticsearch/ContentBuilderUtil.java
flume-ng-sinks/flume-ng-hbase-sink/pom.xml
flume-ng-sinks/flume-ng-hbase-sink/src/test/java/org/apache/flume/sink/hbase/TestAsyncHBaseSink.java
flume-ng-sinks/flume-ng-hbase-sink/src/test/java/org/apache/flume/sink/hbase/TestHBaseSink.java
flume-ng-sinks/flume-ng-hbase2-sink/pom.xml
flume-ng-sinks/flume-ng-hbase2-sink/src/test/java/org/apache/flume/sink/hbase2/TestHBase2Sink.java
flume-ng-sinks/flume-ng-kafka-sink/pom.xml
flume-ng-sinks/flume-ng-kafka-sink/src/test/java/org/apache/flume/sink/kafka/util/TestUtil.java
flume-ng-sinks/flume-ng-morphline-solr-sink/pom.xml
flume-ng-sinks/flume-ng-morphline-solr-sink/src/test/resources/solr/collection1/conf/schema.xml
flume-ng-sinks/pom.xml
flume-ng-sources/flume-jms-source/pom.xml
flume-ng-sources/flume-kafka-source/pom.xml
flume-ng-sources/flume-scribe-source/pom.xml
flume-ng-sources/flume-taildir-source/pom.xml
flume-ng-sources/flume-taildir-source/src/test/java/org/apache/flume/source/taildir/TestTaildirEventReader.java
flume-ng-sources/flume-twitter-source/pom.xml
flume-ng-sources/flume-twitter-source/src/main/java/org/apache/flume/source/twitter/TwitterSource.java
flume-ng-sources/flume-twitter-source/src/test/java/org/apache/flume/source/twitter/TestTwitterSource.java
flume-ng-sources/pom.xml
flume-ng-tests/pom.xml
flume-ng-tests/src/test/java/org/apache/flume/test/agent/TestSpooldirSource.java
flume-shared/flume-shared-kafka-test/pom.xml
flume-shared/pom.xml
flume-tools/pom.xml
mvnw [new file with mode: 0755]
mvnw.cmd [new file with mode: 0755]
pom.xml