Re: [PR] API: Use normalized JSON path to identify Variant fields [iceberg]

2025-04-21 Thread via GitHub
amogh-jahagirdar commented on PR #12835: URL: https://github.com/apache/iceberg/pull/12835#issuecomment-2819521337 thanks @rdblue and thank you @aihuaxu for the spec change for this and for reviewing! I'll go ahead and merge -- This is an automated message from the Apache Git Service. To

Re: [PR] API: Use normalized JSON path to identify Variant fields [iceberg]

2025-04-21 Thread via GitHub
amogh-jahagirdar merged PR #12835: URL: https://github.com/apache/iceberg/pull/12835 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@

Re: [PR] API: Use normalized JSON path to identify Variant fields [iceberg]

2025-04-18 Thread via GitHub
ajantha-bhat commented on PR #12835: URL: https://github.com/apache/iceberg/pull/12835#issuecomment-2816245897 Will cherry-pick this to the next RC once merged. cc: @nastra -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] API: Use normalized JSON path to identify Variant fields [iceberg]

2025-04-17 Thread via GitHub
rdblue commented on code in PR #12835: URL: https://github.com/apache/iceberg/pull/12835#discussion_r2049461135 ## api/src/main/java/org/apache/iceberg/expressions/PathUtil.java: ## @@ -61,4 +74,59 @@ static List parse(String path) { return names; } + + public static

[PR] API: Use normalized JSON path to identify Variant fields [iceberg]

2025-04-17 Thread via GitHub
rdblue opened a new pull request, #12835: URL: https://github.com/apache/iceberg/pull/12835 This updates the Parquet metrics conversion to use normalized JSON paths instead of joining field names using `.`. A normalized path is a more reliable representation that preserves the distinction b