ydb/yql/essentials/core/cbo, branch main

YQL-20095: Tune bugprone-argument-comment

2026-06-30T14:16:39Z

https://clang.llvm.org/extra/clang-tidy/checks/bugprone/argument-comment.html commit_hash:780ff6a63be44998cb2336a05f3dc33e68a676bf

YQL-20095: Enable performance-move-const-arg

2026-06-24T10:13:06Z

https://clang.llvm.org/extra/clang-tidy/checks/performance/move-const-arg.html This check useful, as it shows, when a `std::move` is useless, so to expectations of a programmer are not aligned with reality. The most useful profit is to see `const T&` arguments, that are potentially can be replaced with `T`. Also disabled `CheckTriviallyCopyableMove`, see https://nda.ya.ru/t/Emvgx5Z67fpZY5 why. commit_hash:c80c264a9774721aabbac85b350ccdbb8b9a39a2

Replace unreachable with *_ENSURE(false, ...)

2026-04-01T09:01:32Z

commit_hash:7c1445375e838a9327006528cb455c9899e9b32d

YQL-20095: Enable readability-avoid-const-params-in-decls

2026-03-31T08:53:33Z

https://clang.llvm.org/extra/clang-tidy/checks/readability/avoid-const-params-in-decls.html commit_hash:17e1ec5c3849a38bcb76cd4927e66979bee6c2ec

YQL-20095: Enable modernize-use-designated-initializers

2026-03-17T08:45:55Z

https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-designated-initializers.html commit_hash:d73eb463c7bec2bda4c362aab10af49979a7ddd3

[CBO] Fix interesting orderings lookup: make it order-sensitive, i.e. shuffle [A, B] != [B, A]

2026-03-17T03:00:22Z

# Description for reviewers Interesting orderings FSM, considers shuffles "not natural" (before this PR), i.e. the order doesn't matter. Since our hash function is order-depended this leads to correctness bugs. For example, this SQL query: ```sql $semi_join = ( SELECT * FROM customer c LEFT SEMI JOIN oorder o ON c.C_W_ID = o.O_W_ID AND c.C_D_ID = o.O_D_ID ); SELECT s.C_W_ID, s.C_D_ID, n.NO_O_ID FROM $semi_join s INNER JOIN new_order as n ON s.C_W_ID = n.NO_W_ID AND s.C_D_ID = n.NO_D_ID; ``` Used to produce this plan: ``` ┌> ResultSet └─┬> InnerJoin (Grace) (c.C_D_ID,c.C_W_ID = n.NO_D_ID,n.NO_W_ID) ├─┬> LeftSemiJoin (Grace) (c.C_D_ID,c.C_W_ID = o.O_W_ID,o.O_D_ID) │ ├─┬> HashShuffle (KeyColumns: ["C_W_ID","C_D_ID"], HashFunc: "HashV2") │ │ └──> TableFullScan (Table: customer, ReadColumns: ["C_W_ID (-∞, +∞)","C_D_ID (-∞, +∞)","C_ID (-∞, +∞)"]) │ └─┬> HashShuffle (KeyColumns: ["O_W_ID","O_D_ID"], HashFunc: "HashV2") │ └──> TableFullScan (Table: oorder, ReadColumns: ["O_W_ID (-∞, +∞)","O_D_ID (-∞, +∞)","O_ID (-∞, +∞)"]) └─┬> HashShuffle (KeyColumns: ["NO_W_ID","NO_D_ID"], HashFunc: "HashV2") └──> TableFullScan (Table: new_order, ReadColumns: ["NO_W_ID (-∞, +∞)","NO_D_ID (-∞, +∞)","NO_O_ID (-∞, +∞)"]) ``` This plan is wrong, because InnerJoin reuses \[C\_W\_ID, C\_D\_ID\] shuffle, when it really needs \[C\_D\_ID, C\_W\_ID\] - this will lead to incorrect result if executed on more than 1 node with overwhelming probability. After this PR, it produces a correct plan ``` ┌> ResultSet └─┬> InnerJoin (Grace) (c.C_D_ID,c.C_W_ID = n.NO_D_ID,n.NO_W_ID) ├─┬> HashShuffle (KeyColumns: ["c.C_D_ID","c.C_W_ID"], HashFunc: "HashV2") │ └─┬> LeftSemiJoin (Grace) (c.C_D_ID,c.C_W_ID = o.O_W_ID,o.O_D_ID) │ ├─┬> HashShuffle (KeyColumns: ["C_W_ID","C_D_ID"], HashFunc: "HashV2") │ │ └──> TableFullScan (Table: customer, ReadColumns: ["C_W_ID (-∞, +∞)","C_D_ID (-∞, +∞)","C_ID (-∞, +∞)"]) │ └─┬> HashShuffle (KeyColumns: ["O_W_ID","O_D_ID"], HashFunc: "HashV2") │ └──> TableFullScan (Table: oorder, ReadColumns: ["O_W_ID (-∞, +∞)","O_D_ID (-∞, +∞)","O_ID (-∞, +∞)"]) └─┬> HashShuffle (KeyColumns: ["NO_D_ID","NO_W_ID"], HashFunc: "HashV2") └──> TableFullScan (Table: new_order, ReadColumns: ["NO_W_ID (-∞, +∞)","NO_D_ID (-∞, +∞)","NO_O_ID (-∞, +∞)"]) ``` This mirrors PR made on github: # Changelog entry Type: fix Component: CBO Make interesting orderings order-sensitive commit_hash:2d8e98048cb669bfc7048a227a8b42504664a273

YQL-20095: Enable modernize-use-bool-literals

2026-03-03T07:39:53Z

commit_hash:05fdc8e01569c5d74e2840a310ed1f7a34b72374

YQL-20095: Enable modernize-pass-by-value

2026-03-02T08:32:11Z

commit_hash:4939dfad654f18bb31e40711ee0c39063889f17b

YQL-20095: Enable modernize-loop-convert

2026-02-27T06:12:30Z

- https://clang.llvm.org/extra/clang-tidy/checks/modernize/loop-convert.html Semi-automatic translation was performed. Manually rewritten some types and names. commit_hash:0ad921f0d512dd4a8ccc8af557197ce6a02582e1

YQL-20095: Enable google-explicit-constructor

2026-01-23T13:17:10Z

commit_hash:4d77ad10fd4db303459ec4e45e139967c7fc8196