j1wonpark commented on PR #4320: URL: https://github.com/apache/amoro/pull/4320#issuecomment-5324119984
Thanks for the fix. The NPE is real and this stops it. One thought on the test: it calls `registerOptimizers` directly, so it skips `loadOptimizingQueues` where the NPE actually happens. I tried restoring `optimizers.forEach(...)` while keeping the new method, and the bug comes back but the test still passes. -- 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]
