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 org.apache.calci
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: commits-uns
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 ARRAY_CO
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: commits-uns
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 d
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] AggregateExpandDistinc
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: commits-uns
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 AggregateProjectMergeRu
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 nullifyType(J
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 nullifyType(J
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 spe
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 Softwar
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 Git
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.
T
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: commits-unsubs
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?
H
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 nullifyTyp
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 run
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