[GitHub] [calcite-avatica] stoty commented on pull request #173: CALCITE-5009 Transparent JDBC connection re-creation may lead to data…

2022-03-31 Thread GitBox


stoty commented on pull request #173:
URL: https://github.com/apache/calcite-avatica/pull/173#issuecomment-1085419191


   thank you @F21.


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] F21 commented on pull request #59: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


F21 commented on pull request #59:
URL: 
https://github.com/apache/calcite-avatica-go/pull/59#issuecomment-1085415114


   Just a small nit, can you remove the period at the end of the commit 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-unsubscr...@calcite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong commented on pull request #59: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong commented on pull request #59:
URL: 
https://github.com/apache/calcite-avatica-go/pull/59#issuecomment-1085413896


   > Can you amend the commit message? Please make it `[CALCITE-5077] 
ResetSession implements driver.SessionResetter`
   
   done


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong commented on pull request #57: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085402153


   > 
   
   thanks for your help


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong closed pull request #57: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong closed pull request #57:
URL: https://github.com/apache/calcite-avatica-go/pull/57


   


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] F21 commented on pull request #59: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


F21 commented on pull request #59:
URL: 
https://github.com/apache/calcite-avatica-go/pull/59#issuecomment-1085401723


   Can you amend the commit message? Please make it `[CALCITE-5077] 
ResetSession implements driver.SessionResetter`


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] F21 commented on pull request #57: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


F21 commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085401306


   @liguozhong That's no problem. Let's move the conversation to #59 


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong commented on pull request #57: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085400967


   > Looks good to me! Please squash your commits and make the commit message 
the same as the title of the PR.
   
   https://github.com/apache/calcite-avatica-go/pull/59
   @F21  Hi, I have some problems with rebase, I submitted a new 
PR(https://github.com/apache/calcite-avatica-go/pull/59), can I close this PR 
by merging another PR, I am not good at using rebase to merge multiple 
commits.thanks


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong opened a new pull request #59: [CALCITE-5077] ResetSession implements driver.SessionResetter. #59

2022-03-31 Thread GitBox


liguozhong opened a new pull request #59:
URL: https://github.com/apache/calcite-avatica-go/pull/59


   avatica sql query fails after running for a period of time on our online 
loki, this PR attempts to solve this "NoSuchConnectionException" problem.
   
   level=error ts=2022-03-31T12:51:09.356233496Z caller=table_manager.go:233 
msg="error syncing tables" err="An error was encountered while processing your 
request: NoSuchConnectionException{connectionId='Connection not found: invalid 
id, closed, or expired: ccd95a16-5496-7001-f99b-1ab0e5f19e05'}"
   


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] F21 commented on pull request #57: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


F21 commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085400308


   You can squash the commits by following these steps: 
https://gitbetter.substack.com/p/how-to-squash-git-commits?s=r


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong removed a comment on pull request #57: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong removed a comment on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085397046


   > Looks good to me! Please squash your commits and make the commit message 
the same as the title of the PR.
   
   https://github.com/apache/calcite-avatica-go/pull/58
   @F21  Hi, I have some problems with rebase, I submitted a new 
PR(https://github.com/apache/calcite-avatica-go/pull/58), can I close this PR 
by merging another PR, I am not good at using rebase to merge multiple 
commits.thanks


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong closed pull request #58: [CALCITE-5077] ResetSession implements driver.SessionResetter. #58

2022-03-31 Thread GitBox


liguozhong closed pull request #58:
URL: https://github.com/apache/calcite-avatica-go/pull/58


   


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong edited a comment on pull request #57: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong edited a comment on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085397046


   > Looks good to me! Please squash your commits and make the commit message 
the same as the title of the PR.
   
   https://github.com/apache/calcite-avatica-go/pull/58
   @F21  Hi, I have some problems with rebase, I submitted a new 
PR(https://github.com/apache/calcite-avatica-go/pull/58), can I close this PR 
by merging another PR, I am not good at using rebase to merge multiple 
commits.thanks


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong commented on pull request #57: [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085397046


   > Looks good to me! Please squash your commits and make the commit message 
the same as the title of the PR.
   
   https://github.com/apache/calcite-avatica-go/pull/58
   Hi, I have some problems with rebase, I submitted a new 
PR(https://github.com/apache/calcite-avatica-go/pull/58), can I close this PR 
by merging another PR, I am not good at using rebase to merge multiple 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-unsubscr...@calcite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong opened a new pull request #58: [CALCITE-5077] ResetSession implements driver.SessionResetter. #58

2022-03-31 Thread GitBox


liguozhong opened a new pull request #58:
URL: https://github.com/apache/calcite-avatica-go/pull/58


   avatica sql query fails after running for a period of time on our online 
loki, this PR attempts to solve this "NoSuchConnectionException" problem.
   
   level=error ts=2022-03-31T12:51:09.356233496Z caller=table_manager.go:233 
msg="error syncing tables" err="An error was encountered while processing your 
request: NoSuchConnectionException{connectionId='Connection not found: invalid 
id, closed, or expired: ccd95a16-5496-7001-f99b-1ab0e5f19e05'}"


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] F21 commented on pull request #57: [WIP] [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


F21 commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085359819


   Looks good to me! Please squash your commits and make the commit message the 
same as the title of the PR.


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong edited a comment on pull request #57: [WIP] [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong edited a comment on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085352915


   @F21  done.
   
   this pr has fixed "NoSuchConnectionException" issue in my loki cluster.  and 
"NoSuchConnectionException" no longer appears in the log, but I'm not a 
database expert, @F21  I still need your help to review the code .
   
   
![image](https://user-images.githubusercontent.com/9583245/161183911-74f026f4-0508-4e41-9ba8-7aab2e538cb2.png)
   
   without this pr
   
![image](https://user-images.githubusercontent.com/9583245/161184397-cc3ba3f6-9f45-4d4e-a67f-7e7d7bd0275d.png)
   
   this this pr
   
![image](https://user-images.githubusercontent.com/9583245/161184449-acdb506e-ea27-4dfc-909f-fcaad2584a83.png)
   


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong commented on pull request #57: [WIP] [CALCITE-5077] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085352915


   @F21  done.
   
   this pr has fixed "NoSuchConnectionException" issue in my loki cluster.  and 
"NoSuchConnectionException" no longer appears in the log, but I'm not a 
database expert, @F21  I still need your help to review the code .
   
   
![image](https://user-images.githubusercontent.com/9583245/161183911-74f026f4-0508-4e41-9ba8-7aab2e538cb2.png)
   
   without this pr
   
![image](https://user-images.githubusercontent.com/9583245/161184397-cc3ba3f6-9f45-4d4e-a67f-7e7d7bd0275d.png)
   
   this this pr
   
![image](https://user-images.githubusercontent.com/9583245/161184449-acdb506e-ea27-4dfc-909f-fcaad2584a83.png)
   


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite] wojustme commented on pull request #2743: [CALCITE-5036] RelMdPredicates support to analyze constant key for the operator of IS_NOT_DISTINCT_FROM

2022-03-31 Thread GitBox


wojustme commented on pull request #2743:
URL: https://github.com/apache/calcite/pull/2743#issuecomment-1085336883


   Hi @hsyuan @NobiGo 
   I look back the discussion for this issue and pr, there are not any 
divergence.
   Could you help me merge it?
   Thanks a lot.


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica] F21 commented on pull request #173: CALCITE-5009 Transparent JDBC connection re-creation may lead to data…

2022-03-31 Thread GitBox


F21 commented on pull request #173:
URL: https://github.com/apache/calcite-avatica/pull/173#issuecomment-1085328760


   I've fixed the build, you should be able to rebase on top of master to get 
the tests passing.


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[calcite-avatica-go] branch master updated: Add Apache license header to website publication Github workflows

2022-03-31 Thread francischuang
This is an automated email from the ASF dual-hosted git repository.

francischuang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite-avatica-go.git


The following commit(s) were added to refs/heads/master by this push:
 new 962d7ee  Add Apache license header to website publication Github 
workflows
962d7ee is described below

commit 962d7ee14ba75633ee227a2ea612bf37ed67d05c
Author: Francis Chuang 
AuthorDate: Fri Apr 1 12:59:15 2022 +1100

Add Apache license header to website publication Github workflows
---
 .github/workflows/publish-website.yml | 16 
 1 file changed, 16 insertions(+)

diff --git a/.github/workflows/publish-website.yml 
b/.github/workflows/publish-website.yml
index 8950863..36add65 100644
--- a/.github/workflows/publish-website.yml
+++ b/.github/workflows/publish-website.yml
@@ -1,3 +1,19 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to you under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
 name: Publish website
 
 on:


[calcite] branch master updated: Add Apache license header to website publication Github workflows

2022-03-31 Thread francischuang
This is an automated email from the ASF dual-hosted git repository.

francischuang 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 850357d  Add Apache license header to website publication Github 
workflows
850357d is described below

commit 850357de1b27e0f5d5e07a47ae4044fa415ee67f
Author: Francis Chuang 
AuthorDate: Fri Apr 1 12:46:49 2022 +1100

Add Apache license header to website publication Github workflows
---
 .../workflows/publish-non-release-website-updates.yml| 16 
 .github/workflows/publish-website-on-release.yml | 16 
 2 files changed, 32 insertions(+)

diff --git a/.github/workflows/publish-non-release-website-updates.yml 
b/.github/workflows/publish-non-release-website-updates.yml
index b888b49..699700f 100644
--- a/.github/workflows/publish-non-release-website-updates.yml
+++ b/.github/workflows/publish-non-release-website-updates.yml
@@ -1,3 +1,19 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to you under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
 name: Publish non-release website updates
 
 on:
diff --git a/.github/workflows/publish-website-on-release.yml 
b/.github/workflows/publish-website-on-release.yml
index 16c821b..46f0957 100644
--- a/.github/workflows/publish-website-on-release.yml
+++ b/.github/workflows/publish-website-on-release.yml
@@ -1,3 +1,19 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to you under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
 name: Publish website on release
 
 on:


[calcite-avatica] branch master updated: Add Apache license header to website publication Github workflows

2022-03-31 Thread francischuang
This is an automated email from the ASF dual-hosted git repository.

francischuang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite-avatica.git


The following commit(s) were added to refs/heads/master by this push:
 new a7baa3b  Add Apache license header to website publication Github 
workflows
a7baa3b is described below

commit a7baa3b7a0d59973401392c0a1f22fb50469b5eb
Author: Francis Chuang 
AuthorDate: Fri Apr 1 12:45:36 2022 +1100

Add Apache license header to website publication Github workflows
---
 .../workflows/publish-non-release-website-updates.yml| 16 
 .../workflows/publish-site-and-javadocs-on-release.yml   | 16 
 2 files changed, 32 insertions(+)

diff --git a/.github/workflows/publish-non-release-website-updates.yml 
b/.github/workflows/publish-non-release-website-updates.yml
index e0ef396..c8da8a6 100644
--- a/.github/workflows/publish-non-release-website-updates.yml
+++ b/.github/workflows/publish-non-release-website-updates.yml
@@ -1,3 +1,19 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to you under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
 name: Publish non-release website updates
 
 on:
diff --git a/.github/workflows/publish-site-and-javadocs-on-release.yml 
b/.github/workflows/publish-site-and-javadocs-on-release.yml
index 3b95450..73a4453 100644
--- a/.github/workflows/publish-site-and-javadocs-on-release.yml
+++ b/.github/workflows/publish-site-and-javadocs-on-release.yml
@@ -1,3 +1,19 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to you under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
 name: Publish website on release
 
 on:


[calcite-avatica] branch master updated (b9aa1fb -> cb2cb81)

2022-03-31 Thread francischuang
This is an automated email from the ASF dual-hosted git repository.

francischuang pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/calcite-avatica.git.


from b9aa1fb  [CALCITE-3129] Automate website builds
 add cb2cb81  [CALCITE-5075] Build fails due to rat check on Gemfile.lock

No new revisions were added by this update.

Summary of changes:
 .ratignore | 1 +
 1 file changed, 1 insertion(+)


[GitHub] [calcite-avatica-go] liguozhong commented on pull request #57: [WIP] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085294462


   > Hey thanks for working on this. Can you open an issue at 
https://issues.apache.org/jira/secure/Dashboard.jspa and change the title of 
the PR to `[CALCITE-] Title of PR`? See the updated title in #56 as an 
example.
   
   Ok, I will go to jira to open an issue, I have no PR experience with apache 
foundation.
Thanks for your quick response .


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] F21 commented on pull request #57: [WIP] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


F21 commented on pull request #57:
URL: 
https://github.com/apache/calcite-avatica-go/pull/57#issuecomment-1085100317


   Hey thanks for working on this. Can you open an issue at 
https://issues.apache.org/jira/secure/Dashboard.jspa and change the title of 
the PR to `[CALCITE-] Title of PR`? See the updated title in 
https://github.com/apache/calcite-avatica-go/pull/56 as an example.


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[calcite] branch master updated: [CALCITE-5075] Build fails due to rat check on Gemfile.lock

2022-03-31 Thread rubenql
This is an automated email from the ASF dual-hosted git repository.

rubenql 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 f731ca6  [CALCITE-5075] Build fails due to rat check on Gemfile.lock
f731ca6 is described below

commit f731ca6d9fa8032997a33c4da4cc27da540db262
Author: rubenada 
AuthorDate: Thu Mar 31 14:36:28 2022 +0100

[CALCITE-5075] Build fails due to rat check on Gemfile.lock
---
 .ratignore | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.ratignore b/.ratignore
index b6af226..5b7442f 100644
--- a/.ratignore
+++ b/.ratignore
@@ -39,6 +39,7 @@ site/_layouts/news.html
 site/_layouts/news_item.html
 site/_layouts/page.html
 site/_sass/**
+site/Gemfile.lock
 site/css/screen.scss
 site/fonts/**
 site/js/**


[GitHub] [calcite] rubenada merged pull request #2758: [CALCITE-5075] Build fails due to rat check on Gemfile.lock

2022-03-31 Thread GitBox


rubenada merged pull request #2758:
URL: https://github.com/apache/calcite/pull/2758


   


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite-avatica-go] liguozhong opened a new pull request #57: [WIP] ResetSession implements driver.SessionResetter.

2022-03-31 Thread GitBox


liguozhong opened a new pull request #57:
URL: https://github.com/apache/calcite-avatica-go/pull/57


   avatica sql query fails after running for a period of time on our online 
loki, this PR attempts to solve this "NoSuchConnectionException" problem.
   
   ```shell
   level=error ts=2022-03-31T12:51:09.356233496Z caller=table_manager.go:233 
msg="error syncing tables" err="An error was encountered while processing your 
request: NoSuchConnectionException{connectionId='Connection not found: invalid 
id, closed, or expired: ccd95a16-5496-7001-f99b-1ab0e5f19e05'}"
   ```


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [calcite] rubenada opened a new pull request #2758: [CALCITE-5075] Build fails due to rat check on Gemfile.lock

2022-03-31 Thread GitBox


rubenada opened a new pull request #2758:
URL: https://github.com/apache/calcite/pull/2758


   Add `Gemfile.lock` to `.ratignore` to fix the issue


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[calcite] branch master updated: Fix typo documenting rules for triggering a non-release website update

2022-03-31 Thread francischuang
This is an automated email from the ASF dual-hosted git repository.

francischuang 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 88cc385  Fix typo documenting rules for triggering a non-release 
website update
88cc385 is described below

commit 88cc385f98c551c1aca7ffab101934f1c34fdffd
Author: Francis Chuang 
AuthorDate: Thu Mar 31 22:04:22 2022 +1100

Fix typo documenting rules for triggering a non-release website update
---
 .github/workflows/publish-non-release-website-updates.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/publish-non-release-website-updates.yml 
b/.github/workflows/publish-non-release-website-updates.yml
index 2026b41..b888b49 100644
--- a/.github/workflows/publish-non-release-website-updates.yml
+++ b/.github/workflows/publish-non-release-website-updates.yml
@@ -8,7 +8,7 @@ on:
   - 'site/**' # Trigger only for changes to files in 
the site/ folder
   - '!site/_docs/**'  # except for files in the site/_docs/ 
folder
   - 'site/_docs/history.md'   # unless the file is 
site/_docs/history.md
-  - 'site/_docs/howto.md' # or site/_docs/history.md
+  - 'site/_docs/howto.md' # or site/_docs/howto.md
   - 'site/_docs/powered_by.md'# or site/_docs/powered_by.md
 
 jobs:


[GitHub] [calcite-avatica] stoty commented on pull request #173: CALCITE-5009 Transparent JDBC connection re-creation may lead to data…

2022-03-31 Thread GitBox


stoty commented on pull request #173:
URL: https://github.com/apache/calcite-avatica/pull/173#issuecomment-1084290069


   I don't think that the checkstyle/rat errors are related to this patch.


-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[calcite-site] branch master updated (923022a -> 401cc9b)

2022-03-31 Thread francischuang
This is an automated email from the ASF dual-hosted git repository.

francischuang pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/calcite-site.git.


 discard 923022a  Test commit

This update removed existing revisions from the reference, leaving the
reference pointing at a previous point in the repository history.

 * -- * -- N   refs/heads/master (401cc9b)
\
 O -- O -- O   (923022a)

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

No new revisions were added by this update.

Summary of changes:
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


[calcite-site] branch master updated: Test commit

2022-03-31 Thread francischuang
This is an automated email from the ASF dual-hosted git repository.

francischuang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite-site.git


The following commit(s) were added to refs/heads/master by this push:
 new 923022a  Test commit
923022a is described below

commit 923022a0fa0efaa76ed70543ba3e94d7fd0d8e8c
Author: Francis Chuang 
AuthorDate: Thu Mar 31 19:24:39 2022 +1100

Test commit
---
 index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/index.html b/index.html
index 46de778..c273c30 100644
--- a/index.html
+++ b/index.html
@@ -9,7 +9,7 @@
   
   
 
-
+a