You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Israel Ekpo <is...@aicer.org> on 2013/04/29 07:18:52 UTC

Review Request: FLUME-2031 Removal of Unused Imports from Project

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10830/
-----------------------------------------------------------

Review request for Flume.


Description
-------

FLUME-2031 Clean Up Task - Removal of Unused Imports from Project

modified:   flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/CheckpointRebuilder.java
modified:   flume-ng-clients/flume-ng-log4jappender/src/main/java/org/apache/flume/clients/log4jappender/Log4jAppender.java
modified:   flume-ng-configuration/src/main/java/org/apache/flume/conf/channel/ChannelConfiguration.java
modified:   flume-ng-configuration/src/main/java/org/apache/flume/conf/channel/ChannelSelectorConfiguration.java
modified:   flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkProcessorConfiguration.java
modified:   flume-ng-core/src/main/java/org/apache/flume/ChannelSelector.java
modified:   flume-ng-core/src/main/java/org/apache/flume/SinkProcessor.java
modified:   flume-ng-core/src/main/java/org/apache/flume/channel/ReplicatingChannelSelector.java
modified:   flume-ng-core/src/main/java/org/apache/flume/serialization/LineDeserializer.java
modified:   flume-ng-core/src/main/java/org/apache/flume/sink/AbstractRpcSink.java
modified:   flume-ng-core/src/main/java/org/apache/flume/sink/AvroSink.java
modified:   flume-ng-core/src/main/java/org/apache/flume/sink/SinkGroup.java
modified:   flume-ng-core/src/main/java/org/apache/flume/source/SyslogParser.java
modified:   flume-ng-core/src/test/java/org/apache/flume/client/avro/TestReliableSpoolingFileEventReader.java
modified:   flume-ng-core/src/test/java/org/apache/flume/instrumentation/TestMonitoredCounterGroup.java
modified:   flume-ng-core/src/test/java/org/apache/flume/serialization/TestDurablePositionTracker.java
modified:   flume-ng-core/src/test/java/org/apache/flume/serialization/TestResettableFileInputStream.java
modified:   flume-ng-core/src/test/java/org/apache/flume/sink/TestAvroSink.java
modified:   flume-ng-core/src/test/java/org/apache/flume/source/TestAvroSource.java
modified:   flume-ng-core/src/test/java/org/apache/flume/source/TestPollableSourceRunner.java
modified:   flume-ng-core/src/test/java/org/apache/flume/source/TestStressSource.java
modified:   flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/EventStatus.java
modified:   flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/Priority.java
modified:   flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEvent.java
modified:   flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEventServer.java
modified:   flume-ng-legacy-sources/flume-thrift-source/src/main/java/org/apache/flume/source/thriftLegacy/ThriftLegacySource.java
modified:   flume-ng-legacy-sources/flume-thrift-source/src/test/java/org/apache/flume/source/thriftLegacy/TestThriftLegacySource.java
modified:   flume-ng-sdk/src/main/java/org/apache/flume/api/FailoverRpcClient.java
modified:   flume-ng-sdk/src/main/java/org/apache/flume/api/NettyAvroRpcClient.java
modified:   flume-ng-sdk/src/main/java/org/apache/flume/thrift/Status.java
modified:   flume-ng-sdk/src/main/java/org/apache/flume/thrift/ThriftFlumeEvent.java
modified:   flume-ng-sdk/src/main/java/org/apache/flume/thrift/ThriftSourceProtocol.java
modified:   flume-ng-sdk/src/test/java/org/apache/flume/api/TestNettyAvroRpcClient.java
modified:   flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSink.java
modified:   flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java
modified:   flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestUseRawLocalFileSystem.java
modified:   flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/LogEntry.java
modified:   flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ResultCode.java
modified:   flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/Scribe.java
modified:   flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ScribeSource.java


This addresses bug FLUME-2031.
    https://issues.apache.org/jira/browse/FLUME-2031


Diffs
-----

  flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/CheckpointRebuilder.java 7883d0e 
  flume-ng-clients/flume-ng-log4jappender/src/main/java/org/apache/flume/clients/log4jappender/Log4jAppender.java d61f807 
  flume-ng-configuration/src/main/java/org/apache/flume/conf/channel/ChannelConfiguration.java 1e1a46f 
  flume-ng-configuration/src/main/java/org/apache/flume/conf/channel/ChannelSelectorConfiguration.java e68e129 
  flume-ng-configuration/src/main/java/org/apache/flume/conf/sink/SinkProcessorConfiguration.java 9ce0058 
  flume-ng-core/src/main/java/org/apache/flume/ChannelSelector.java f86aec7 
  flume-ng-core/src/main/java/org/apache/flume/SinkProcessor.java c0c7c29 
  flume-ng-core/src/main/java/org/apache/flume/channel/ReplicatingChannelSelector.java 8a0d2bd 
  flume-ng-core/src/main/java/org/apache/flume/serialization/LineDeserializer.java d8e5ae2 
  flume-ng-core/src/main/java/org/apache/flume/sink/AbstractRpcSink.java 892c949 
  flume-ng-core/src/main/java/org/apache/flume/sink/AvroSink.java 34f9fa6 
  flume-ng-core/src/main/java/org/apache/flume/sink/SinkGroup.java d23f9ea 
  flume-ng-core/src/main/java/org/apache/flume/source/SyslogParser.java bf3305c 
  flume-ng-core/src/test/java/org/apache/flume/client/avro/TestReliableSpoolingFileEventReader.java 9d708c1 
  flume-ng-core/src/test/java/org/apache/flume/instrumentation/TestMonitoredCounterGroup.java e417fb3 
  flume-ng-core/src/test/java/org/apache/flume/serialization/TestDurablePositionTracker.java e52affb 
  flume-ng-core/src/test/java/org/apache/flume/serialization/TestResettableFileInputStream.java 5ad6a0a 
  flume-ng-core/src/test/java/org/apache/flume/sink/TestAvroSink.java ac47ee9 
  flume-ng-core/src/test/java/org/apache/flume/source/TestAvroSource.java c699241 
  flume-ng-core/src/test/java/org/apache/flume/source/TestPollableSourceRunner.java 4d4222d 
  flume-ng-core/src/test/java/org/apache/flume/source/TestStressSource.java 28270f4 
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/EventStatus.java 7af9e77 
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/Priority.java 40826dc 
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEvent.java f83cba4 
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/com/cloudera/flume/handlers/thrift/ThriftFlumeEventServer.java ff32c45 
  flume-ng-legacy-sources/flume-thrift-source/src/main/java/org/apache/flume/source/thriftLegacy/ThriftLegacySource.java 47913dc 
  flume-ng-legacy-sources/flume-thrift-source/src/test/java/org/apache/flume/source/thriftLegacy/TestThriftLegacySource.java 8e08f22 
  flume-ng-sdk/src/main/java/org/apache/flume/api/FailoverRpcClient.java db6905a 
  flume-ng-sdk/src/main/java/org/apache/flume/api/NettyAvroRpcClient.java 8285129 
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/Status.java 634c323 
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/ThriftFlumeEvent.java fc0b678 
  flume-ng-sdk/src/main/java/org/apache/flume/thrift/ThriftSourceProtocol.java 7f966b0 
  flume-ng-sdk/src/test/java/org/apache/flume/api/TestNettyAvroRpcClient.java 1e6d2b2 
  flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSink.java 5b7cec9 
  flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestHDFSEventSinkOnMiniCluster.java 6e11624 
  flume-ng-sinks/flume-hdfs-sink/src/test/java/org/apache/flume/sink/hdfs/TestUseRawLocalFileSystem.java ffbdde0 
  flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/LogEntry.java be703a9 
  flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ResultCode.java 210e21e 
  flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/Scribe.java eba8d2e 
  flume-ng-sources/flume-scribe-source/src/main/java/org/apache/flume/source/scribe/ScribeSource.java b3a8de3 

Diff: https://reviews.apache.org/r/10830/diff/


Testing
-------

After the changes: mvn clean install works successfully with no errors.


Thanks,

Israel Ekpo