[HBASE-8288] HBaseFileSystem: Refactoring and correct semantics for createPath methods

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.94.6
• Fix Version/s: 0.94.7
• Component/s: Filesystem Integration
• Assignee: Himanshu Vashishtha
• Reporter: Himanshu Vashishtha
• Created: 08/Apr/13 05:36
• Updated: 27/Apr/13 15:55
• Resolved: 08/Apr/13 19:53

Description

This jira is for two issues I see in the HBaseFileSystem class:
1) Load testing on a 7 node cluster using ycsb insert workload shows that static initialization of conf properties results in a slightly better throughput. Though the initialization uses HBaseConfiguration.create() call which is expensive (and I tried to avoid that in its first version), this class is used for most of the filesystem class, and had to invoke an additional checkAndSetXX call before making the fs call because it is not certain whether the retry properties are set or not. Having initialize them in static block removes that limitation.

2) Correct semantics for CreatePathXXX method. In case the overwrite flag is false and file already exists, underlying fs throws an exception. It should be re-thrown to the caller.