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]

Reply via email to