Anonymitaet commented on issue #19492:
URL: https://github.com/apache/pulsar/issues/19492#issuecomment-1430735017

   @visortelle Thanks! This issue contains two types of improvements:
   
   ### Doc improvements
   
   ☘️ **TO-DOs**
   Add the `environment variables` section to the following pages.
   - Reference site - [`pulsar-perf` home 
page](https://pulsar.apache.org/reference/#/next/pulsar-perf/)
   - Reference site - [`pulsar-admin` home 
page](https://pulsar.apache.org/reference/#/next/pulsar-admin/)
   - Reference site - [`pulsar-client` home 
page](https://pulsar.apache.org/reference/#/next/pulsar-client/)
   
   ☘️ **Reasons**
   This section is shown on 2.10.x and earlier markdown docs (e.g., 
https://pulsar.apache.org/docs/2.10.x/reference-cli-tools/#pulsar-perf) but not 
available on 2.11.x and later Reference site. 
   
   Since the reference pages of `pulsar-perf`, `pulsar-admin`, and 
`pulsar-client` are [generated 
automatically](https://pulsar.apache.org/contribute/document-contribution/#update-command-line-tool-docs),
 we can add the `environment variables` section to the their homepage as a 
temporary workaround. We can optimize it once we find better solutions.
   
   @momo-jun @DaveDuggins @D-2-Ed WDYT?
   
   ### Code improvements
   
   I've double-checked and tested the current code behavior and then recorded 
it in https://github.com/apache/pulsar/issues/19526. Feel free to correct me if 
I'm inaccurate, thank you!
   


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