chucheng92 commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1867156919
The commit name "Add DATE_SUB test and DATE_DIFF" but you actually add
DATE_ADD and DATE_DIFF. could you recheck it?
--
This is an automated message from the Apache Git Service.
To
chucheng92 commented on code in PR #3586:
URL: https://github.com/apache/calcite/pull/3586#discussion_r1433384145
##
testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java:
##
@@ -11859,6 +11859,42 @@ void testTimestampDiff(boolean coercionEnabled) {
sonarcloud[bot] commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1866441880
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
sonarcloud[bot] commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1866332917
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
sonarcloud[bot] commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1866326301
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
caicancai commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1866257065
@chucheng92 Regarding the failure information related to date_add(date
'2011-02-24', CAST(NULL AS interval)), I will add it in the next pr. Because
date_sub does not have a
caicancai commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1865397151
> could you also check reference.md?
reference.md has the corresponding function description
--
This is an automated message from the Apache Git Service.
To respond to
chucheng92 commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1865396363
could you also check reference.md?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
caicancai commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1865358655
cc @chucheng92
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
sonarcloud[bot] commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1864561154
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
caicancai commented on code in PR #3586:
URL: https://github.com/apache/calcite/pull/3586#discussion_r1432731370
##
testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java:
##
@@ -11920,6 +11957,24 @@ void testTimestampDiff(boolean coercionEnabled) {
caicancai commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1863778855
> Can u change your PR name to `Add DATE_SUB test and DATE_DIFF test on
SqlOperatorTest`? It is more recommended to use uppercase letters for function
names.
Ok,thank you
--
chucheng92 commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1863778061
Can u change your PR name to `Add DATE_SUB test and DATE_DIFF test on
SqlOperatorTest`, It is more recommended to use uppercase letters for function
names.
--
This is an automated
chucheng92 commented on code in PR #3586:
URL: https://github.com/apache/calcite/pull/3586#discussion_r1432198009
##
testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java:
##
@@ -11920,6 +11957,24 @@ void testTimestampDiff(boolean coercionEnabled) {
sonarcloud[bot] commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1862897559
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
caicancai commented on PR #3586:
URL: https://github.com/apache/calcite/pull/3586#issuecomment-1862857142
> I didn’t see implementation-related logic in your PR, so I can understand
that this PR just adds test cases of `date_diff` and `date_add` to
SqlOperatorTest, right? In this case, it
16 matches
Mail list logo