metesynnada commented on PR #6679: URL: https://github.com/apache/arrow-datafusion/pull/6679#issuecomment-1596713815
My results are aligned with yours, ``` ┏━━━━━━━━━━━━━━┳━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┓ ┃ Query ┃ main ┃ adapt_datastructure ┃ Change ┃ ┡━━━━━━━━━━━━━━╇━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━┩ │ QQuery 1 │ 382.88ms │ 384.07ms │ no change │ │ QQuery 2 │ 124.07ms │ 116.29ms │ +1.07x faster │ │ QQuery 3 │ 174.73ms │ 174.30ms │ no change │ │ QQuery 4 │ 110.42ms │ 110.51ms │ no change │ │ QQuery 5 │ 227.84ms │ 226.10ms │ no change │ │ QQuery 6 │ 108.11ms │ 109.13ms │ no change │ │ QQuery 7 │ 394.28ms │ 352.03ms │ +1.12x faster │ │ QQuery 8 │ 256.21ms │ 253.69ms │ no change │ │ QQuery 9 │ 403.20ms │ 385.42ms │ no change │ │ QQuery 10 │ 299.90ms │ 299.38ms │ no change │ │ QQuery 11 │ 88.98ms │ 82.60ms │ +1.08x faster │ │ QQuery 12 │ 202.39ms │ 203.64ms │ no change │ │ QQuery 13 │ 439.47ms │ 420.72ms │ no change │ │ QQuery 14 │ 151.04ms │ 150.24ms │ no change │ │ QQuery 15 │ 114.99ms │ 114.34ms │ no change │ │ QQuery 16 │ 126.88ms │ 90.89ms │ +1.40x faster │ │ QQuery 17 │ 955.93ms │ 942.46ms │ no change │ │ QQuery 18 │ 1183.84ms │ 979.92ms │ +1.21x faster │ │ QQuery 19 │ 311.80ms │ 312.30ms │ no change │ │ QQuery 20 │ 353.71ms │ 338.11ms │ no change │ │ QQuery 21 │ 634.34ms │ 539.71ms │ +1.18x faster │ │ QQuery 22 │ 77.22ms │ 76.97ms │ no change │ └──────────────┴───────────┴─────────────────────┴───────────────┘ ``` Currently, I am reviewing the code. The algorithm is neat, however, I want to find a good way to integrate the symmetric hash join into this algorithm. I think it can be possible. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
