[CURATOR-329] SharedValue.start() method does not update to current value if version of value currently stored in zookeeper is 0 (zero)

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 2.10.0
• Fix Version/s: 2.11.0, 3.2.0
• Component/s: Recipes
• Assignee: Unassigned
• Reporter: Algirdas Rascius
• Created: 24/May/16 13:37
• Updated: 24/May/16 22:53
• Resolved: 24/May/16 22:53

Description

When zookeeper node stores value with version 0 and SharedCount instance is created and started for this node, getCount() method returns seedValue passed to SharedCount constructor instead of actual value stored in zookeeper node. This only happens when zookeeper node version is 0 (zero).

For example:

// Node /count does not exists before the test
SharedCount count1 = new SharedCount(client, "/count", 10);
SharedCount count2 = new SharedCount(client, "/count", 20);
count1.start();
count2.start();
Assert.assertEquals(count1.getCount(), 10);
Assert.assertEquals(count2.getCount(), 10); // Should return 10, but returns 20 instead

Issue arrises because when constructed class SharedValue initializes instance variable currentValue with version 0 and does not update value in updateValue(..) method if version returned by zookeeper is 0.

Issue Links

• links to: GitHub Pull Request #153