From 215d6c316e3a46ddf95dc1a253fd112047b5fb72 Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 29 Dec 2025 20:55:14 +0000 Subject: [PATCH] Add CODEC/compression expression support in EXPLAIN output Add codec rendering to the Column() function in the explain module. CODEC expressions are now properly output as Function nodes with ExpressionList children containing the individual codec functions. This enables EXPLAIN AST output for column declarations with CODEC specifications like CODEC(LZ4), CODEC(ZSTD(10)), CODEC(GCD, LZ4), etc. Updates 62 metadata.json files to remove explain_todo entries for tests that now pass. --- internal/explain/explain.go | 31 ++++++++++++++++++ .../00162_mmap_compression_none/metadata.json | 2 +- .../metadata.json | 6 ---- .../metadata.json | 18 +---------- .../metadata.json | 9 +----- .../metadata.json | 6 +--- .../metadata.json | 8 +---- .../metadata.json | 9 +++++- .../metadata.json | 8 +---- parser/testdata/00870_t64_codec/metadata.json | 6 +--- .../00871_t64_codec_signed/metadata.json | 6 +--- .../00872_t64_bit_codec/metadata.json | 6 +--- .../00873_t64_codec_date/metadata.json | 6 +--- .../metadata.json | 8 +---- .../metadata.json | 3 -- .../metadata.json | 1 - .../metadata.json | 1 - .../metadata.json | 10 +----- .../00950_test_gorilla_codec/metadata.json | 1 - .../00957_delta_diff_bug/metadata.json | 2 +- .../01061_alter_codec_with_type/metadata.json | 7 +--- .../01272_suspicious_codecs/metadata.json | 30 +---------------- .../01296_codecs_bad_arguments/metadata.json | 2 +- .../01356_wrong_filter-type_bug/metadata.json | 6 +--- .../01375_compact_parts_codecs/metadata.json | 7 +--- .../metadata.json | 10 +----- .../metadata.json | 6 +--- .../01390_check_table_codec/metadata.json | 2 -- .../01462_test_codec_on_alias/metadata.json | 9 +----- .../metadata.json | 4 +-- .../metadata.json | 3 +- .../01622_codec_zstd_long/metadata.json | 6 +--- .../01683_codec_encrypted/metadata.json | 2 +- .../01701_if_tuple_segfault/metadata.json | 1 - .../01780_column_sparse_pk/metadata.json | 3 +- .../01911_logical_error_minus/metadata.json | 2 -- .../02313_test_fpc_codec/metadata.json | 2 -- .../metadata.json | 2 +- .../02377_modify_column_from_lc/metadata.json | 4 +-- .../metadata.json | 7 +--- .../metadata.json | 6 +--- .../02578_ipv4_codec_t64/metadata.json | 6 +--- .../metadata.json | 4 +-- .../metadata.json | 1 - .../metadata.json | 6 +--- .../metadata.json | 1 - parser/testdata/02872_gcd_codec/metadata.json | 32 +------------------ .../metadata.json | 7 +--- .../metadata.json | 7 +++- .../metadata.json | 1 - .../metadata.json | 1 - .../metadata.json | 1 - .../03093_bug_gcd_codec/metadata.json | 7 +--- .../metadata.json | 1 - .../03224_invalid_alter/metadata.json | 3 -- .../metadata.json | 3 -- .../metadata.json | 2 -- .../metadata.json | 2 -- .../metadata.json | 2 +- .../metadata.json | 7 +--- .../03702_alter_codec_index/metadata.json | 12 +------ .../03702_alter_codec_pk/metadata.json | 11 +------ 62 files changed, 85 insertions(+), 300 deletions(-) diff --git a/internal/explain/explain.go b/internal/explain/explain.go index 684042032b..c73943c68b 100644 --- a/internal/explain/explain.go +++ b/internal/explain/explain.go @@ -197,6 +197,9 @@ func Column(sb *strings.Builder, col *ast.ColumnDeclaration, depth int) { if col.Default != nil || hasEphemeralDefault { children++ } + if col.Codec != nil { + children++ + } fmt.Fprintf(sb, "%sColumnDeclaration %s (children %d)\n", indent, col.Name, children) if col.Type != nil { Node(sb, col.Type, depth+1) @@ -207,6 +210,34 @@ func Column(sb *strings.Builder, col *ast.ColumnDeclaration, depth int) { // EPHEMERAL columns without explicit default value show defaultValueOfTypeName function fmt.Fprintf(sb, "%s Function defaultValueOfTypeName\n", indent) } + if col.Codec != nil { + explainCodecExpr(sb, col.Codec, indent+" ", depth+1) + } +} + +// explainCodecExpr handles CODEC expressions in column declarations +func explainCodecExpr(sb *strings.Builder, codec *ast.CodecExpr, indent string, depth int) { + // CODEC is rendered as a Function with one child (ExpressionList of codecs) + fmt.Fprintf(sb, "%sFunction CODEC (children 1)\n", indent) + fmt.Fprintf(sb, "%s ExpressionList (children %d)\n", indent, len(codec.Codecs)) + for _, c := range codec.Codecs { + explainCodecFunction(sb, c, indent+" ", depth+2) + } +} + +// explainCodecFunction handles individual codec functions (e.g., LZ4, ZSTD(10), Gorilla(1)) +func explainCodecFunction(sb *strings.Builder, fn *ast.FunctionCall, indent string, depth int) { + if len(fn.Arguments) == 0 { + // Codec without parameters: just the function name + fmt.Fprintf(sb, "%sFunction %s\n", indent, fn.Name) + } else { + // Codec with parameters: function with ExpressionList of arguments + fmt.Fprintf(sb, "%sFunction %s (children 1)\n", indent, fn.Name) + fmt.Fprintf(sb, "%s ExpressionList (children %d)\n", indent, len(fn.Arguments)) + for _, arg := range fn.Arguments { + Node(sb, arg, depth+2) + } + } } func Index(sb *strings.Builder, idx *ast.IndexDefinition, depth int) { diff --git a/parser/testdata/00162_mmap_compression_none/metadata.json b/parser/testdata/00162_mmap_compression_none/metadata.json index af48d4c110..0967ef424b 100644 --- a/parser/testdata/00162_mmap_compression_none/metadata.json +++ b/parser/testdata/00162_mmap_compression_none/metadata.json @@ -1 +1 @@ -{"explain_todo":{"stmt2":true}} +{} diff --git a/parser/testdata/00804_test_alter_compression_codecs/metadata.json b/parser/testdata/00804_test_alter_compression_codecs/metadata.json index a46e625301..6fc63d8933 100644 --- a/parser/testdata/00804_test_alter_compression_codecs/metadata.json +++ b/parser/testdata/00804_test_alter_compression_codecs/metadata.json @@ -5,14 +5,8 @@ "stmt20": true, "stmt21": true, "stmt27": true, - "stmt3": true, - "stmt30": true, - "stmt31": true, - "stmt32": true, - "stmt36": true, "stmt41": true, "stmt43": true, - "stmt7": true, "stmt8": true } } diff --git a/parser/testdata/00804_test_custom_compression_codecs/metadata.json b/parser/testdata/00804_test_custom_compression_codecs/metadata.json index cce7fadbd2..0967ef424b 100644 --- a/parser/testdata/00804_test_custom_compression_codecs/metadata.json +++ b/parser/testdata/00804_test_custom_compression_codecs/metadata.json @@ -1,17 +1 @@ -{ - "explain_todo": { - "stmt22": true, - "stmt23": true, - "stmt24": true, - "stmt25": true, - "stmt26": true, - "stmt27": true, - "stmt28": true, - "stmt38": true, - "stmt4": true, - "stmt49": true, - "stmt50": true, - "stmt58": true, - "stmt69": true - } -} +{} diff --git a/parser/testdata/00804_test_custom_compression_codes_log_storages/metadata.json b/parser/testdata/00804_test_custom_compression_codes_log_storages/metadata.json index 823e966b3f..0967ef424b 100644 --- a/parser/testdata/00804_test_custom_compression_codes_log_storages/metadata.json +++ b/parser/testdata/00804_test_custom_compression_codes_log_storages/metadata.json @@ -1,8 +1 @@ -{ - "explain_todo": { - "stmt16": true, - "stmt28": true, - "stmt4": true, - "stmt40": true - } -} +{} diff --git a/parser/testdata/00804_test_deflate_qpl_codec_compression/metadata.json b/parser/testdata/00804_test_deflate_qpl_codec_compression/metadata.json index b65b07d7a6..0967ef424b 100644 --- a/parser/testdata/00804_test_deflate_qpl_codec_compression/metadata.json +++ b/parser/testdata/00804_test_deflate_qpl_codec_compression/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt4": true - } -} +{} diff --git a/parser/testdata/00804_test_delta_codec_compression/metadata.json b/parser/testdata/00804_test_delta_codec_compression/metadata.json index 3e98ec5dca..00bc5a787d 100644 --- a/parser/testdata/00804_test_delta_codec_compression/metadata.json +++ b/parser/testdata/00804_test_delta_codec_compression/metadata.json @@ -1,14 +1,8 @@ { "explain_todo": { "stmt12": true, - "stmt18": true, - "stmt19": true, "stmt20": true, "stmt24": true, - "stmt30": true, - "stmt31": true, - "stmt36": true, - "stmt5": true, - "stmt6": true + "stmt36": true } } diff --git a/parser/testdata/00804_test_delta_codec_no_type_alter/metadata.json b/parser/testdata/00804_test_delta_codec_no_type_alter/metadata.json index 4048771797..691704c669 100644 --- a/parser/testdata/00804_test_delta_codec_no_type_alter/metadata.json +++ b/parser/testdata/00804_test_delta_codec_no_type_alter/metadata.json @@ -1 +1,8 @@ -{"explain_todo":{"stmt4":true,"stmt5":true,"stmt6":true,"stmt7":true,"stmt8":true,"stmt9":true}} +{ + "explain_todo": { + "stmt5": true, + "stmt6": true, + "stmt7": true, + "stmt9": true + } +} diff --git a/parser/testdata/00804_test_zstd_qat_codec_compression/metadata.json b/parser/testdata/00804_test_zstd_qat_codec_compression/metadata.json index 19830977ac..0967ef424b 100644 --- a/parser/testdata/00804_test_zstd_qat_codec_compression/metadata.json +++ b/parser/testdata/00804_test_zstd_qat_codec_compression/metadata.json @@ -1,7 +1 @@ -{ - "explain_todo": { - "stmt4": true, - "stmt5": true, - "stmt6": true - } -} +{} diff --git a/parser/testdata/00870_t64_codec/metadata.json b/parser/testdata/00870_t64_codec/metadata.json index ef58f80315..0967ef424b 100644 --- a/parser/testdata/00870_t64_codec/metadata.json +++ b/parser/testdata/00870_t64_codec/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt2": true - } -} +{} diff --git a/parser/testdata/00871_t64_codec_signed/metadata.json b/parser/testdata/00871_t64_codec_signed/metadata.json index ef58f80315..0967ef424b 100644 --- a/parser/testdata/00871_t64_codec_signed/metadata.json +++ b/parser/testdata/00871_t64_codec_signed/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt2": true - } -} +{} diff --git a/parser/testdata/00872_t64_bit_codec/metadata.json b/parser/testdata/00872_t64_bit_codec/metadata.json index ef58f80315..0967ef424b 100644 --- a/parser/testdata/00872_t64_bit_codec/metadata.json +++ b/parser/testdata/00872_t64_bit_codec/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt2": true - } -} +{} diff --git a/parser/testdata/00873_t64_codec_date/metadata.json b/parser/testdata/00873_t64_codec_date/metadata.json index ef58f80315..0967ef424b 100644 --- a/parser/testdata/00873_t64_codec_date/metadata.json +++ b/parser/testdata/00873_t64_codec_date/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt2": true - } -} +{} diff --git a/parser/testdata/00910_zookeeper_custom_compression_codecs_replicated_long/metadata.json b/parser/testdata/00910_zookeeper_custom_compression_codecs_replicated_long/metadata.json index ae10cd1b65..06c3006063 100644 --- a/parser/testdata/00910_zookeeper_custom_compression_codecs_replicated_long/metadata.json +++ b/parser/testdata/00910_zookeeper_custom_compression_codecs_replicated_long/metadata.json @@ -2,15 +2,9 @@ "explain_todo": { "stmt10": true, "stmt15": true, - "stmt25": true, - "stmt26": true, "stmt28": true, "stmt32": true, "stmt40": true, - "stmt42": true, - "stmt48": true, - "stmt5": true, - "stmt56": true, - "stmt6": true + "stmt42": true } } diff --git a/parser/testdata/00910_zookeeper_test_alter_compression_codecs_long/metadata.json b/parser/testdata/00910_zookeeper_test_alter_compression_codecs_long/metadata.json index 1942c1bbff..bd3158a87c 100644 --- a/parser/testdata/00910_zookeeper_test_alter_compression_codecs_long/metadata.json +++ b/parser/testdata/00910_zookeeper_test_alter_compression_codecs_long/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt12": true, "stmt13": true, "stmt14": true, "stmt15": true, @@ -21,8 +20,6 @@ "stmt44": true, "stmt45": true, "stmt46": true, - "stmt5": true, - "stmt6": true, "stmt9": true } } diff --git a/parser/testdata/00926_adaptive_index_granularity_pk/metadata.json b/parser/testdata/00926_adaptive_index_granularity_pk/metadata.json index d4dd07ed05..d227d77cb4 100644 --- a/parser/testdata/00926_adaptive_index_granularity_pk/metadata.json +++ b/parser/testdata/00926_adaptive_index_granularity_pk/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt39": true, "stmt43": true } } diff --git a/parser/testdata/00950_test_double_delta_codec/metadata.json b/parser/testdata/00950_test_double_delta_codec/metadata.json index 9a00eacae6..e9f2f3b7c0 100644 --- a/parser/testdata/00950_test_double_delta_codec/metadata.json +++ b/parser/testdata/00950_test_double_delta_codec/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt2": true, "stmt28": true, "stmt3": true, "stmt4": true, diff --git a/parser/testdata/00950_test_double_delta_codec_types/metadata.json b/parser/testdata/00950_test_double_delta_codec_types/metadata.json index 00fe25163a..ca584b3e28 100644 --- a/parser/testdata/00950_test_double_delta_codec_types/metadata.json +++ b/parser/testdata/00950_test_double_delta_codec_types/metadata.json @@ -1,13 +1,5 @@ { "explain_todo": { - "stmt10": true, - "stmt14": true, - "stmt15": true, - "stmt17": true, - "stmt2": true, - "stmt3": true, - "stmt4": true, - "stmt5": true, - "stmt7": true + "stmt17": true } } diff --git a/parser/testdata/00950_test_gorilla_codec/metadata.json b/parser/testdata/00950_test_gorilla_codec/metadata.json index 6294db7a1a..2dda3f42fa 100644 --- a/parser/testdata/00950_test_gorilla_codec/metadata.json +++ b/parser/testdata/00950_test_gorilla_codec/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt3": true, "stmt4": true, "stmt5": true, "stmt6": true, diff --git a/parser/testdata/00957_delta_diff_bug/metadata.json b/parser/testdata/00957_delta_diff_bug/metadata.json index 51dfabe749..0967ef424b 100644 --- a/parser/testdata/00957_delta_diff_bug/metadata.json +++ b/parser/testdata/00957_delta_diff_bug/metadata.json @@ -1 +1 @@ -{"explain_todo":{"stmt3":true}} +{} diff --git a/parser/testdata/01061_alter_codec_with_type/metadata.json b/parser/testdata/01061_alter_codec_with_type/metadata.json index 9a8cc69c0b..0967ef424b 100644 --- a/parser/testdata/01061_alter_codec_with_type/metadata.json +++ b/parser/testdata/01061_alter_codec_with_type/metadata.json @@ -1,6 +1 @@ -{ - "explain_todo": { - "stmt2": true, - "stmt4": true - } -} +{} diff --git a/parser/testdata/01272_suspicious_codecs/metadata.json b/parser/testdata/01272_suspicious_codecs/metadata.json index df55975f0b..0967ef424b 100644 --- a/parser/testdata/01272_suspicious_codecs/metadata.json +++ b/parser/testdata/01272_suspicious_codecs/metadata.json @@ -1,29 +1 @@ -{ - "explain_todo": { - "stmt10": true, - "stmt11": true, - "stmt12": true, - "stmt13": true, - "stmt14": true, - "stmt15": true, - "stmt16": true, - "stmt2": true, - "stmt29": true, - "stmt30": true, - "stmt31": true, - "stmt32": true, - "stmt33": true, - "stmt34": true, - "stmt35": true, - "stmt36": true, - "stmt37": true, - "stmt38": true, - "stmt39": true, - "stmt4": true, - "stmt5": true, - "stmt6": true, - "stmt7": true, - "stmt8": true, - "stmt9": true - } -} +{} diff --git a/parser/testdata/01296_codecs_bad_arguments/metadata.json b/parser/testdata/01296_codecs_bad_arguments/metadata.json index cc2f3624ef..0967ef424b 100644 --- a/parser/testdata/01296_codecs_bad_arguments/metadata.json +++ b/parser/testdata/01296_codecs_bad_arguments/metadata.json @@ -1 +1 @@ -{"explain_todo":{"stmt4":true}} +{} diff --git a/parser/testdata/01356_wrong_filter-type_bug/metadata.json b/parser/testdata/01356_wrong_filter-type_bug/metadata.json index ef58f80315..0967ef424b 100644 --- a/parser/testdata/01356_wrong_filter-type_bug/metadata.json +++ b/parser/testdata/01356_wrong_filter-type_bug/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt2": true - } -} +{} diff --git a/parser/testdata/01375_compact_parts_codecs/metadata.json b/parser/testdata/01375_compact_parts_codecs/metadata.json index 9438414527..0967ef424b 100644 --- a/parser/testdata/01375_compact_parts_codecs/metadata.json +++ b/parser/testdata/01375_compact_parts_codecs/metadata.json @@ -1,6 +1 @@ -{ - "explain_todo": { - "stmt10": true, - "stmt18": true - } -} +{} diff --git a/parser/testdata/01380_coded_delta_exception_code/metadata.json b/parser/testdata/01380_coded_delta_exception_code/metadata.json index b7ce0dc73e..0967ef424b 100644 --- a/parser/testdata/01380_coded_delta_exception_code/metadata.json +++ b/parser/testdata/01380_coded_delta_exception_code/metadata.json @@ -1,9 +1 @@ -{ - "explain_todo": { - "stmt1": true, - "stmt2": true, - "stmt3": true, - "stmt4": true, - "stmt5": true - } -} +{} diff --git a/parser/testdata/01386_negative_float_constant_key_condition/metadata.json b/parser/testdata/01386_negative_float_constant_key_condition/metadata.json index 1295a45747..0967ef424b 100644 --- a/parser/testdata/01386_negative_float_constant_key_condition/metadata.json +++ b/parser/testdata/01386_negative_float_constant_key_condition/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt3": true - } -} +{} diff --git a/parser/testdata/01390_check_table_codec/metadata.json b/parser/testdata/01390_check_table_codec/metadata.json index c05c1b856b..25122ac4f4 100644 --- a/parser/testdata/01390_check_table_codec/metadata.json +++ b/parser/testdata/01390_check_table_codec/metadata.json @@ -1,8 +1,6 @@ { "explain_todo": { - "stmt3": true, "stmt5": true, - "stmt7": true, "stmt9": true } } diff --git a/parser/testdata/01462_test_codec_on_alias/metadata.json b/parser/testdata/01462_test_codec_on_alias/metadata.json index c05c1b856b..0967ef424b 100644 --- a/parser/testdata/01462_test_codec_on_alias/metadata.json +++ b/parser/testdata/01462_test_codec_on_alias/metadata.json @@ -1,8 +1 @@ -{ - "explain_todo": { - "stmt3": true, - "stmt5": true, - "stmt7": true, - "stmt9": true - } -} +{} diff --git a/parser/testdata/01504_compression_multiple_streams/metadata.json b/parser/testdata/01504_compression_multiple_streams/metadata.json index 6b491e85ae..07df712607 100644 --- a/parser/testdata/01504_compression_multiple_streams/metadata.json +++ b/parser/testdata/01504_compression_multiple_streams/metadata.json @@ -3,8 +3,6 @@ "stmt13": true, "stmt20": true, "stmt3": true, - "stmt30": true, - "stmt37": true, - "stmt38": true + "stmt30": true } } diff --git a/parser/testdata/01582_move_to_prewhere_compact_parts/metadata.json b/parser/testdata/01582_move_to_prewhere_compact_parts/metadata.json index 7a88052028..e47c967938 100644 --- a/parser/testdata/01582_move_to_prewhere_compact_parts/metadata.json +++ b/parser/testdata/01582_move_to_prewhere_compact_parts/metadata.json @@ -1,7 +1,6 @@ { "explain_todo": { "stmt11": true, - "stmt6": true, - "stmt8": true + "stmt6": true } } diff --git a/parser/testdata/01622_codec_zstd_long/metadata.json b/parser/testdata/01622_codec_zstd_long/metadata.json index 1a1b0e251b..b563327205 100644 --- a/parser/testdata/01622_codec_zstd_long/metadata.json +++ b/parser/testdata/01622_codec_zstd_long/metadata.json @@ -1,9 +1,5 @@ { "explain_todo": { - "stmt10": true, - "stmt11": true, - "stmt7": true, - "stmt8": true, - "stmt9": true + "stmt7": true } } diff --git a/parser/testdata/01683_codec_encrypted/metadata.json b/parser/testdata/01683_codec_encrypted/metadata.json index 17b60ef4f1..0967ef424b 100644 --- a/parser/testdata/01683_codec_encrypted/metadata.json +++ b/parser/testdata/01683_codec_encrypted/metadata.json @@ -1 +1 @@ -{"explain_todo":{"stmt2":true,"stmt6":true}} +{} diff --git a/parser/testdata/01701_if_tuple_segfault/metadata.json b/parser/testdata/01701_if_tuple_segfault/metadata.json index 97a2a51761..661bded8e9 100644 --- a/parser/testdata/01701_if_tuple_segfault/metadata.json +++ b/parser/testdata/01701_if_tuple_segfault/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt2": true, "stmt7": true, "stmt8": true, "stmt9": true diff --git a/parser/testdata/01780_column_sparse_pk/metadata.json b/parser/testdata/01780_column_sparse_pk/metadata.json index d6e4146747..9238486a6e 100644 --- a/parser/testdata/01780_column_sparse_pk/metadata.json +++ b/parser/testdata/01780_column_sparse_pk/metadata.json @@ -1,7 +1,6 @@ { "explain_todo": { "stmt17": true, - "stmt18": true, - "stmt9": true + "stmt18": true } } diff --git a/parser/testdata/01911_logical_error_minus/metadata.json b/parser/testdata/01911_logical_error_minus/metadata.json index 1e72ad79ab..d2b61a36fd 100644 --- a/parser/testdata/01911_logical_error_minus/metadata.json +++ b/parser/testdata/01911_logical_error_minus/metadata.json @@ -1,12 +1,10 @@ { "explain_todo": { - "stmt11": true, "stmt12": true, "stmt13": true, "stmt14": true, "stmt15": true, "stmt16": true, - "stmt3": true, "stmt4": true, "stmt5": true, "stmt6": true, diff --git a/parser/testdata/02313_test_fpc_codec/metadata.json b/parser/testdata/02313_test_fpc_codec/metadata.json index 07e037ddcc..48a7016a67 100644 --- a/parser/testdata/02313_test_fpc_codec/metadata.json +++ b/parser/testdata/02313_test_fpc_codec/metadata.json @@ -1,11 +1,9 @@ { "explain_todo": { - "stmt13": true, "stmt14": true, "stmt15": true, "stmt16": true, "stmt17": true, - "stmt3": true, "stmt4": true, "stmt5": true, "stmt6": true, diff --git a/parser/testdata/02343_read_from_s3_compressed_blocks/metadata.json b/parser/testdata/02343_read_from_s3_compressed_blocks/metadata.json index af48d4c110..0967ef424b 100644 --- a/parser/testdata/02343_read_from_s3_compressed_blocks/metadata.json +++ b/parser/testdata/02343_read_from_s3_compressed_blocks/metadata.json @@ -1 +1 @@ -{"explain_todo":{"stmt2":true}} +{} diff --git a/parser/testdata/02377_modify_column_from_lc/metadata.json b/parser/testdata/02377_modify_column_from_lc/metadata.json index 7d2781a8cf..ab9202e88e 100644 --- a/parser/testdata/02377_modify_column_from_lc/metadata.json +++ b/parser/testdata/02377_modify_column_from_lc/metadata.json @@ -1,7 +1,5 @@ { "explain_todo": { - "stmt11": true, - "stmt4": true, - "stmt5": true + "stmt11": true } } diff --git a/parser/testdata/02536_delta_gorilla_corruption/metadata.json b/parser/testdata/02536_delta_gorilla_corruption/metadata.json index 15d79beb93..0967ef424b 100644 --- a/parser/testdata/02536_delta_gorilla_corruption/metadata.json +++ b/parser/testdata/02536_delta_gorilla_corruption/metadata.json @@ -1,6 +1 @@ -{ - "explain_todo": { - "stmt11": true, - "stmt5": true - } -} +{} diff --git a/parser/testdata/02538_nullable_array_tuple_timeseries/metadata.json b/parser/testdata/02538_nullable_array_tuple_timeseries/metadata.json index ef58f80315..0967ef424b 100644 --- a/parser/testdata/02538_nullable_array_tuple_timeseries/metadata.json +++ b/parser/testdata/02538_nullable_array_tuple_timeseries/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt2": true - } -} +{} diff --git a/parser/testdata/02578_ipv4_codec_t64/metadata.json b/parser/testdata/02578_ipv4_codec_t64/metadata.json index ef58f80315..0967ef424b 100644 --- a/parser/testdata/02578_ipv4_codec_t64/metadata.json +++ b/parser/testdata/02578_ipv4_codec_t64/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt2": true - } -} +{} diff --git a/parser/testdata/02725_alias_columns_should_not_allow_compression_codec/metadata.json b/parser/testdata/02725_alias_columns_should_not_allow_compression_codec/metadata.json index 39429336f8..ef382ce51e 100644 --- a/parser/testdata/02725_alias_columns_should_not_allow_compression_codec/metadata.json +++ b/parser/testdata/02725_alias_columns_should_not_allow_compression_codec/metadata.json @@ -1,8 +1,6 @@ { "explain_todo": { - "stmt3": true, "stmt4": true, - "stmt5": true, - "stmt6": true + "stmt5": true } } diff --git a/parser/testdata/02796_projection_date_filter_on_view/metadata.json b/parser/testdata/02796_projection_date_filter_on_view/metadata.json index a4f8773303..ef382ce51e 100644 --- a/parser/testdata/02796_projection_date_filter_on_view/metadata.json +++ b/parser/testdata/02796_projection_date_filter_on_view/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt3": true, "stmt4": true, "stmt5": true } diff --git a/parser/testdata/02864_test_ipv4_type_mismatch/metadata.json b/parser/testdata/02864_test_ipv4_type_mismatch/metadata.json index ef58f80315..0967ef424b 100644 --- a/parser/testdata/02864_test_ipv4_type_mismatch/metadata.json +++ b/parser/testdata/02864_test_ipv4_type_mismatch/metadata.json @@ -1,5 +1 @@ -{ - "explain_todo": { - "stmt2": true - } -} +{} diff --git a/parser/testdata/02870_move_partition_to_volume_io_throttling/metadata.json b/parser/testdata/02870_move_partition_to_volume_io_throttling/metadata.json index 4797b680f6..6c28a962d3 100644 --- a/parser/testdata/02870_move_partition_to_volume_io_throttling/metadata.json +++ b/parser/testdata/02870_move_partition_to_volume_io_throttling/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt2": true, "stmt4": true, "stmt5": true, "stmt7": true, diff --git a/parser/testdata/02872_gcd_codec/metadata.json b/parser/testdata/02872_gcd_codec/metadata.json index 2ee82e2769..0967ef424b 100644 --- a/parser/testdata/02872_gcd_codec/metadata.json +++ b/parser/testdata/02872_gcd_codec/metadata.json @@ -1,31 +1 @@ -{ - "explain_todo": { - "stmt1": true, - "stmt10": true, - "stmt11": true, - "stmt12": true, - "stmt13": true, - "stmt2": true, - "stmt26": true, - "stmt27": true, - "stmt28": true, - "stmt29": true, - "stmt3": true, - "stmt30": true, - "stmt31": true, - "stmt44": true, - "stmt45": true, - "stmt46": true, - "stmt47": true, - "stmt5": true, - "stmt56": true, - "stmt57": true, - "stmt62": true, - "stmt63": true, - "stmt68": true, - "stmt73": true, - "stmt76": true, - "stmt8": true, - "stmt9": true - } -} +{} diff --git a/parser/testdata/02884_parallel_window_functions_bug/metadata.json b/parser/testdata/02884_parallel_window_functions_bug/metadata.json index 682bda1cbc..0967ef424b 100644 --- a/parser/testdata/02884_parallel_window_functions_bug/metadata.json +++ b/parser/testdata/02884_parallel_window_functions_bug/metadata.json @@ -1,6 +1 @@ -{ - "explain_todo": { - "stmt1": true, - "stmt2": true - } -} +{} diff --git a/parser/testdata/02947_merge_tree_index_table_1/metadata.json b/parser/testdata/02947_merge_tree_index_table_1/metadata.json index dff2841d26..638822386f 100644 --- a/parser/testdata/02947_merge_tree_index_table_1/metadata.json +++ b/parser/testdata/02947_merge_tree_index_table_1/metadata.json @@ -1 +1,6 @@ -{"explain_todo":{"stmt10":true,"stmt11":true,"stmt2":true,"stmt3":true}} +{ + "explain_todo": { + "stmt11": true, + "stmt3": true + } +} diff --git a/parser/testdata/03001_consider_lwd_when_merge/metadata.json b/parser/testdata/03001_consider_lwd_when_merge/metadata.json index 9c1d811cfd..9c5b90bd57 100644 --- a/parser/testdata/03001_consider_lwd_when_merge/metadata.json +++ b/parser/testdata/03001_consider_lwd_when_merge/metadata.json @@ -3,7 +3,6 @@ "stmt10": true, "stmt12": true, "stmt14": true, - "stmt2": true, "stmt7": true, "stmt8": true } diff --git a/parser/testdata/03035_argMinMax_numeric_non_extreme_bug/metadata.json b/parser/testdata/03035_argMinMax_numeric_non_extreme_bug/metadata.json index e5e4780514..1295a45747 100644 --- a/parser/testdata/03035_argMinMax_numeric_non_extreme_bug/metadata.json +++ b/parser/testdata/03035_argMinMax_numeric_non_extreme_bug/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt1": true, "stmt3": true } } diff --git a/parser/testdata/03038_move_partition_to_oneself_deadlock/metadata.json b/parser/testdata/03038_move_partition_to_oneself_deadlock/metadata.json index 45b9ec9cc7..8f6d4ba033 100644 --- a/parser/testdata/03038_move_partition_to_oneself_deadlock/metadata.json +++ b/parser/testdata/03038_move_partition_to_oneself_deadlock/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt3": true, "stmt5": true, "stmt6": true, "stmt7": true diff --git a/parser/testdata/03093_bug_gcd_codec/metadata.json b/parser/testdata/03093_bug_gcd_codec/metadata.json index aeb01f1428..0967ef424b 100644 --- a/parser/testdata/03093_bug_gcd_codec/metadata.json +++ b/parser/testdata/03093_bug_gcd_codec/metadata.json @@ -1,6 +1 @@ -{ - "explain_todo": { - "stmt1": true, - "stmt4": true - } -} +{} diff --git a/parser/testdata/03094_grouparraysorted_memory/metadata.json b/parser/testdata/03094_grouparraysorted_memory/metadata.json index e5e4780514..1295a45747 100644 --- a/parser/testdata/03094_grouparraysorted_memory/metadata.json +++ b/parser/testdata/03094_grouparraysorted_memory/metadata.json @@ -1,6 +1,5 @@ { "explain_todo": { - "stmt1": true, "stmt3": true } } diff --git a/parser/testdata/03224_invalid_alter/metadata.json b/parser/testdata/03224_invalid_alter/metadata.json index a375356c3e..57e491f07c 100644 --- a/parser/testdata/03224_invalid_alter/metadata.json +++ b/parser/testdata/03224_invalid_alter/metadata.json @@ -1,8 +1,5 @@ { "explain_todo": { - "stmt1": true, - "stmt10": true, - "stmt20": true, "stmt21": true, "stmt22": true, "stmt23": true, diff --git a/parser/testdata/03254_last_2_samples_aggregate_function/metadata.json b/parser/testdata/03254_last_2_samples_aggregate_function/metadata.json index ed410aa7ca..c07b339f46 100644 --- a/parser/testdata/03254_last_2_samples_aggregate_function/metadata.json +++ b/parser/testdata/03254_last_2_samples_aggregate_function/metadata.json @@ -3,10 +3,7 @@ "stmt11": true, "stmt12": true, "stmt16": true, - "stmt2": true, - "stmt3": true, "stmt4": true, - "stmt5": true, "stmt6": true } } diff --git a/parser/testdata/03254_last_2_samples_aggregate_function_simple/metadata.json b/parser/testdata/03254_last_2_samples_aggregate_function_simple/metadata.json index 0549ae005e..a74c293460 100644 --- a/parser/testdata/03254_last_2_samples_aggregate_function_simple/metadata.json +++ b/parser/testdata/03254_last_2_samples_aggregate_function_simple/metadata.json @@ -1,7 +1,5 @@ { "explain_todo": { - "stmt2": true, - "stmt3": true, "stmt4": true, "stmt8": true, "stmt9": true diff --git a/parser/testdata/03254_timeseries_instant_value_aggregate_functions/metadata.json b/parser/testdata/03254_timeseries_instant_value_aggregate_functions/metadata.json index b37186c2a4..af33896a08 100644 --- a/parser/testdata/03254_timeseries_instant_value_aggregate_functions/metadata.json +++ b/parser/testdata/03254_timeseries_instant_value_aggregate_functions/metadata.json @@ -1,8 +1,6 @@ { "explain_todo": { - "stmt3": true, "stmt4": true, - "stmt6": true, "stmt7": true, "stmt9": true } diff --git a/parser/testdata/03353_codec_zstd_doubledelta_data_corruption/metadata.json b/parser/testdata/03353_codec_zstd_doubledelta_data_corruption/metadata.json index af48d4c110..0967ef424b 100644 --- a/parser/testdata/03353_codec_zstd_doubledelta_data_corruption/metadata.json +++ b/parser/testdata/03353_codec_zstd_doubledelta_data_corruption/metadata.json @@ -1 +1 @@ -{"explain_todo":{"stmt2":true}} +{} diff --git a/parser/testdata/03364_gorilla_codec_parameters/metadata.json b/parser/testdata/03364_gorilla_codec_parameters/metadata.json index bc141058a4..0967ef424b 100644 --- a/parser/testdata/03364_gorilla_codec_parameters/metadata.json +++ b/parser/testdata/03364_gorilla_codec_parameters/metadata.json @@ -1,6 +1 @@ -{ - "explain_todo": { - "stmt2": true, - "stmt3": true - } -} +{} diff --git a/parser/testdata/03702_alter_codec_index/metadata.json b/parser/testdata/03702_alter_codec_index/metadata.json index e48c96c7d3..0967ef424b 100644 --- a/parser/testdata/03702_alter_codec_index/metadata.json +++ b/parser/testdata/03702_alter_codec_index/metadata.json @@ -1,11 +1 @@ -{ - "explain_todo": { - "stmt11": true, - "stmt15": true, - "stmt16": true, - "stmt4": true, - "stmt5": true, - "stmt7": true, - "stmt9": true - } -} +{} diff --git a/parser/testdata/03702_alter_codec_pk/metadata.json b/parser/testdata/03702_alter_codec_pk/metadata.json index 40f920c1e8..0967ef424b 100644 --- a/parser/testdata/03702_alter_codec_pk/metadata.json +++ b/parser/testdata/03702_alter_codec_pk/metadata.json @@ -1,10 +1 @@ -{ - "explain_todo": { - "stmt11": true, - "stmt12": true, - "stmt4": true, - "stmt5": true, - "stmt7": true, - "stmt8": true - } -} +{}