You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by "xuzifu666 (via GitHub)" <gi...@apache.org> on 2025/09/15 12:16:26 UTC
[PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
xuzifu666 opened a new pull request, #4542:
URL: https://github.com/apache/calcite/pull/4542
jira: https://issues.apache.org/jira/browse/CALCITE-7186
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Character[] to VARCHAR in Java UDF [calcite]
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #4542:
URL: https://github.com/apache/calcite/pull/4542#issuecomment-3333226870
## [](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542) **Quality Gate passed**
Issues
 [2 New issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [8.3% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_duplicated_lines_density&view=list)
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542)
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "caicancai (via GitHub)" <gi...@apache.org>.
caicancai commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2349235965
##########
core/src/test/java/org/apache/calcite/sql/type/SqlTypeUtilTest.java:
##########
@@ -198,6 +198,20 @@ private void checkConvert(SqlTypeCoercionRule rules,
assertThat(fieldTypeNames, isListOf("INTEGER", "INTEGER"));
}
+ /**
+ * Test for <a href="https://issues.apache.org/jira/browse/CALCITE-7186">[CALCITE-7186]</a>
+ * Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF.
+ */
+ @Test void testJavaToSqlCharacterArrayAndByteArrayMapping() {
+ SqlTypeName sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Byte[].class);
+ assertThat("Byte[].class should map to SqlTypeName.VARBINARY",
+ sqlTypeName, is(SqlTypeName.VARBINARY));
+
+ sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Character[].class);
+ assertThat("Character[].class should map to SqlTypeName.VARBINARY",
Review Comment:
SqlTypeName.VARBINARY -> SqlTypeName.VARCHAR
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2349427121
##########
core/src/test/java/org/apache/calcite/sql/type/SqlTypeUtilTest.java:
##########
@@ -198,6 +198,20 @@ private void checkConvert(SqlTypeCoercionRule rules,
assertThat(fieldTypeNames, isListOf("INTEGER", "INTEGER"));
}
+ /**
+ * Test for <a href="https://issues.apache.org/jira/browse/CALCITE-7186">[CALCITE-7186]</a>
+ * Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF.
+ */
+ @Test void testJavaToSqlCharacterArrayAndByteArrayMapping() {
+ SqlTypeName sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Byte[].class);
+ assertThat("Byte[].class should map to SqlTypeName.VARBINARY",
+ sqlTypeName, is(SqlTypeName.VARBINARY));
+
+ sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Character[].class);
+ assertThat("Character[].class should map to SqlTypeName.VARBINARY",
Review Comment:
CI not error in previous version may be OK here, because it is just a error out message.
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #4542:
URL: https://github.com/apache/calcite/pull/4542#issuecomment-3292022401
## [](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [100.0% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_duplicated_lines_density&view=list)
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542)
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Character[] to VARCHAR in Java UDF [calcite]
Posted by "mihaibudiu (via GitHub)" <gi...@apache.org>.
mihaibudiu merged PR #4542:
URL: https://github.com/apache/calcite/pull/4542
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "caicancai (via GitHub)" <gi...@apache.org>.
caicancai commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2349419177
##########
core/src/test/java/org/apache/calcite/sql/type/SqlTypeUtilTest.java:
##########
@@ -198,6 +198,20 @@ private void checkConvert(SqlTypeCoercionRule rules,
assertThat(fieldTypeNames, isListOf("INTEGER", "INTEGER"));
}
+ /**
+ * Test for <a href="https://issues.apache.org/jira/browse/CALCITE-7186">[CALCITE-7186]</a>
+ * Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF.
+ */
+ @Test void testJavaToSqlCharacterArrayAndByteArrayMapping() {
+ SqlTypeName sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Byte[].class);
+ assertThat("Byte[].class should map to SqlTypeName.VARBINARY",
+ sqlTypeName, is(SqlTypeName.VARBINARY));
+
+ sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Character[].class);
+ assertThat("Character[].class should map to SqlTypeName.VARBINARY",
Review Comment:
There is another strange point. The CI did not report an error in the previous question. Maybe it is another error. I will find time to fix it.
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #4542:
URL: https://github.com/apache/calcite/pull/4542#issuecomment-3292712161
## [](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542) **Quality Gate passed**
Issues
 [0 New issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [100.0% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_duplicated_lines_density&view=list)
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542)
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Character[] to VARCHAR in Java UDF [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2354249569
##########
testkit/src/main/java/org/apache/calcite/util/Smalls.java:
##########
@@ -1492,4 +1492,24 @@ public static ByteString eval(String s) {
}
}
}
+
+ /** User-defined function that convert Character[] to VARCHAR. */
+ public static class CharacterArrayFunction {
Review Comment:
I have changed the title of the jira and pr. I recorded the problem with the byte[] type in the new jira https://issues.apache.org/jira/browse/CALCITE-7187. Here I changed the comment content to: User-defined function with return type Character[].
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "caicancai (via GitHub)" <gi...@apache.org>.
caicancai commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2349235965
##########
core/src/test/java/org/apache/calcite/sql/type/SqlTypeUtilTest.java:
##########
@@ -198,6 +198,20 @@ private void checkConvert(SqlTypeCoercionRule rules,
assertThat(fieldTypeNames, isListOf("INTEGER", "INTEGER"));
}
+ /**
+ * Test for <a href="https://issues.apache.org/jira/browse/CALCITE-7186">[CALCITE-7186]</a>
+ * Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF.
+ */
+ @Test void testJavaToSqlCharacterArrayAndByteArrayMapping() {
+ SqlTypeName sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Byte[].class);
+ assertThat("Byte[].class should map to SqlTypeName.VARBINARY",
+ sqlTypeName, is(SqlTypeName.VARBINARY));
+
+ sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Character[].class);
+ assertThat("Character[].class should map to SqlTypeName.VARBINARY",
Review Comment:
SqlTypeName.VARBINARY -> SqlTypeName.VARCHAR ？
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Character[] to VARCHAR in Java UDF [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 commented on PR #4542:
URL: https://github.com/apache/calcite/pull/4542#issuecomment-3333103853
> Can you please squash the commits?
@mihaibudiu OK, had squashed commits.
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 commented on PR #4542:
URL: https://github.com/apache/calcite/pull/4542#issuecomment-3299472198
> The issue talks about user-defined functions, but there is no test with user-defined functions.
I had added udf test for it, and move discussion to jira.
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "mihaibudiu (via GitHub)" <gi...@apache.org>.
mihaibudiu commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2353239167
##########
testkit/src/main/java/org/apache/calcite/util/Smalls.java:
##########
@@ -1492,4 +1492,24 @@ public static ByteString eval(String s) {
}
}
}
+
+ /** User-defined function that convert Character[] to VARCHAR. */
+ public static class CharacterArrayFunction {
Review Comment:
I am not sure I understand this comment; Character[] is a Java type, while VARCHAR is a SQL type.
Is there a test for the BINARY case which is described in the issue title?
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Character[] to VARCHAR in Java UDF [calcite]
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #4542:
URL: https://github.com/apache/calcite/pull/4542#issuecomment-3301271649
## [](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542) **Quality Gate passed**
Issues
 [3 New issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [6.2% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_duplicated_lines_density&view=list)
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542)
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2349246246
##########
core/src/test/java/org/apache/calcite/sql/type/SqlTypeUtilTest.java:
##########
@@ -198,6 +198,20 @@ private void checkConvert(SqlTypeCoercionRule rules,
assertThat(fieldTypeNames, isListOf("INTEGER", "INTEGER"));
}
+ /**
+ * Test for <a href="https://issues.apache.org/jira/browse/CALCITE-7186">[CALCITE-7186]</a>
+ * Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF.
+ */
+ @Test void testJavaToSqlCharacterArrayAndByteArrayMapping() {
+ SqlTypeName sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Byte[].class);
+ assertThat("Byte[].class should map to SqlTypeName.VARBINARY",
+ sqlTypeName, is(SqlTypeName.VARBINARY));
+
+ sqlTypeName = JavaToSqlTypeConversionRules.instance().lookup(Character[].class);
+ assertThat("Character[].class should map to SqlTypeName.VARBINARY",
Review Comment:
Thanks for reminder，it's my mistake.
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Character[] to VARCHAR in Java UDF [calcite]
Posted by "dssysolyatin (via GitHub)" <gi...@apache.org>.
dssysolyatin commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2356200942
##########
testkit/src/main/java/org/apache/calcite/util/Smalls.java:
##########
@@ -1492,4 +1492,24 @@ public static ByteString eval(String s) {
}
}
}
+
+ /** User-defined function with return type Character[]. */
+ public static class CharacterArrayFunction {
+ public static Character[] eval(String s) {
+ if (s == null) {
+ return null;
+ }
+ try {
+ Character[] characters = new Character[s.length()];
+ int index = 0;
Review Comment:
Use simple for loop
```
Character[] characters = new Character[s.length()];
for (int i = 0; i < s.length(); i++) {
characters[i] = s.charAt(i);
}
return characters;
```
##########
testkit/src/main/java/org/apache/calcite/util/Smalls.java:
##########
@@ -1492,4 +1492,24 @@ public static ByteString eval(String s) {
}
}
}
+
+ /** User-defined function with return type Character[]. */
+ public static class CharacterArrayFunction {
+ public static Character[] eval(String s) {
+ if (s == null) {
+ return null;
+ }
+ try {
Review Comment:
Why do we need a try-catch here? If we really need it, why does it return null instead of a meaningful error?
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Character[] to VARCHAR in Java UDF [calcite]
Posted by "xuzifu666 (via GitHub)" <gi...@apache.org>.
xuzifu666 commented on code in PR #4542:
URL: https://github.com/apache/calcite/pull/4542#discussion_r2357363035
##########
testkit/src/main/java/org/apache/calcite/util/Smalls.java:
##########
@@ -1492,4 +1492,24 @@ public static ByteString eval(String s) {
}
}
}
+
+ /** User-defined function with return type Character[]. */
+ public static class CharacterArrayFunction {
+ public static Character[] eval(String s) {
+ if (s == null) {
+ return null;
+ }
+ try {
+ Character[] characters = new Character[s.length()];
+ int index = 0;
Review Comment:
Thanks for your review and I had addressed them. @dssysolyatin
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [CALCITE-7186] Add mapping from Character[] to VARCHAR in Java UDF [calcite]
Posted by "sonarqubecloud[bot] (via GitHub)" <gi...@apache.org>.
sonarqubecloud[bot] commented on PR #4542:
URL: https://github.com/apache/calcite/pull/4542#issuecomment-3305243428
## [](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542) **Quality Gate passed**
Issues
 [2 New issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [0 Accepted issues](https://sonarcloud.io/project/issues?id=apache_calcite&pullRequest=4542&issueStatuses=ACCEPTED)
Measures
 [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache_calcite&pullRequest=4542&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true)
 [8.3% Coverage on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_coverage&view=list)
 [0.0% Duplication on New Code](https://sonarcloud.io/component_measures?id=apache_calcite&pullRequest=4542&metric=new_duplicated_lines_density&view=list)
[See analysis details on SonarQube Cloud](https://sonarcloud.io/dashboard?id=apache_calcite&pullRequest=4542)
--
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-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org