You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@kyuubi.apache.org by GitBox <gi...@apache.org> on 2021/10/20 01:27:17 UTC

[GitHub] [incubator-kyuubi] simon824 commented on pull request #1259: [KYUUBI #1258] Implement /sessions/execpoolsize and /sessions/execpoolactivecount

simon824 commented on pull request #1259:
URL: https://github.com/apache/incubator-kyuubi/pull/1259#issuecomment-947230516


   > @simon824 thanks for your contribution.
   > 
   > After serious thinking, I think these two APIs are a bit ugly. It's my problem, No intention to offend you.
   > 
   > I am considering two candidates:
   > 
   > * `/sessions/execpool/size` & `/sessions/execpool/activecount`
   > * `/sessions/execpool/statistic` which returns a packaged object contains size & activecount and so on
   > 
   > Which one do you prefer?
   
   Thanks a lot for your review , I think option 2 is better.


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

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