[GitHub] reddycharan commented on issue #1798: [tools] package bookkeeper tools into a separated distribution

2018-11-09 Thread GitBox
reddycharan commented on issue #1798: [tools] package bookkeeper tools into a 
separated distribution
URL: https://github.com/apache/bookkeeper/pull/1798#issuecomment-437536329
 
 
   ok. lgtm.


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] reddycharan commented on issue #1798: [tools] package bookkeeper tools into a separated distribution

2018-11-09 Thread GitBox
reddycharan commented on issue #1798: [tools] package bookkeeper tools into a 
separated distribution
URL: https://github.com/apache/bookkeeper/pull/1798#issuecomment-437532130
 
 
   @sijie thanks for explanation. Yes, I was little confused about purpose of 
this package. 
   
   When you say "At many environments, you want to deliver a set of tools to 
your customers to interact with or operate bookkeeper clusters."
   
   do you mean to say this would be for customers who wants to download this 
package to their control nodes (not bookie nodes) and run bookkeeper shell 
script? so they would be able to run only commands in 'cluster'/'ledger' group 
but not in 'bookie' (commands which access local filesystem of bookie) group?


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] reddycharan commented on issue #1798: [tools] package bookkeeper tools into a separated distribution

2018-11-09 Thread GitBox
reddycharan commented on issue #1798: [tools] package bookkeeper tools into a 
separated distribution
URL: https://github.com/apache/bookkeeper/pull/1798#issuecomment-437515038
 
 
   @sijie i feel word 'tools' is kind of misleading. It kind of gives 
impression that it just contains tools to operate / administer bookkeeper like 
bookkeeper, bookkeeper-cli, bookkeeper http server stuff..reconsider the name 
for this package. And add more info. to this commit for what is the purpose for 
this package and how it differs from all/server package .
   
   when you say "People can just download the tools package and use it to 
operate bookkeeper clusters without downloading the server package.", one would 
ask what is wrong in using server package?


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] reddycharan commented on issue #1798: [tools] package bookkeeper tools into a separated distribution

2018-11-09 Thread GitBox
reddycharan commented on issue #1798: [tools] package bookkeeper tools into a 
separated distribution
URL: https://github.com/apache/bookkeeper/pull/1798#issuecomment-437497286
 
 
   Hey @sijie whats the purpose of this change? what are you moving and how 
would it help?


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