BewareMyPower commented on PR #236:
URL: 
https://github.com/apache/pulsar-client-cpp/pull/236#issuecomment-1499163053

   When I tried to write shell scripts for `pulsar-shell`, I found it's hard to 
split it into two lines. For example, I met the following error:
   
   ```
   [5/29] ? admin namespaces create public/default -c standalone
   [6/29] Executing admin namespaces grant-permission
   The following options are required: [--actions], [--role]
   ```
   
   while my script is:
   
   ```bash
   admin namespaces create public/default -c standalone
   admin namespaces grant-permission \
       --actions produce,consume --role anonymous \
       public/default
   ```
   
   The backslash seems not recognized, which is bad from my view.
   
   BTW, I think the readability is not bad even with the raw REST APIs.
   1. I wrote the comments for each REST call.
   2. The `curl` command is encapsulated into a shell function `put <path> 
<body>`.
   
   In addition, it can also be easily used to verify an older Pulsar image (< 
2.11.0) to test the backward compatibility.
   
   @RobertIndie 


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