[GitHub] [incubator-iceberg] rdblue commented on pull request #939: Support UpdateEvent for Update Operations

[GitBox <gi...@apache.org>]

rdblue commented on pull request #939:
URL: https://github.com/apache/incubator-iceberg/pull/939#issuecomment-620094481


   Thanks @rdsr, this looks mostly good, although I think the file lists in the event might be incomplete. I'd suggest removing them and just sending the summary, unless you have a use case for them.
   
   I'd also like to get fast appends and snapshot manager changes done as soon as we can -- ideally in this PR. That way, we can guarantee that all changes to table contents produce events. Events aren't as useful unless you know you're getting all of the changes.


----------------------------------------------------------------
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org