liurenjie1024 commented on PR #960: URL: https://github.com/apache/iceberg-rust/pull/960#issuecomment-2646853950
> @liurenjie1024 I believe FastAppend is for directly appending `DataFiles` to the snapshot, whereas this pr takes existing parquet file paths, parses the Parquet metadata and converts it into `DataFiles` which are then fast appended. @ZENOTME Would you like to verify this? I have concerns to put this into transaction api. It seems that what's necessary is to build a `DataFile` from exisitng parquet file, then user could call fast append to do it. But this is typically a dangerous operation because the schema in parquet is not verified. Is it possible to use `FileWriter` to write data to parquet in your case? -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
