[GitHub] [kyuubi] zwangsheng commented on issue #4738: [Bug] Cannot get the `GROUP` shared level Engine info using admin RESTful API.

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

zwangsheng commented on issue #4738:
URL: https://github.com/apache/kyuubi/issues/4738#issuecomment-1515905013

   As https://github.com/apache/kyuubi/discussions/4684 tells:
   
   1. Engine register self with `/serverSpace_version_GROUP_engineType`/`primary group name`[/`subdomain`]
   
   2. RestApi find engine with `/serverSpace_version_CONNECTION_engineType`/`user`[/`subdomain`]
   
   3. We should respect design with `/serverSpace_version_GROUP_engineType`/`primary group name`[/`subdomain`]
   
   So, let's fix `AdminResource#getEngineSpace()`'s logic, keep same with `EngineRef#engineSpace`
   
   


-- 
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