Skip to content

feat: add apply in transaction to support stack action #22

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.

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

Merged
merged 3 commits into from
Mar 12, 2025

Conversation

ZENOTME
Copy link

@ZENOTME ZENOTME commented Mar 12, 2025

upstream track: apache#949

After this PR we can append multiple table update and it's require for #21

@ZENOTME ZENOTME force-pushed the zj/apply_metadata branch from ee61e4c to 8184255 Compare March 12, 2025 08:04
@ZENOTME
Copy link
Author

ZENOTME commented Mar 12, 2025

cc @Li0k

Copy link

@Li0k Li0k left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@ZENOTME ZENOTME force-pushed the zj/apply_metadata branch from 8184255 to 645afba Compare March 12, 2025 11:04
@ZENOTME ZENOTME requested review from chenzl25 and xxchan March 12, 2025 11:14
@Li0k Li0k merged commit 37e5618 into dev_rebase_main_20250307 Mar 12, 2025
11 checks passed
@chenzl25 chenzl25 mentioned this pull request Mar 13, 2025
Li0k added a commit that referenced this pull request Mar 25, 2025
Li0k pushed a commit that referenced this pull request Mar 25, 2025
* add apply in transaction to support stack action

* fix test

* store current table instead of current metadata

---------

Co-authored-by: ZENOTME <[email protected]>
xxchan pushed a commit that referenced this pull request Mar 25, 2025
* add apply in transaction to support stack action

* fix test

* store current table instead of current metadata

---------

Co-authored-by: ZENOTME <[email protected]>
xxchan pushed a commit that referenced this pull request Mar 25, 2025
* add apply in transaction to support stack action

* fix test

* store current table instead of current metadata

---------

Co-authored-by: ZENOTME <[email protected]>
Sign up for free to 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.

3 participants