aiborodin
aiborodin
According to the profile in my previous comment https://github.com/apache/iceberg/pull/13340#discussion_r2156145524, schema caching would not be sufficient and we also need to cache field accessors and converters to minimise the CPU overhead....
Thank you for merging and reviewing the change @pvary! I appreciate your and @mxm's valuable feedback, and it's a pleasure to have you as reviewers. I raised this PR to...
@rdblue What do you think?
Thank you for the review @rdblue, I appreciate your feedback! Given that https://github.com/apache/iceberg/pull/14509 has been merged, we should probably go ahead with that API. I still think there's value in...
@rdblue What do you think?
> I think that difference could lead to needing two separate values when combining history validation with other validations that use startingSnapshotId. Wouldn't both the history validation and other validations...
> This validation is likely going to be based on the current version when the transaction started. But would it be reasonable for the operation that has read a different...