xiedeyantu commented on PR #5408: URL: https://github.com/apache/calcite/pull/5408#issuecomment-6798612032
> [@xiedeyantu](https://github.com/xiedeyantu) what have you observed locally in terms of execution time of the problematic test before and after the patch? btw, could you please check who's been working on the TopDownGeneralDecorrelator and request them a review here? @rubenada I reran the problematic query locally with top-down decorrelation enabled (macOS ARM64, JDK 17.0.12, -Xmx1536m). In fresh JVMs, the first execution took about 29.9 s before the patch and 1.13 s after it. These timings include planning, execution, and fetching results, excluding JVM startup and connection setup. Both returned the expected two rows. I know this part of the code was written by @silundong , with @mihaibudiu as the reviewer. I’m not sure if they have the time to fix this bug or review this PR. -- 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]
