You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by wh...@apache.org on 2014/04/09 20:10:37 UTC

svn commit: r1586059 - in /hadoop/common/trunk/hadoop-tools/hadoop-streaming: ./ dev-support/ src/main/java/org/apache/hadoop/record/ src/main/java/org/apache/hadoop/record/compiler/ src/main/java/org/apache/hadoop/record/meta/

Author: wheat9
Date: Wed Apr  9 18:10:36 2014
New Revision: 1586059

URL: http://svn.apache.org/r1586059
Log:
HADOOP-10485. Remove dead classes in hadoop-streaming. Contributed by Haohui Mai.

Removed:
    hadoop/common/trunk/hadoop-tools/hadoop-streaming/src/main/java/org/apache/hadoop/record/CsvRecordInput.java
    hadoop/common/trunk/hadoop-tools/hadoop-streaming/src/main/java/org/apache/hadoop/record/XmlRecordInput.java
    hadoop/common/trunk/hadoop-tools/hadoop-streaming/src/main/java/org/apache/hadoop/record/XmlRecordOutput.java
    hadoop/common/trunk/hadoop-tools/hadoop-streaming/src/main/java/org/apache/hadoop/record/compiler/
    hadoop/common/trunk/hadoop-tools/hadoop-streaming/src/main/java/org/apache/hadoop/record/meta/
Modified:
    hadoop/common/trunk/hadoop-tools/hadoop-streaming/dev-support/findbugs-exclude.xml
    hadoop/common/trunk/hadoop-tools/hadoop-streaming/pom.xml

Modified: hadoop/common/trunk/hadoop-tools/hadoop-streaming/dev-support/findbugs-exclude.xml
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-tools/hadoop-streaming/dev-support/findbugs-exclude.xml?rev=1586059&r1=1586058&r2=1586059&view=diff
==============================================================================
--- hadoop/common/trunk/hadoop-tools/hadoop-streaming/dev-support/findbugs-exclude.xml (original)
+++ hadoop/common/trunk/hadoop-tools/hadoop-streaming/dev-support/findbugs-exclude.xml Wed Apr  9 18:10:36 2014
@@ -20,7 +20,6 @@
   <Match>
     <Or>
       <Package name="org.apache.hadoop.streaming" />
-      <Class name="org.apache.hadoop.record.compiler.JFile" />
     </Or>
     <Bug pattern="NP_ALWAYS_NULL"/>
   </Match>
@@ -42,10 +41,6 @@
   </Match>
 
   <Match>
-    <Package name="org.apache.hadoop.record.compiler.generated" />
-  </Match>
-
-  <Match>
     <Package name="org.apache.hadoop.record" />
     <Or>
       <Bug pattern="EI_EXPOSE_REP" />
@@ -53,19 +48,4 @@
       <Bug pattern="MS_PKGPROTECT" />
     </Or>
   </Match>
-
-  <Match>
-    <Class name="org.apache.hadoop.record.meta.Utils" />
-    <Method name="skip" />
-    <Bug pattern="BC_UNCONFIRMED_CAST" />
-  </Match>
-
-  <!--
-     The compareTo method is actually a dummy method that just
-     throws excpetions. So, no need to override equals. Ignore
-  -->
-  <Match>
-    <Class name="org.apache.hadoop.record.meta.RecordTypeInfo" />
-    <Bug pattern="EQ_COMPARETO_USE_OBJECT_EQUALS" />
-  </Match>
 </FindBugsFilter>

Modified: hadoop/common/trunk/hadoop-tools/hadoop-streaming/pom.xml
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hadoop-tools/hadoop-streaming/pom.xml?rev=1586059&r1=1586058&r2=1586059&view=diff
==============================================================================
--- hadoop/common/trunk/hadoop-tools/hadoop-streaming/pom.xml (original)
+++ hadoop/common/trunk/hadoop-tools/hadoop-streaming/pom.xml Wed Apr  9 18:10:36 2014
@@ -42,11 +42,6 @@
       <scope>test</scope>
     </dependency>
     <dependency>
-      <groupId>org.apache.ant</groupId>
-      <artifactId>ant</artifactId>
-      <scope>provided</scope>
-    </dependency>
-    <dependency>
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-annotations</artifactId>
       <scope>provided</scope>