mihaibudiu commented on PR #3264:
URL: https://github.com/apache/calcite/pull/3264#issuecomment-2224683145
This PR was approved, but never merged. What's the plan?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
This is an automated email from the ASF dual-hosted git repository.
mbudiu 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 374091b0a7 [CALCITE-6449] Enable PostgreSQL impleme
mihaibudiu merged PR #3833:
URL: https://github.com/apache/calcite/pull/3833
--
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-unsubscr...@calcite
sonarcloud[bot] commented on PR #3851:
URL: https://github.com/apache/calcite/pull/3851#issuecomment-2224267096
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
Passed')](https://sonarclo
NobiGo commented on code in PR #3851:
URL: https://github.com/apache/calcite/pull/3851#discussion_r1674917200
##
core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java:
##
@@ -982,11 +982,17 @@ void propagateCostImprovements(RelNode rel) {
if (!relNode.g
sonarcloud[bot] commented on PR #3851:
URL: https://github.com/apache/calcite/pull/3851#issuecomment-2224236338
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
Passed')](https://sonarclo
NobiGo commented on PR #3852:
URL: https://github.com/apache/calcite/pull/3852#issuecomment-2224235956
@normanj-bitquill
1) I find different data source has different behaviour about NULL value:
```
spark-sql (default)> select array(1,2)>array(1,null);
true
Time taken:
cyrilou242 commented on code in PR #3853:
URL: https://github.com/apache/calcite/pull/3853#discussion_r1674756688
##
ubenchmark/src/jmh/java/org/apache/calcite/benchmarks/ParserInstantiationBenchmark.java:
##
@@ -0,0 +1,115 @@
+/*
+ * Licensed to the Apache Software Foundation (
cyrilou242 commented on code in PR #3853:
URL: https://github.com/apache/calcite/pull/3853#discussion_r1674753689
##
ubenchmark/src/jmh/java/org/apache/calcite/benchmarks/ParserInstantiationBenchmark.java:
##
@@ -0,0 +1,115 @@
+/*
+ * Licensed to the Apache Software Foundation (
cyrilou242 commented on code in PR #3853:
URL: https://github.com/apache/calcite/pull/3853#discussion_r1674747392
##
ubenchmark/src/jmh/java/org/apache/calcite/benchmarks/ParserInstantiationBenchmark.java:
##
@@ -0,0 +1,115 @@
+/*
+ * Licensed to the Apache Software Foundation (
cyrilou242 commented on code in PR #3853:
URL: https://github.com/apache/calcite/pull/3853#discussion_r1674746023
##
ubenchmark/src/jmh/java/org/apache/calcite/benchmarks/ParserInstantiationBenchmark.java:
##
@@ -0,0 +1,115 @@
+/*
+ * Licensed to the Apache Software Foundation (
cyrilou242 commented on code in PR #3853:
URL: https://github.com/apache/calcite/pull/3853#discussion_r1674719255
##
ubenchmark/src/jmh/java/org/apache/calcite/benchmarks/ParserInstantiationBenchmark.java:
##
@@ -0,0 +1,115 @@
+/*
+ * Licensed to the Apache Software Foundation (
cyrilou242 commented on PR #3853:
URL: https://github.com/apache/calcite/pull/3853#issuecomment-2223972517
> Thanks for the benchmark @cyrilou242 ! I left some small comments but
looks good.
>
> The purpose of this benchmark is different from that in
[CALCITE-1012](https://issues.apa
normanj-bitquill commented on code in PR #3801:
URL: https://github.com/apache/calcite/pull/3801#discussion_r1674707220
##
core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java:
##
@@ -563,12 +563,58 @@ static RelDataType deriveTypeSplit(SqlOperatorBinding
oper
normanj-bitquill commented on code in PR #3801:
URL: https://github.com/apache/calcite/pull/3801#discussion_r1674706865
##
core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java:
##
@@ -563,12 +563,58 @@ static RelDataType deriveTypeSplit(SqlOperatorBinding
oper
normanj-bitquill commented on code in PR #3801:
URL: https://github.com/apache/calcite/pull/3801#discussion_r1674706234
##
core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java:
##
@@ -563,12 +563,58 @@ static RelDataType deriveTypeSplit(SqlOperatorBinding
oper
normanj-bitquill commented on PR #3801:
URL: https://github.com/apache/calcite/pull/3801#issuecomment-2223905686
Updated with the following behaviour:
### Amazon Redshift
* Can provide only two arguments. The replacement string will be the empty
string.
https://docs.aws.amaz
normanj-bitquill commented on code in PR #3833:
URL: https://github.com/apache/calcite/pull/3833#discussion_r1674648077
##
core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java:
##
@@ -1690,22 +1690,40 @@ private static RelDataType
deriveTypeMapFromEntries(SqlO
normanj-bitquill commented on PR #3852:
URL: https://github.com/apache/calcite/pull/3852#issuecomment-2223846601
> the natural question is whether this works for other recursive types, such
as `int array array`...
@mihaibudiu I'm not sure. This PR currently handles arrays and rows. Th
normanj-bitquill commented on PR #3833:
URL: https://github.com/apache/calcite/pull/3833#issuecomment-2223756122
The commits have been squashed.
--
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 th
sonarcloud[bot] commented on PR #3833:
URL: https://github.com/apache/calcite/pull/3833#issuecomment-2223740736
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
Passed')](https://sonarclo
sonarcloud[bot] commented on PR #3801:
URL: https://github.com/apache/calcite/pull/3801#issuecomment-2223702244
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
Passed')](https://sonarclo
This is an automated email from the ASF dual-hosted git repository.
asolimando 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 8581dba433 [CALCITE-6388] PsTableFunction throw
asolimando merged PR #3777:
URL: https://github.com/apache/calcite/pull/3777
--
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-unsubscr...@calcite
sonarcloud[bot] commented on PR #3777:
URL: https://github.com/apache/calcite/pull/3777#issuecomment-2223050330
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
Passed')](https://sonarclo
sonarcloud[bot] commented on PR #3777:
URL: https://github.com/apache/calcite/pull/3777#issuecomment-951164
## [![Quality Gate
Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png
'Quality Gate
Passed')](https://sonarclo
asolimando commented on PR #3777:
URL: https://github.com/apache/calcite/pull/3777#issuecomment-932790
The force push is just to change the latest commit message (forgot that it
must now start with an uppercase letter), no impact
--
This is an automated message from the Apache Git Ser
zabetak commented on code in PR #3853:
URL: https://github.com/apache/calcite/pull/3853#discussion_r1673961753
##
ubenchmark/src/jmh/java/org/apache/calcite/benchmarks/ParserInstantiationBenchmark.java:
##
@@ -0,0 +1,115 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
kgyrtkirk merged PR #3835:
URL: https://github.com/apache/calcite/pull/3835
--
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-unsubscr...@calcite.
This is an automated email from the ASF dual-hosted git repository.
kgyrtkirk 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 73846cceb1 [CALCITE-6435] SqlToRel conversion of
This is an automated email from the ASF dual-hosted git repository.
rubenql 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 a98508fd36 [CALCITE-6460] SortRemoveConstantKeysRu
rubenada merged PR #3850:
URL: https://github.com/apache/calcite/pull/3850
--
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-unsubscr...@calcite.a
cyrilou242 opened a new pull request, #3853:
URL: https://github.com/apache/calcite/pull/3853
This benchmark will help to measure the progress on the discussion here:
https://lists.apache.org/thread/xw35sdy1w1k8lvn1q1lr7xb93bkj0lpq
As of today the parsers instantiation is pretty slow an
33 matches
Mail list logo