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
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,
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
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
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:
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
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
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.
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
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
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.
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:
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.
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.
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
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
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
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
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
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
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
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
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
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
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-]
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
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
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:
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
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
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
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
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
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
34 matches
Mail list logo