pan3793 commented on code in PR #4147:
URL: https://github.com/apache/kyuubi/pull/4147#discussion_r1082908356


##########
charts/kyuubi/templates/_helpers.tpl:
##########
@@ -16,33 +16,18 @@
 */}}
 
 {{/*
-Expand the name of the chart.
+A comma separated string of enabled frontend protocols, e.g. 
"MYSQL,REST,THRIFT_BINARY".

Review Comment:
   Sorry for the late reply, it's during the Chinese Spring Festival holiday.
   
   - THRIFT_BINARY is much much stable, w/ lots of production use cases.
   - THRIFT_HTTP should work well w/ a single node, but may not be in the HA 
mode, since the HTTP is the short connection, and Kyuubi Server does not 
implement the shared session state across multi-instances. A workaround is to 
use client-IP hash strategy in the load balancer so that the same client always 
accesses the same server.
   - REST the batch API(the Livy-like API) should be considered as GA since 
1.7, which fully supports HA mode and has production use cases in different 
companies. The rest of the API has a similar issue w/ THRIFT_HTTP in HA
   - MYSQL 1)network protocol is implemented, 2)authentication does not work, 
3)it does not support MySQL metadata queries which access the 
`information_schemas`. Basically, you can access Kyuubi using `mysql` cli or 
`mysql-connector-j` w/o authentication.
   - TRINO it's under development, I guess it can work limited in 1.7



-- 
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: notifications-unsubscr...@kyuubi.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org
For additional commands, e-mail: notifications-h...@kyuubi.apache.org

Reply via email to