XiaoHongbo-Hope commented on code in PR #1009:
URL: https://github.com/apache/paimon-rust/pull/1009#discussion_r4175864730


##########
bindings/python/python/pypaimon_rust/datafusion.pyi:
##########
@@ -87,7 +87,14 @@ class TableRead:
     def read(self, splits: Sequence[Split]) -> List[pyarrow.RecordBatch]: ...
 
 class ReadBuilder:
-    def with_projection(self, columns: List[str]) -> "ReadBuilder": ...
+    def with_projection(

Review Comment:
   Thanks, updated in 8fdc6bd. The Rust Python binding now exposes 
`with_read_type` for a Paimon ROW read type and no longer adds a 
Variant-specific argument to `with_projection`. PyPaimon #10336 converts its 
existing public `variant_fields` option to that read type. Local binding tests 
(112) and paired PyPaimon tests (247) pass.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to