rorueda commented on code in PR #3653:
URL: https://github.com/apache/calcite/pull/3653#discussion_r1518608296
##
core/src/test/java/org/apache/calcite/util/format/FormatElementEnumTest.java:
##
@@ -77,8 +77,32 @@ class FormatElementEnumTest {
jduo commented on PR #3712:
URL: https://github.com/apache/calcite/pull/3712#issuecomment-1987007339
Would it be possible to merge these changes? I'm looking to implement
CALCITE-6308 on top of this.
--
This is an automated message from the Apache Git Service.
To respond to the message,
sonarcloud[bot] commented on PR #3653:
URL: https://github.com/apache/calcite/pull/3653#issuecomment-1986901137
## [ {
rorueda commented on PR #3653:
URL: https://github.com/apache/calcite/pull/3653#issuecomment-1986898121
Since I had to rebase it onto master to address the last change request, I
squashed the commits.
--
This is an automated message from the Apache Git Service.
To respond to the message,
julianhyde commented on PR #3653:
URL: https://github.com/apache/calcite/pull/3653#issuecomment-1986932598
It always struck me as strange that this change is using Calendar. Day of
the week is simple if you convert to Julian date. JulianOn Mar 9, 2024, at
07:59, rorueda ***@***.***>
rubenada commented on PR #3435:
URL: https://github.com/apache/calcite/pull/3435#issuecomment-1986958025
Thanks @mihaibudiu , no further comment from my side. LGTM.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
caicancai commented on PR #3705:
URL: https://github.com/apache/calcite/pull/3705#issuecomment-1987040158
@chucheng92 I fixed a similar problem. If you have time, can you help me
take a look at this PR? Thank you.
https://github.com/apache/calcite/pull/3721
--
This is an automated
JiajunBernoulli commented on code in PR #3712:
URL: https://github.com/apache/calcite/pull/3712#discussion_r1518707371
##
site/_docs/reference.md:
##
@@ -2821,8 +2821,10 @@ In the following:
| b m p s | REPEAT(string, integer) | Returns a string
consisting of
JiajunBernoulli merged PR #3722:
URL: https://github.com/apache/calcite/pull/3722
--
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.
To unsubscribe, e-mail:
This is an automated email from the ASF dual-hosted git repository.
jiajunxie pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/calcite.git
The following commit(s) were added to refs/heads/main by this push:
new ad08ce5fbd Link to calcite notes about runnable
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch site
in repository https://gitbox.apache.org/repos/asf/calcite.git
The following commit(s) were added to refs/heads/site by this push:
new fdd201f7c4 Link to calcite notes about
This is an automated email from the ASF dual-hosted git repository.
asf-ci-deploy pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/calcite-site.git
The following commit(s) were added to refs/heads/main by this push:
new af602a693 Website deployed from
YiwenWu commented on PR #3712:
URL: https://github.com/apache/calcite/pull/3712#issuecomment-1987103098
> Would it be possible to merge these changes? I'm looking to implement
[CALCITE-6309](https://issues.apache.org/jira/browse/CALCITE-6309) on top of
this.
@jduo I think this mr
sonarcloud[bot] commented on PR #3712:
URL: https://github.com/apache/calcite/pull/3712#issuecomment-1987104572
## [ | Returns a string
consisting of
caicancai commented on code in PR #3721:
URL: https://github.com/apache/calcite/pull/3721#discussion_r1518535007
##
core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java:
##
@@ -1149,6 +1149,56 @@ private static RelDataType
mapReturnType(SqlOperatorBinding
sonarcloud[bot] commented on PR #3711:
URL: https://github.com/apache/calcite/pull/3711#issuecomment-1986800959
## [