Skip to content

Conversation

@cjj2010
Copy link

@cjj2010 cjj2010 commented Jan 31, 2026

/** Test case for
* <a href="https://issues.apache.org/jira/browse/CALCITE-7406">[CALCITE-7406]
* Add abs function (enabled in Mongodb library)</a>. */
@Test void testAbsAlisa() {
Copy link
Contributor

Choose a reason for hiding this comment

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

is this supposed to be "alias"?

Copy link
Author

Choose a reason for hiding this comment

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

is this supposed to be "alias"?

Yes, I made a spelling mistake and it has been corrected. Thank you

@xuzifu666 xuzifu666 added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Feb 2, 2026
@xuzifu666
Copy link
Member

I think you can squash the commits now.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 2, 2026

@cjj2010
Copy link
Author

cjj2010 commented Feb 2, 2026

I think you can squash the commits now.

done,thanks

@xuzifu666 xuzifu666 merged commit 383315b into apache:main Feb 2, 2026
20 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