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.

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

[CALCITE-6224] Add LOG2 function (enabled in Mysql, Spark library) #3707

Closed
wants to merge 1 commit into from

Conversation

caicancai
Copy link
Member

No description provided.

@caicancai caicancai marked this pull request as draft February 27, 2024 06:17
@caicancai
Copy link
Member Author

caicancai commented Feb 27, 2024

I added tanclary as a co-author, maybe I am not qualified to be called the author.
I will review it later to ensure there are no more problems with this PR

@caicancai caicancai closed this Feb 27, 2024
@caicancai caicancai reopened this Feb 27, 2024
@caicancai caicancai marked this pull request as ready for review February 27, 2024 09:29
@caicancai caicancai marked this pull request as draft February 27, 2024 10:25
Copy link
Contributor

@tanclary tanclary left a comment

Choose a reason for hiding this comment

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

I think this looks good. Thanks for fixing all of the issues. I will approve but allow time for any other comments.

@caicancai caicancai marked this pull request as ready for review February 28, 2024 01:50
@JiajunBernoulli
Copy link
Contributor

LGTM

@JiajunBernoulli
Copy link
Contributor

Please squash them, we can merge it soon.

@caicancai
Copy link
Member Author

Please squash them, we can merge it soon.

done

@tanclary tanclary closed this in c546d4a Feb 29, 2024
@tanclary
Copy link
Contributor

@caicancai I just squashed and merged you should find the commit in the main branch now, thanks!

@caicancai
Copy link
Member Author

@caicancai I just squashed and merged you should find the commit in the main branch now, thanks!

@tanclary I am very grateful for your help in this PR. I will soon improve the log function of mysql. Action is the best feedback.

@caicancai caicancai deleted the help-caican-6224 branch February 29, 2024 02:00
Copy link

sonarcloud bot commented Feb 29, 2024

github-actions bot pushed a commit to macroguo-ghy/calcite that referenced this pull request Mar 4, 2024
Co-authored-by: Tanner Clary <tannerclary@google.com>
close apache#3707
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