Skip to content

implement table OptimizeAsync#155

Merged
mightyshazam merged 2 commits intodelta-incubator:mainfrom
adamjones2:optimize
Aug 6, 2025
Merged

implement table OptimizeAsync#155
mightyshazam merged 2 commits intodelta-incubator:mainfrom
adamjones2:optimize

Conversation

@adamjones2
Copy link
Contributor

fulfills #154

@mightyshazam could you review?

mightyshazam
mightyshazam previously approved these changes Jul 29, 2025
Copy link
Collaborator

@mightyshazam mightyshazam left a comment

Choose a reason for hiding this comment

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

Lgtm

@adamjones2
Copy link
Contributor Author

Hey @mightyshazam, sorry after fixing some compiler warnings it dismissed your review, could you reapprove it?

@adamjones2
Copy link
Contributor Author

Thanks! Is there any blocker to merging it?

@mightyshazam
Copy link
Collaborator

Thanks! Is there any blocker to merging it?

No. I just assumed you were going to merge it.

@mightyshazam mightyshazam merged commit bf7ea30 into delta-incubator:main Aug 6, 2025
8 of 11 checks passed
@adamjones2
Copy link
Contributor Author

Cool! I would have but it looked like GitHub didn't give me permissions to as a non-maintainer, possibly I just didn't spot the button somewhere 😅.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants