fix stats for runtime filter in some cases ___ ### **PR Type** Bug fix ___ ### **Description** - Added calls to `recalcStatsByRuntimeFilter` in various functions to ensure stats are recalculated after applying runtime filters. - Refactored `recalcStatsByRuntimeFilter` to use `builder` for selectivity calculation instead of passing `runtimeFilterSel`. - Adjusted stats recalculation logic for scan and join nodes in `recalcStatsByRuntimeFilter`. ___ ### **Changes walkthrough** 📝 <table><thead><tr><th></th><th align="left">Relevant files</th></tr></thead><tbody><tr><td><strong>Bug fix </strong></td><td><table> <tr> <td> <details> <summary><strong>apply_indices.go</strong><dd><code>Add stats recalculation for runtime filter in applyIndicesForJoins</code></dd></summary> <hr> pkg/sql/plan/apply_indices.go <li>Added call to <code>recalcStatsByRuntimeFilter</code> after appending runtime <br>filter to <code>RuntimeFilterBuildList</code>.<br> </details> </td> <td><a href="https://github.com/matrixorigin/matrixone/pull/16491/files#diff-b92e434ebc0c08092943a8c5a8abaed315f3e5722db3ffde4e3fbbb878d325b2">+1/-0</a>      </td> </tr> <tr> <td> <details> <summary><strong>build_constraint_util.go</strong><dd><code>Add stats recalculation for runtime filter in build constraint utility</code></dd></summary> <hr> pkg/sql/plan/build_constraint_util.go <li>Added call to <code>recalcStatsByRuntimeFilter</code> in <code>appendPrimaryConstrantPlan</code> <br>function.<br> </details> </td> <td><a href="https://github.com/matrixorigin/matrixone/pull/16491/files#diff-6eb56175b52eb3302f1d9ae1cde88ced82c25bb8dd90562238cc8d9757adb76b">+2/-0</a>      </td> </tr> <tr> <td> <details> <summary><strong>build_dml_util.go</strong><dd><code>Add stats recalculation for runtime filter in DML utility</code></dd></summary> <hr> pkg/sql/plan/build_dml_util.go <li>Added call to <code>recalcStatsByRuntimeFilter</code> in <code>appendDeleteIndexTablePlan</code> <br>function.<br> </details> </td> <td><a href="https://github.com/matrixorigin/matrixone/pull/16491/files#diff-095fb233d51021791cb24454839b013236680bbc6bbc22e0d2f6741ac8fe7dff">+1/-0</a>      </td> </tr> <tr> <td> <details> <summary><strong>runtime_filter.go</strong><dd><code>Update stats recalculation method calls in runtime filter</code></dd></summary> <hr> pkg/sql/plan/runtime_filter.go <li>Modified calls to <code>recalcStatsByRuntimeFilter</code> to use <code>builder</code> instead of <br><code>runtimeFilterSel</code>.<br> </details> </td> <td><a href="https://github.com/matrixorigin/matrixone/pull/16491/files#diff-597fe1a4b6c932d815482502c1ae60c5f8c69921e00b27dba2f5e4ea5b25b54b">+2/-3</a>      </td> </tr> <tr> <td> <details> <summary><strong>stats.go</strong><dd><code>Refactor stats recalculation logic in stats utility</code>            </dd></summary> <hr> pkg/sql/plan/stats.go <li>Refactored <code>recalcStatsByRuntimeFilter</code> to use <code>builder</code> for selectivity <br>calculation.<br> <li> Adjusted stats recalculation logic for scan and join nodes.<br> </details> </td> <td><a href="https://github.com/matrixorigin/matrixone/pull/16491/files#diff-3b3d55fa9884dcf8980f90043a05b26a04d0153ae89fcf032cec998752c0cafa">+11/-7</a>    </td> </tr> </table></td></tr></tr></tbody></table> ___ > 💡 **PR-Agent usage**: >Comment `/help` on the PR to get a list of all available PR-Agent tools and their descriptions Approved by: @ouyuanning

Commits

Commits on Jun 21, 2024

fix some little bug of stats (matrixorigin#17035)

fix stats for runtime filter in some cases (matrixorigin#16491)

fix stats for prefix_eq function (matrixorigin#16666)

fix

fix a bug

Merge branch '1.2-dev' into stats1.2

Merge branch '1.2-dev' into stats1.2

fix some little bug of stats (matrixorigin#17035 )

fix stats for runtime filter in some cases (matrixorigin#16491 )

fix stats for prefix_eq function (matrixorigin#16666 )