[CAMEL-8768] hdfs2 component overwrite option is also being applied to directory filesystem path

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.15.2
• Fix Version/s: 2.15.3, 2.16.0
• Component/s: camel-hdfs
• Labels: hadoop
• Estimated Complexity: Unknown
• Flags: Patch
• Assignee: Claus Ibsen
• Reporter: Hassan Faouaz
• Created: 12/May/15 14:45
• Updated: 09/Jul/15 10:08
• Resolved: 09/Jul/15 10:08

Description

If you need to produce files into an existing HDFS2 path, the default behavoir is overwrite the path, which will delete all existing files on HDFS. If overwrite option is disabled, then the component will complain that the existing HDFS directory exists and will not work.

The propose solution is to add the following if statement to ignore check if the HDFS directory exists. The overwrite option should only be used for files not directories.

code snippet in HdfsOutputStream.java and patch is attached

if (ret.info.getFileSystem().exists(new Path(ret.actualPath))) {
//only check of not directory
if (!ret.info.getFileSystem().isDirectory(new Path(ret.actualPath))) {
if (configuration.isOverwrite())

{ ret.info.getFileSystem().delete(new Path(ret.actualPath), true); }

else

{ throw new RuntimeCamelException("The file already exists"); }

}
}