ZENOTME commented on PR #401: URL: https://github.com/apache/iceberg-rust/pull/401#issuecomment-2171684939
> > Can we at least split the code that makes `Predicate` serializable out of this PR into it's own PR? I think that is pretty uncontroversial and very useful on its own. > > I agree with @sdd about this and suggest to split it into three prs: > > 1. Ser/de of datum > 2. Ser/de of expression > 3. Refine arrow builder > > We can still keep this pr open and waiting for 1 and 2 to get merged. I separate a PR for 1 and 2 #406 because I find that 2 is dependent on 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-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
