Kamal Chandraprakash
Kamal Chandraprakash
Thanks for the patch @jeqo! I was able to reproduce the exception mentioned in the ticket by calling close in-middle of collecting the aborted transactions. The fix doesn't resolve the...
@jeqo Any updates on this PR?
The patch LGTM. It needs second set of eyes as we are re-opening a closed index. cc @showuon @satishd PTAL.
@aheev Thanks for the PR! Could you add a comment to understand why PR#14136 was abandoned and how this PR addresses it?
@jiafu1115 The already uploaded segments are eligible for deletion from broker. So, when remote storage is down, then those segments can be deleted as per the local retention settings and...
Nice find! Thanks for the patch, adding new configs require KIP to proceed.
@showuon Since this qualify as a bug, shall we go-ahead with internal config? And, open a KIP in-parallel to expose the config to users. WDYT?
Thanks for fixing this issue!
It is good to document this change for users. When running from CLI, we have to include a trailing slash to escape the dollar: The below one will change the...
Thanks for the PR! Went over first pass, have below questions: > Startup race condition: RemoteLogManager initialized with default quotas (Long.MAX_VALUE = unlimited) and relied on dynamic config updates to...