adnanhemani commented on PR #1965: URL: https://github.com/apache/polaris/pull/1965#issuecomment-3169274628
Thanks for this, @RussellSpitzer. 1. I added the additional retries for redundancy "best-effort". But I'm also aligned if we don't do this manually and leave it to the AWK SDK. 2. I'm not opposed to not tracking the futures that we added to the ExecutorService - but I had been previously chided for not handling these in other PRs in this repo (not sure I remember by who). @snazy - can you please add your thoughts on this? I'm glad to remove the `futures` ConcurrentHashMap if we do not need to track the futures. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
