mattisonchao commented on pull request #13611:
URL: https://github.com/apache/pulsar/pull/13611#issuecomment-1005352599


   @codelipenghui  @ericsyh @eolivelli @BewareMyPower 
   
   PTAL :) 
   
   **Another things:**
   
   Although this PR can fix the issue #13479,  But I have some advice as below:
   
   1. ``healthcheck`` keyword name is common , maybe we can change this to some 
complex name.
   2. I think we can collect all internal or system theme names into one 
container, because they are currently everywhere. Unclear and difficult to 
maintain.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to