Skip to content

Conversation

@iwanttobepowerful
Copy link
Contributor

@iwanttobepowerful
Copy link
Contributor Author

@suibianwanwank @mihaibudiu @rubenada @NobiGo
can you help review this PR? Thx.

@iwanttobepowerful iwanttobepowerful force-pushed the CALCITE-7394 branch 2 times, most recently from f872b5c to 46c7a76 Compare January 25, 2026 10:11
Copy link
Member

@xuzifu666 xuzifu666 left a comment

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud
Copy link

@mihaibudiu mihaibudiu added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Jan 27, 2026
@mihaibudiu mihaibudiu merged commit 47fdf61 into apache:main Jan 27, 2026
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants