Commit 400e578
committed
Update expected explain outputs and tests for SQL validation round-trip
The validation round-trip (RelNode → SQL → validate → RelNode) introduces
plan changes such as extra LogicalProject nodes, type annotations on MAP
keys, and reordered aggregate operands. Update YAML expected outputs and
test assertions to match.
- Catch Throwable (not just Exception) in validate() for AssertionError
from unsupported RelNodes like LogicalGraphLookup
- Update 17 YAML expected output files for calcite and calcite_no_pushdown
- Add 4 new alternative YAML files for non-deterministic plan variants
- Add separate YAML for boolean string literal filter test
- Accept "inferred array element type" error in NoMv missing field test1 parent 38bb66e commit 400e578
25 files changed
Lines changed: 131 additions & 75 deletions
File tree
- core/src/main/java/org/opensearch/sql/executor
- integ-test/src/test
- java/org/opensearch/sql/calcite/remote
- resources/expectedOutput
- calcite_no_pushdown
- calcite
- big5
- clickbench
Lines changed: 13 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
26 | | - | |
27 | 26 | | |
28 | 27 | | |
29 | 28 | | |
| |||
44 | 43 | | |
45 | 44 | | |
46 | 45 | | |
47 | | - | |
48 | 46 | | |
49 | 47 | | |
50 | 48 | | |
| |||
55 | 53 | | |
56 | 54 | | |
57 | 55 | | |
58 | | - | |
59 | 56 | | |
60 | 57 | | |
61 | 58 | | |
| |||
297 | 294 | | |
298 | 295 | | |
299 | 296 | | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
300 | 309 | | |
301 | 310 | | |
302 | 311 | | |
| |||
308 | 317 | | |
309 | 318 | | |
310 | 319 | | |
311 | | - | |
312 | | - | |
313 | | - | |
314 | | - | |
315 | | - | |
316 | | - | |
317 | | - | |
318 | | - | |
| 320 | + | |
319 | 321 | | |
320 | 322 | | |
321 | 323 | | |
| |||
Lines changed: 18 additions & 14 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2137 | 2137 | | |
2138 | 2138 | | |
2139 | 2139 | | |
2140 | | - | |
2141 | | - | |
2142 | 2140 | | |
2143 | | - | |
2144 | | - | |
| 2141 | + | |
| 2142 | + | |
| 2143 | + | |
| 2144 | + | |
2145 | 2145 | | |
2146 | 2146 | | |
2147 | 2147 | | |
2148 | 2148 | | |
2149 | 2149 | | |
2150 | 2150 | | |
2151 | 2151 | | |
2152 | | - | |
| 2152 | + | |
| 2153 | + | |
2153 | 2154 | | |
2154 | 2155 | | |
2155 | 2156 | | |
| |||
2176 | 2177 | | |
2177 | 2178 | | |
2178 | 2179 | | |
2179 | | - | |
2180 | | - | |
2181 | 2180 | | |
2182 | | - | |
2183 | | - | |
| 2181 | + | |
| 2182 | + | |
| 2183 | + | |
| 2184 | + | |
2184 | 2185 | | |
2185 | 2186 | | |
2186 | 2187 | | |
| |||
2189 | 2190 | | |
2190 | 2191 | | |
2191 | 2192 | | |
2192 | | - | |
| 2193 | + | |
| 2194 | + | |
2193 | 2195 | | |
2194 | 2196 | | |
2195 | 2197 | | |
| |||
2253 | 2255 | | |
2254 | 2256 | | |
2255 | 2257 | | |
2256 | | - | |
| 2258 | + | |
| 2259 | + | |
2257 | 2260 | | |
2258 | 2261 | | |
2259 | 2262 | | |
| |||
2650 | 2653 | | |
2651 | 2654 | | |
2652 | 2655 | | |
2653 | | - | |
2654 | | - | |
| 2656 | + | |
| 2657 | + | |
2655 | 2658 | | |
2656 | 2659 | | |
2657 | 2660 | | |
2658 | 2661 | | |
2659 | | - | |
| 2662 | + | |
| 2663 | + | |
2660 | 2664 | | |
2661 | 2665 | | |
2662 | 2666 | | |
| |||
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
225 | 225 | | |
226 | 226 | | |
227 | 227 | | |
228 | | - | |
| 228 | + | |
| 229 | + | |
229 | 230 | | |
230 | 231 | | |
231 | 232 | | |
| |||
Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
2 | 1 | | |
3 | 2 | | |
4 | 3 | | |
| |||
10 | 9 | | |
11 | 10 | | |
12 | 11 | | |
13 | | - | |
| 12 | + | |
Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
2 | 1 | | |
3 | 2 | | |
4 | 3 | | |
| |||
16 | 15 | | |
17 | 16 | | |
18 | 17 | | |
19 | | - | |
| 18 | + | |
Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
2 | 1 | | |
3 | 2 | | |
4 | 3 | | |
| |||
19 | 18 | | |
20 | 19 | | |
21 | 20 | | |
22 | | - | |
| 21 | + | |
0 commit comments