Re: [PR] Push the runtime filter from HashJoin down to SeqScan or AM. [cloudberry]

["zhangyue-hashdata (via GitHub)" <gi...@apache.org>]

zhangyue-hashdata commented on PR #724:
URL: https://github.com/apache/cloudberry/pull/724#issuecomment-2567252216

   > from tpcds 10s details table, there are some bad cases.
   
   **21,24,30,42,49,54,68-1,99**
   
   I retested these SQL statements that exhibited performance regression, and the latest test results show no noticeable performance difference when toggling **gp_enable_runtime_filter_pushdown**. So, I speculate that the performance regression in these SQL statements might be associated with testing method. Previously I tested by running the entire suite of 99 TPC-DS queries with **gp_enable_runtime_filter_pushdown** enabled, and then again with it disabled.
   
   Therefore, a more appropriate method would be to execute the same SQL statement multiple times with **gp_enable_runtime_filter_pushdown** both enabled and disabled, respectively, and then take the average of those runs for comparison. I will follow this testing method for retesting and observe if there's any performance regression.


-- 
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: commits-unsubscribe@cloudberry.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cloudberry.apache.org
For additional commands, e-mail: commits-help@cloudberry.apache.org