Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor(IcebergTableOperatorUpsert): use DeleteWriteBuilder.forTable instead of setting table params manually #44

Merged
merged 1 commit into from
Oct 25, 2021
Merged

refactor(IcebergTableOperatorUpsert): use DeleteWriteBuilder.forTable instead of setting table params manually #44

merged 1 commit into from
Oct 25, 2021

Commits on Oct 25, 2021

  1. refactor(IcebergTableOperatorUpsert): use `DeleteWriteBuilder.forTabl… 

    …e` instead of setting table params manually

We were setting the parameters "manually" when there is a `forTable`
helper in the iceberg API.

Also, this refactors the `getDeleteFile` to return a `Optional`,
providing more clarity that there can be no delete file for a given
batch
    @racevedoo
    racevedoo committed Oct 25, 2021
    Configuration menu
    Copy the full SHA
    9da7976 View commit details
    Browse the repository at this point in the history