-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[OPTIQ-282] {fn QUARTER(date)} is unsupported #13
Conversation
Hello Committers, Could you please review ? |
I added a test for null values tester.checkNull("quarter(cast(null as date))"); and got an error. I think that if you change RexNode res =
rexBuilder.makeCall(
resType,
SqlStdOperatorTable.EXTRACT_DATE,
ImmutableList.of(rexBuilder.makeFlag(TimeUnitRange.MONTH), x)); and remote the |
I also wanted to check whether the JDBC escape syntax works. I added tester.checkScalar("{fn QUARTER(DATE '1969-1-1')}", null, ""); and it failed. You need to add an entry to JdbcToInternalLookupTable. The patch looks good, and we're close. Fix these two things and I will commit. |
Checked in as a059f76. |
formatting changes calcite-2552
No description provided.