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

[native] Minor changes in PrestoTask. #18760

Merged
merged 1 commit into from
Dec 3, 2022
Merged

[native] Minor changes in PrestoTask. #18760

merged 1 commit into from
Dec 3, 2022

Conversation

amitkdutta
Copy link
Contributor

No description provided.

@amitkdutta amitkdutta requested a review from a team as a code owner December 2, 2022 21:10
@amitkdutta amitkdutta requested a review from xiaoxmeng December 2, 2022 21:10
mshang816
mshang816 approved these changes Dec 2, 2022
View reviewed changes
Copy link
Contributor

@mshang816 mshang816 left a comment

Choose a reason for hiding this comment

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

LGTM

ajaygeorge
ajaygeorge approved these changes Dec 3, 2022
View reviewed changes
Copy link
Contributor

@ajaygeorge ajaygeorge left a comment

Choose a reason for hiding this comment

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

LGTM

@amitkdutta amitkdutta merged commit a586c90 into prestodb:master Dec 3, 2022
@wanglinsong wanglinsong mentioned this pull request Jan 12, 2023
Merged
30 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajaygeorge ajaygeorge ajaygeorge approved these changes

@mshang816 mshang816 mshang816 approved these changes

@xiaoxmeng xiaoxmeng Awaiting requested review from xiaoxmeng

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@amitkdutta @ajaygeorge @mshang816