> Can you please consider evaluating if Auditor.checkAllLedgers needs its own > set of resources or not? If it needs its own set of resources then is it ok > to go partial path?
for current code, it needs its own zk, otherwise it is self-deadlock. after my change #1619, it doesn't need its own resources. > So AFAIU this issue is not completely found because of the fact that partial > new resource creation is allowed in original commit the problem will be gone if we don't call sync calls in async callbacks. so my fix #1619 should be the correct fix. please take a look. [ Full content available at: https://github.com/apache/bookkeeper/pull/1608 ] This message was relayed via gitbox.apache.org for [email protected]
