[GitHub] [kyuubi] touchida commented on pull request #5015: [KYUUBI #5010] Make Kyuubi server's thrift frontend connection URL configurable

["touchida (via GitHub)" <gi...@apache.org>]

touchida commented on PR #5015:
URL: https://github.com/apache/kyuubi/pull/5015#issuecomment-1635339808

   The following failed test will not be related to my PR, and it succeeds in my local environment.
   ```
   DockerizedZkServiceDiscoverySuite:
   - publish instance to discovery service
   - KYUUBI #304: Stop engine service gracefully when related node is deleted
   - distribute lock *** FAILED ***
     lockLatch.await(5000L, MILLISECONDS) was false (DiscoveryClientTests.scala:146)
   - getAndIncrement method test
   - setData method test
   - acl for zookeeper
   - set up zookeeper auth
   - stop engine in time while zk ensemble terminates
   - watcher for zookeeper
   Run completed in 47 seconds, 328 milliseconds.
   Total number of tests run: 9
   Suites: completed 2, aborted 0
   Tests: succeeded 8, failed 1, canceled 0, ignored 0, pending 0
   *** 1 TEST FAILED ***
   ```
   
   @pan3793
   Could you kindly review my PR again when you have time?


-- 
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-unsubscribe@kyuubi.apache.org

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


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