[GitHub] rdhabalia commented on issue #1223: Add a `backend` admin restful endpoint for query backend information

2018-02-13 Thread GitBox 
rdhabalia commented on issue #1223: Add a `backend` admin restful endpoint for 
query backend information
URL: https://github.com/apache/incubator-pulsar/pull/1223#issuecomment-365474351
 
 
   > the purpose of this PR is to provide a method for application to know 
which zk, bk that a pulsar cluster is using.
   InternalConfiguration/InternalData ?
   
   I think we can add more configurations other than zk in future as it can 
surface internal configuration. So, should can keep it a generic  name eg: 
`InternalConfiguration`? 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] rdhabalia commented on issue #1223: Add a `backend` admin restful endpoint for query backend information

2018-02-12 Thread GitBox 
rdhabalia commented on issue #1223: Add a `backend` admin restful endpoint for 
query backend information
URL: https://github.com/apache/incubator-pulsar/pull/1223#issuecomment-365130333
 
 
   > I think the class name Backend and BackendData are ambiguous.
   
   or, we can use `InternalConfiguration/InternalData` instead of Backend 
because for topic's internal stats we use `internal-stats` term so, internal 
may give more meaning?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services