Skip to content

Commit 2260f06

Browse files
vustefclaude
andcommitted
Fix dict_to_pairs Julia template and redact credentials in pretty printer
Julia dict_to_pairs now produces Vector{Tuple} instead of Vector{Pair}. Credentials are redacted from pretty-printed output for security. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent fe496d9 commit 2260f06

5 files changed

Lines changed: 362 additions & 375 deletions

File tree

meta/src/meta/codegen_templates.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ class BuiltinTemplate:
135135
"make_empty_bytes": BuiltinTemplate("UInt8[]"),
136136
"dict_from_list": BuiltinTemplate("Dict({0})"),
137137
"dict_get": BuiltinTemplate("get({0}, {1}, nothing)"),
138-
"dict_to_pairs": BuiltinTemplate("sort(collect(pairs({0})))"),
138+
"dict_to_pairs": BuiltinTemplate("sort([(k, v) for (k, v) in {0}])"),
139139
"has_proto_field": BuiltinTemplate("_has_proto_field({0}, Symbol({1}))"),
140140
"string_to_upper": BuiltinTemplate("uppercase({0})"),
141141
"string_in_list": BuiltinTemplate("({0} in {1})"),

meta/src/meta/grammar.y

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1066,7 +1066,7 @@ iceberg_config
10661066
$3: String = $$.catalog_uri
10671067
$4: Optional[String] = $$.scope if $$.scope != "" else None
10681068
$5: Optional[Sequence[Tuple[String, String]]] = builtin.dict_to_pairs($$.properties) if not builtin.is_empty(builtin.dict_to_pairs($$.properties)) else None
1069-
$6: Optional[Sequence[Tuple[String, String]]] = builtin.dict_to_pairs($$.credentials) if not builtin.is_empty(builtin.dict_to_pairs($$.credentials)) else None
1069+
$6: Optional[Sequence[Tuple[String, String]]] = builtin.none()
10701070

10711071
iceberg_config_scope
10721072
: "(" "scope" STRING ")"

0 commit comments

Comments
 (0)