Replies: 4 comments
0 replies
|
@carloea2 @Yicong-Huang : any input? @kz930 Please lead the discussion and close it if when it's done. |
0 replies
|
It looks good to me |
0 replies
|
If it looks good, then I would close this discussion. This is the current implementation so far. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Design of the verification itself is in #6072 — this is only about why it was slow and what fixed it.
How we check one operator
Three steps, and each one that needs Python used to start a brand-new Python process:
So one check = 2 Python starts for a Scala operator, 3 for a Python-based one. And we do one check per enum option (line chart mode = line / dots / line+dots), so a single operator often means several checks.
The problem
The fix
PYTHONPATH), so workers are never mixed between them.TEXERA_TEST_PYTHON_WORKER=0turns pooling off entirely.The result
Same suite, same 53 operators, same machine:
How to test it
Point
UDF_PYTHON_PATHat a Python 3.12 that has pandas, plotly, pyarrow and betterproto, then:UDF_PYTHON_PATH="$PWD/../venv312/bin/python" sbt "WorkflowCompilingService/testOnly *OperatorBehaviorSpec"All reactions