ibzib commented on a change in pull request #12385:
URL: https://github.com/apache/beam/pull/12385#discussion_r499941214
##########
File path: sdks/python/apache_beam/runners/portability/flink_runner_test.py
##########
@@ -53,361 +54,386 @@
from apache_beam.transforms import userstate
from apache_beam.transforms.sql import SqlTransform
+# Run as
+#
+# pytest flink_runner_test.py \
+# [--test_pipeline_options "--flink_job_server_jar=/path/to/job_server.jar
\
Review comment:
Oh yeah, `--test_pipeline_options` is now required (even though it
should be possible to leave it empty). Boyuan, would you mind filing a PR to
fix this?
----------------------------------------------------------------
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]
