lhotari commented on code in PR #877:
URL: https://github.com/apache/pulsar-site/pull/877#discussion_r1548676145


##########
docs/admin-api-functions.md:
##########
@@ -46,7 +46,7 @@ pulsar-admin functions create \
     --inputs test-input-topic \
     --output persistent://public/default/test-output-topic \
     --classname org.apache.pulsar.functions.api.examples.ExclamationFunction \
-    --jar /examples/api-examples.jar
+    --jar $PWD/examples/api-examples.jar

Review Comment:
   I think `$PWD` gives a good hint that it's an absolute path. At least I have 
found it useful with `pulsar-admin` when I have the files in the current 
directory and I want to reference the files with an absolute path. I guess 
there are also other ways.  Perhaps it could be explained to the user that 
`$PWD` is the current working directory. They will also find out if they try it 
or google it. :) 
   In this particular case, the original `/examples/api-examples.jar` doesn't 
seem to make much sense. That's why I added the `$PWD` prefix.



-- 
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: commits-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to