You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flume.apache.org by pr...@apache.org on 2011/12/08 00:27:05 UTC

svn commit: r1211704 - /incubator/flume/branches/flume-728/flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSWriterFactory.java

Author: prasadm
Date: Wed Dec  7 23:27:05 2011
New Revision: 1211704

URL: http://svn.apache.org/viewvc?rev=1211704&view=rev
Log:
FLUME-858. Fix comparison in HDFSWriterFactory
(Jarek Jarcec Cecho via Prasad Mujumdar)
.

Modified:
    incubator/flume/branches/flume-728/flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSWriterFactory.java

Modified: incubator/flume/branches/flume-728/flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSWriterFactory.java
URL: http://svn.apache.org/viewvc/incubator/flume/branches/flume-728/flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSWriterFactory.java?rev=1211704&r1=1211703&r2=1211704&view=diff
==============================================================================
--- incubator/flume/branches/flume-728/flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSWriterFactory.java (original)
+++ incubator/flume/branches/flume-728/flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/HDFSWriterFactory.java Wed Dec  7 23:27:05 2011
@@ -30,11 +30,11 @@ public class HDFSWriterFactory {
   }
 
   public HDFSWriter getWriter(String fileType) throws IOException {
-    if (fileType == SequenceFileType) {
+    if (fileType.equals(SequenceFileType)) {
       return new HDFSSequenceFile();
-    } else if (fileType == DataStreamType) {
+    } else if (fileType.equals(DataStreamType)) {
       return new HDFSDataStream();
-    } else if (fileType == CompStreamType) {
+    } else if (fileType.equals(CompStreamType)) {
       return new HDFSCompressedDataStream();
     } else {
       throw new IOException("File type " + fileType + " not supported");