Gabriel Drouin
Gabriel Drouin
Opened a PR for this issue [here](https://github.com/linkedin/venice/pull/1734).
NB: I believe this change could lead to race conditions due to `Utils.sleep`, although there are other methods in `HelixUtils` which are using `Utils.sleep` as well, and all methods using...
> > NB: I believe this change could lead to race conditions due to Utils.sleep, although there are other methods in HelixUtils which are using Utils.sleep as well, and all...
> > NB: I believe this change could lead to race conditions due to Utils.sleep, although there are other methods in HelixUtils which are using Utils.sleep as well, and all...
Thank you for the review @kvargha. I have commited my changes based on your suggestions. Please let me know of any possible further improvements.
Thank you @kvargha. Changes have been applied in 99b5fa7. Please see my comment [here](https://github.com/linkedin/venice/pull/1734#discussion_r2067717442) regarding implementation details for `ZkDataAccessException`.
Quick note here that we discussed implementing the exponential backoff for other methods in `HelixUtils` and corresponding tests in `TestHelixUtils`, which will be done in another PR once this current...
@kvargha > Code looks good to me, but it looks like SpotBugs is failing. > > Try running this locally and look for the HTML file path it outputs detailing...