[OAK-3164] MemoryNodeStore issues duplicate checkpoint

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.0.18, 1.2.4, 1.3.4, 1.4
• Component/s: core
• Assignee: Chetan Mehrotra
• Reporter: Chetan Mehrotra
• Created: 29/Jul/15 14:42
• Updated: 02/Mar/16 12:01
• Resolved: 31/Jul/15 06:55

Description

MemoryNodeStore can possibly issue duplicate checkpoint. In below logic if checkpoints has one entry then on subsequent call to issue checkpoint the name would be set to checkpoint1 only. This causes AsyncIndexUpdate test to fail in some cases

    @Override
    public String checkpoint(long lifetime, @Nonnull Map<String, String> properties) {
        checkArgument(lifetime > 0);
        checkNotNull(properties);
        String checkpoint = "checkpoint" + checkpoints.size();
        checkpoints.put(checkpoint, new Checkpoint(getRoot(), properties));
        return checkpoint;
    }

Issue Links

• blocks: OAK-3155 AsyncIndex stats do not capture execution for runs where no indexing is performed