You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@curator.apache.org by "tisonkun (via GitHub)" <gi...@apache.org> on 2023/09/14 18:08:28 UTC
[GitHub] [curator] tisonkun opened a new pull request, #480: CURATOR-690. CuratorCache fails to load the cache if there are more than 64k child znodes
tisonkun opened a new pull request, #480:
URL: https://github.com/apache/curator/pull/480
This closes https://github.com/apache/curator/pull/479.
--
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: commits-unsubscribe@curator.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [curator] tisonkun merged pull request #480: CURATOR-690. CuratorCache fails to load the cache if there are more than 64k child znodes
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun merged PR #480:
URL: https://github.com/apache/curator/pull/480
--
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: commits-unsubscribe@curator.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [curator] tisonkun commented on pull request #480: CURATOR-690. CuratorCache fails to load the cache if there are more than 64k child znodes
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on PR #480:
URL: https://github.com/apache/curator/pull/480#issuecomment-1720343712
The revert is trivial and the previous solution is reviewed.
Merging...
--
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: commits-unsubscribe@curator.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [curator] kezhuw commented on pull request #480: CURATOR-690. CuratorCache fails to load the cache if there are more than 64k child znodes
Posted by "kezhuw (via GitHub)" <gi...@apache.org>.
kezhuw commented on PR #480:
URL: https://github.com/apache/curator/pull/480#issuecomment-1720380459
Late, but +1 on this reverting.
Tiered phaser may be more suitable in high contention cases, `OutstandingOps` is simple enough for us to evaluate and the cache loading is one-shot. So we don't have to pay extra complexity coming from tiering.
Thank you for your contribution! @Ryan0751
--
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: commits-unsubscribe@curator.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org