Skip to content

fix new requirements handling - #1631

Merged
Fokko merged 2 commits into
apache:mainfrom
ZENOTME:fix_requirement
Feb 9, 2025
Merged

fix new requirements handling#1631
Fokko merged 2 commits into
apache:mainfrom
ZENOTME:fix_requirement

Conversation

@ZENOTME

Copy link
Copy Markdown
Contributor

Resolve#1628.

@ZENOTME

Copy link
Copy Markdown
ContributorAuthor

cc @kevinjqliu@Fokko@HonahX

@kevinjqliukevinjqliu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this!

Comment threadpyiceberg/table/__init__.py Outdated

@FokkoFokko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oof, good catch @ZENOTME

Co-authored-by: Kevin Liu <kevinjqliu@users.noreply.github.com>

@HonahXHonahX left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this!

@Fokko
Fokko merged commit dd175aa into apache:mainFeb 9, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] Transaction new requirements handling

4 participants

@ZENOTME@Fokko@kevinjqliu@HonahX