arvindKandpal-ksolves opened a new pull request, #8853:
URL: https://github.com/apache/hbase/pull/8853

   Backport of #8852 to branch-3.
   
   ### Problem
   The hbtop usage showed -D options after the other options:
   
         usage: hbase hbtop [opts] [-D<property=value>]*
   
   But -D only works before the other options, so following the usage fails 
with UnrecognizedOptionException. The help also only showed a ZooKeeper 
example, which the default RpcConnectionRegistry does not read.
   
   ### Changes
   - `HBTop.java`: usage is now `hbase hbtop [-D<property=value>]* [opts]`, 
with a note that -D must come first. The help shows 
`hbase.client.bootstrap.servers` for the default registry and keeps the 
ZooKeeper example for `ZKConnectionRegistry`.
   - The doc change from #8852 is not included, since the website is built from 
master only.
   
   ### Testing
   - hbase-hbtop unit tests pass on branch-3. No new test, since only help text 
changed.
   


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

  • [PR] HBASE-30518 [hbtop] Usa... arvindKandpal-ksolves (via GitHub) via issues

Reply via email to