Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue merged PR #12304: URL: https://github.com/apache/iceberg/pull/12304 -- 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...@iceberg.ap

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on PR #12304: URL: https://github.com/apache/iceberg/pull/12304#issuecomment-2673141341 Thanks for the reviews, @danielcweeks and @aihuaxu! -- 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

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964535688 ## api/src/main/java/org/apache/iceberg/variants/VariantData.java: ## @@ -0,0 +1,67 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more c

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964535327 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ VariantValue

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964535894 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964535327 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ VariantValue

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964492886 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ VariantValue

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964492397 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964386149 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964390948 ## api/src/main/java/org/apache/iceberg/variants/VariantData.java: ## @@ -0,0 +1,67 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964388546 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964387424 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964386149 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964369050 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ Variant

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964346663 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ Variant

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1963985031 ## api/src/main/java/org/apache/iceberg/expressions/BoundExtract.java: ## @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964228087 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ VariantValue

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1964226588 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1963985031 ## api/src/main/java/org/apache/iceberg/expressions/BoundExtract.java: ## @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1963978534 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ VariantValue

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1963955290 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1963955290 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or mo

Re: [PR] API: Move variant to API and add extract expression [iceberg]

rdblue commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1963953648 ## api/src/main/java/org/apache/iceberg/expressions/UnboundPredicate.java: ## @@ -124,13 +125,17 @@ public Expression bind(StructType struct, boolean caseSensitive) {

Re: [PR] API: Move variant to API and add extract expression [iceberg]

aihuaxu commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1960965228 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] API: Move variant to API and add extract expression [iceberg]

aihuaxu commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1961993026 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] API: Move variant to API and add extract expression [iceberg]

aihuaxu commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1960945852 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ VariantValue

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1960239029 ## api/src/main/java/org/apache/iceberg/expressions/BoundExtract.java: ## @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1960220240 ## api/src/main/java/org/apache/iceberg/variants/VariantDataUtil.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] API: Move variant to API and add extract expression [iceberg]

danielcweeks commented on code in PR #12304: URL: https://github.com/apache/iceberg/pull/12304#discussion_r1960211875 ## api/src/main/java/org/apache/iceberg/variants/Variant.java: ## @@ -25,4 +25,24 @@ public interface Variant { /** Returns the variant value. */ Variant

[PR] API: Move variant to API and add extract expression [iceberg]

rdblue opened a new pull request, #12304: URL: https://github.com/apache/iceberg/pull/12304 This adds `Expressions.extract` to extract a value from a variant in Iceberg filters. The new method, `Expressions.extract(column, path, type)`, accepts a column name, a JSON path, and a type.