hequn8128 commented on a change in pull request #11142:
[FLINK-16167][python][doc] update python_shell document execution
URL: https://github.com/apache/flink/pull/11142#discussion_r381312279
##########
File path: docs/ops/python_shell.md
##########
@@ -28,7 +28,7 @@ It can be used in a local setup as well as in a cluster
setup.
To use the shell with an integrated Flink cluster just execute:
{% highlight bash %}
-bin/pyflink-shell.sh local
+pyflink-shell.sh local
Review comment:
@jingwen-ywb Thanks a lot for the PR. Good job!
I think the reason that to remove the `bin` is we can install pyflink with
Pypi so that the pyflink-shell.sh is no the system PATH automatically. In this
case, it's better to also add the document about how to install pyflink with
Pypi.. Meanwhile, we can remove the document about installing apache-beam since
it will be installed during install pyflink.
What do you think?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
