flyrain commented on PR #3256: URL: https://github.com/apache/polaris/pull/3256#issuecomment-3752448023
Thanks for the update. For those who have not been following the dev mailing list, I recommend reviewing the discussion there. A good entry point is https://lists.apache.org/thread/dzzdg5l3xoqgysdmp7mh3y8mp59w31tl, though you are welcome to read the thread from the beginning for full context. This PR was originally filed to address the issue described in https://github.com/apache/polaris/issues/2365. After further investigation, it appears that this was a false alarm. Pierre later pointed out that there may be other factors contributing to memory footprint pressure in certain use cases, as described here https://github.com/apache/polaris/issues/2365#issuecomment-3723184230. Those concerns can be addressed independently via a small change proposed in https://github.com/apache/polaris/pull/3415, please take a look to see if that solves the problem. It'd be really nice to run this with the same setup we used to validate the current PR which is this PR fixed the issue. If #3415 solved the issue, can we use it until we settle with permanent solution(delegation service)? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
