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

[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