[OAK-4992] Use the role name as prefix for directory used by SegmentNodeStoreFactory

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.5.13, 1.6.0
• Component/s: segment-tar, segmentmk
• Assignee: Chetan Mehrotra
• Reporter: Chetan Mehrotra
• Created: 25/Oct/16 05:22
• Updated: 14/Nov/16 11:35
• Resolved: 25/Oct/16 14:55

Description

Currently SegmentNodeStoreFactory uses a directory named segmentstore under repository home to store the segment files. To allow easier setup where multiple NodeStore instance do not overstep on each other this name should be prefixed with role

Note that code tries to do the same currently with logic below but it ends up always creating segmentstore folder under repository home. As a fix role should be used in getDirectory method

    private File getBaseDirectory() {
        String directory = property(DIRECTORY);

        if (directory != null) {
            return new File(directory);
        }

        if (role == null) {
            return new File("tarmk");
        } else {
            return new File("tarmk-" + role);
        }
    }

    private File getDirectory() {
        return new File(getBaseDirectory(), "segmentstore");
    }