GGraziadei opened a new pull request, #8714:
URL: https://github.com/apache/storm/pull/8714
## What is the purpose of the change
Refactoring ZooKeeper reads to optimize request volume. Removed unnecessary
`.exists()` calls by handling missing nodes reactively. More details in the
linked issue.
Proposed reading pattern
```
GIVEN path, wantWatch:
result <- READ path (requesting a watch if wantWatch) // request #1
if result is present:
return result.value // watch already set if requested
else: // node is absent (e.g. `NoNodeException`)
if wantWatch:
WATCH path // request #2, only here
return EMPTY
on any other error:
FAIL
```
## How was the change tested
- Unit tests
- Added new test cases for node delete-delete conflicts and `gestVersion`
In the context of #8712
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
