> - For portable running tests: by looking at the
> portableValidatesRunnerTask in flink_job_server.gradle, it seems it's
> the same set of Java tests but using portability framework to validate
> (submit to job server and run the protable pipeline in a specific
> runner). Is my understanding correct?

That's correct.

> - For python tests: Looks like flinkValidatesRunner is using LOOPBACK
> SDK worker type in the tests. Not sure what LOOPBACK does. Is it used
> for testing? Currently Samza portable runner supports PROCESS worker.

This avoids building and using containers for this test. We had a number of issues with Docker on Jenkins and wanted to lower build time for PreCommit tests. Loopback means that an embedded Python environment will be started which listens on localhost. It's comparable to Java's EmbeddedSdkHarness.

-Max

On 24.04.19 20:10, Xinyu Liu wrote:
Thanks for the useful pointers! We are looking forward to integrating both Portable and Python-specific tests for Samza runner. A few questions:

- For portable running tests: by looking at the portableValidatesRunnerTask in flink_job_server.gradle, it seems it's the same set of Java tests but using portability framework to validate (submit to job server and run the protable pipeline in a specific runner). Is my understanding correct?

- For python tests: Looks like flinkValidatesRunner is using LOOPBACK SDK worker type in the tests. Not sure what LOOPBACK does. Is it used for testing? Currently Samza portable runner supports PROCESS worker.

Thanks,
Xinyu



On Wed, Apr 24, 2019 at 2:45 AM Maximilian Michels <m...@apache.org <mailto:m...@apache.org>> wrote:

     > If you are interested in portable python pipeline validation, I
    think
     > fn_api_runner_test would also help.

    Just to note, Ankur mentioned flinkCompatibilityMatrix, that one uses
    fn_api_runner_test with some tooling on top to bring up the test
    cluster.

    On 23.04.19 19:23, Boyuan Zhang wrote:
     > Hi Daniel,
     > If you are interested in portable python pipeline validation, I
    think
     > fn_api_runner_test
     >
    
<https://github.com/apache/beam/blob/master/sdks/python/apache_beam/runners/portability/fn_api_runner_test.py>

     > would also help.
     >
     > On Tue, Apr 23, 2019 at 10:19 AM Pablo Estrada
    <pabl...@google.com <mailto:pabl...@google.com>
     > <mailto:pabl...@google.com <mailto:pabl...@google.com>>> wrote:
     >
     >     This is cool, Daniel : ) Glad to see the Samza runner moving
    forward.
     >     Best
     >     -P.
     >
     >     On Tue, Apr 23, 2019 at 2:52 AM Maximilian Michels
    <m...@apache.org <mailto:m...@apache.org>
     >     <mailto:m...@apache.org <mailto:m...@apache.org>>> wrote:
     >
     >         Hi Daniel,
     >
     >         Note that there is also Portable Validates Runner which
    runs Java
     >         portability tests. I don't know if you have integrated
    with that
     >         one
     >         already.
     >
     >         Thanks,
     >         Max
     >
     >         On 23.04.19 02:28, Ankur Goenka wrote:
     >          > Hi Daniel,
     >          >
     >          > We use flinkCompatibilityMatrix [1] to check the Flink
     >         compatibility
     >          > with python. This is python equivalent to validatesRunner
     >         tests in java
     >          > for portable runners.
     >          > I think we can reuse it for Samza Portable runner with
    minor
     >         refactoring.
     >          >
     >          > [1]
     >          >
     >
    
https://github.com/apache/beam/blob/bdb1a713a120a887e71e85c77879dc4446a58541/sdks/python/build.gradle#L305
     >          >
     >          > On Mon, Apr 22, 2019 at 3:21 PM Daniel Chen
     >         <danx...@gmail.com <mailto:danx...@gmail.com>
    <mailto:danx...@gmail.com <mailto:danx...@gmail.com>>
     >          > <mailto:danx...@gmail.com <mailto:danx...@gmail.com>
    <mailto:danx...@gmail.com <mailto:danx...@gmail.com>>>> wrote:
     >          >
     >          >     Hi everyone,
     >          >
     >          >     I'm working on improving the validation of the Python
     >         portable Samza
     >          >     runner. For java, we have the gradle task (
     >         :validatesRunner) that
     >          >     runs the runner validation tests.
     >          >     I am looking for pointers on how to similarly
     >         integrate/enable the
     >          >     portability and Python tests for the Samza runner.
     >          >
     >          >     Any help will be greatly appreciated.
     >          >
     >          >     Thanks,
     >          >     Daniel
     >          >
     >

Reply via email to