[PR] [variant] Introduce withVariantAccess in ReadBuilder [paimon]

["Zouxxyy (via GitHub)" <gi...@apache.org>]

Zouxxyy opened a new pull request, #6685:
URL: https://github.com/apache/paimon/pull/6685

   <!-- Please specify the module before the PR name: [core] ... or [flink] ... -->
   
   ### Purpose
   
   new function in `ReadBuilder` for variant push down
   
   ```java
   /**
    * Push variant access to the reader.
    *
    * @param variantAccessInfo variant access info
    * @since 1.4.0
    */
   ReadBuilder withVariantAccess(VariantAccessInfo[] variantAccessInfo);
   ```
   
   `VariantAccessInfo`
   
   ```java
   /** Variant access information for a variant column. */
   public class VariantAccessInfo implements Serializable {
   
       private static final long serialVersionUID = 1L;
   
       // The name of the variant column.
       private final String columnName;
   
       // Extracted fields from the variant.
       private final List<VariantField> variantFields;
   
       /** Variant field extracted from the variant. */
       public static class VariantField implements Serializable {
   
           private static final long serialVersionUID = 1L;
   
           // The data field of the variant field.
           private final DataField dataField;
   
           // The `path` parameter of VariantGet.
           private final String path;
   
           private final VariantCastArgs castArgs;
   }
   ```
   
   ### Tests
   
   <!-- List UT and IT cases to verify this change -->
   
   ### API and Format
   
   <!-- Does this change affect API or storage format -->
   
   ### Documentation
   
   <!-- Does this change introduce a new feature -->
   


-- 
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-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [variant] Introduce withVariantAccess in ReadBuilder [paimon]

["JingsongLi (via GitHub)" <gi...@apache.org>]

JingsongLi commented on PR #6685:
URL: https://github.com/apache/paimon/pull/6685#issuecomment-3584213013

   +1


-- 
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-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [variant] Introduce withVariantAccess in ReadBuilder [paimon]

["JingsongLi (via GitHub)" <gi...@apache.org>]

JingsongLi commented on code in PR #6685:
URL: https://github.com/apache/paimon/pull/6685#discussion_r2564805325


##########
paimon-core/src/main/java/org/apache/paimon/table/source/ReadBuilder.java:
##########
@@ -79,6 +80,11 @@ public interface ReadBuilder extends Serializable {
     /** Returns read row type. */
     RowType readType();
 
+    /** Returns actual read row type. */
+    default RowType actualReadType() {

Review Comment:
   This interface is a bit confusing to me. What is the difference in usage between it and readType?
   



-- 
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-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [variant] Introduce withVariantAccess in ReadBuilder [paimon]

["JingsongLi (via GitHub)" <gi...@apache.org>]

JingsongLi merged PR #6685:
URL: https://github.com/apache/paimon/pull/6685


-- 
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-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org