Skip to content

[CALCITE-3592] Implement bitnot scalar function#1672

Open
wangxlong wants to merge 1 commit intoapache:mainfrom
wangxlong:CALCITE-3592
Open

[CALCITE-3592] Implement bitnot scalar function#1672
wangxlong wants to merge 1 commit intoapache:mainfrom
wangxlong:CALCITE-3592

Conversation

@wangxlong
Copy link
Contributor

@wangxlong wangxlong commented Dec 17, 2019

Implement bitnot scalar function
See jira CALCIT-3592

@wangxlong wangxlong force-pushed the CALCITE-3592 branch 2 times, most recently from 3f74be2 to c959e07 Compare December 18, 2019 05:11
public class SqlBitNotOperator extends SqlPrefixOperator {

SqlBitNotOperator() {
super("~", SqlKind.BITNOT, 0, ReturnTypes.ARG0_NULLABLE_IF_EMPTY,
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO, no need to create an extra class.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you @chunweilei . I have delete extra class.

@wangxlong wangxlong requested a review from chunweilei December 18, 2019 13:12
@vlsi vlsi force-pushed the master branch 2 times, most recently from 49cb002 to 8768a23 Compare December 29, 2019 12:07
@wangxlong wangxlong changed the title [CALCITE-3592]Babel parser support "~" SqlPrefixOperator for bitwise not [CALCITE-3592]Implement bitnot scalar function Feb 7, 2020
@danny0405 danny0405 changed the title [CALCITE-3592]Implement bitnot scalar function [CALCITE-3592] Implement bitnot scalar function Feb 17, 2020
Copy link
Contributor

@danny0405 danny0405 left a comment

Choose a reason for hiding this comment

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

+1, from the doc of SnowFlake, there is not need to add any syntax extension to Babel ~

[1] https://docs.snowflake.net/manuals/sql-reference/functions/bitnot.html

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