jamesstarr commented on a change in pull request #2585:
URL: https://github.com/apache/calcite/pull/2585#discussion_r737982613
##
File path:
core/src/main/java/org/apache/calcite/rel/metadata/janino/DescriptiveCacheKey.java
##
@@ -16,14 +16,18 @@
*/
package
chunweilei merged pull request #2559:
URL: https://github.com/apache/calcite/pull/2559
--
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.
chunwei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git
The following commit(s) were added to refs/heads/master by this push:
new f61541d [CALCITE-4822] Add functions
chunweilei closed pull request #2589:
URL: https://github.com/apache/calcite/pull/2589
--
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.
chunwei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git
The following commit(s) were added to refs/heads/master by this push:
new 5bec337 Fix typo in reference.md
5bec337 is
This is an automated email from the ASF dual-hosted git repository.
chunwei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git
The following commit(s) were added to refs/heads/master by this push:
new 3af1d78 [CALCITE-4818]
chunweilei closed pull request #2560:
URL: https://github.com/apache/calcite/pull/2560
--
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:
xy2953396112 commented on a change in pull request #2546:
URL: https://github.com/apache/calcite/pull/2546#discussion_r737969304
##
File path:
core/src/main/java/org/apache/calcite/rel/rules/AggregateProjectMergeRule.java
##
@@ -125,14 +125,17 @@ public
NobiGo commented on a change in pull request #2559:
URL: https://github.com/apache/calcite/pull/2559#discussion_r737959127
##
File path:
core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java
##
@@ -2657,6 +2664,38 @@ private static RelDataType
NobiGo commented on a change in pull request #2559:
URL: https://github.com/apache/calcite/pull/2559#discussion_r737958944
##
File path:
core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java
##
@@ -2657,6 +2664,38 @@ private static RelDataType
NobiGo commented on a change in pull request #2559:
URL: https://github.com/apache/calcite/pull/2559#discussion_r737958837
##
File path: site/_docs/reference.md
##
@@ -2513,6 +2513,9 @@ semantics.
| C | Operator syntax| Description
|:-
joshelser commented on pull request #132:
URL: https://github.com/apache/calcite-avatica/pull/132#issuecomment-953398364
Also, fyi @stoty
--
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
jcamachor commented on a change in pull request #2585:
URL: https://github.com/apache/calcite/pull/2585#discussion_r737893951
##
File path:
core/src/main/java/org/apache/calcite/rel/metadata/janino/CacheGenerator.java
##
@@ -0,0 +1,419 @@
+/*
+ * Licensed to the Apache
jbalint commented on pull request #2560:
URL: https://github.com/apache/calcite/pull/2560#issuecomment-953114778
@tledkov-gridgain can you squash (and rebase) and then we'll merge?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
asolimando commented on pull request #2592:
URL: https://github.com/apache/calcite/pull/2592#issuecomment-953100170
Never mind, I have added the description because there was a conflict to fix
with master (gradle.properties).
--
This is an automated message from the Apache Git Service.
This is an automated email from the ASF dual-hosted git repository.
zabetak pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git
The following commit(s) were added to refs/heads/master by this push:
new 2280879 [CALCITE-4858] Use Log4j2 instead of
zabetak closed pull request #2587:
URL: https://github.com/apache/calcite/pull/2587
--
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:
asolimando edited a comment on pull request #2592:
URL: https://github.com/apache/calcite/pull/2592#issuecomment-952730479
On a second thought, there are some important info in the PR description
that I think should be in the git description as well, could you add it when
merging the PR?
asolimando commented on pull request #2592:
URL: https://github.com/apache/calcite/pull/2592#issuecomment-952730479
On a second thought, there are some important info in the PR description
that I think should be in the git description as well, could you add it when
merging the PR?
snuyanzin commented on a change in pull request #2559:
URL: https://github.com/apache/calcite/pull/2559#discussion_r737287102
##
File path:
core/src/main/java/org/apache/calcite/adapter/enumerable/RexImpTable.java
##
@@ -2657,6 +2664,38 @@ private static RelDataType
snuyanzin commented on a change in pull request #2559:
URL: https://github.com/apache/calcite/pull/2559#discussion_r737273675
##
File path: site/_docs/reference.md
##
@@ -2513,6 +2513,9 @@ semantics.
| C | Operator syntax| Description
|:-
asolimando edited a comment on pull request #2592:
URL: https://github.com/apache/calcite/pull/2592#issuecomment-952638170
Thank you both for the review, I have squashed the commits and updated the
commit message, once the test are over (even though there are no code changes
since last
asolimando commented on pull request #2592:
URL: https://github.com/apache/calcite/pull/2592#issuecomment-952638170
Thank you both for the review, I have squashed the commits and updated the
commit message, once the test are over I think it can be merged.
--
This is an automated message
23 matches
Mail list logo