[GIRAPH-743] DiskBackedPartitionStore needs file existence checks around file deletion and directory creation

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.1.0
• Fix Version/s: 1.1.0
• Component/s: graph
• Labels: ooc
• Assignee: Unassigned
• Reporter: Craig Muchinsky
• Created: 21/Aug/13 02:09
• Updated: 19/Jan/14 12:32
• Resolved: 19/Jan/14 12:32

Description

The changes made to DiskBackedPartitionStore with GIRAPH-736 introduced return code checks on File.delete() and File.mkdirs() calls, and if they are unsuccessful it is logged as an ERROR. This really only makes sense if the file already exists or the directory doesn't exist respectively. In addition to these checks, the ERROR messages that are logged should use the absolute file path instead of the relative path.

Issue Links

• is part of: GIRAPH-825 Fix DiskBackedPartitionStore to work with current trunk