Skip to content

Conversation

@siying
Copy link
Contributor

@siying siying commented Jul 25, 2024

What changes were proposed in this pull request?

Divide PythonStreamingDataSourceSuite into PythonStreamingDataSourceSuite, PythonStreamingDataSourceSimpleSuite and PythonStreamingDataSourceWriteSuite

Why are the changes needed?

It is reported that PythonStreamingDataSourceSuite runs too long. We need to divide it to allow better parallelism.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Watch CI to cover the three new tests and pass.

Was this patch authored or co-authored using generative AI tooling?

No.

Copy link
Contributor

@chaoqin-li1123 chaoqin-li1123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@allisonwang-db
Copy link
Contributor

Thanks, merging to master!

ilicmarkodb pushed a commit to ilicmarkodb/spark that referenced this pull request Jul 29, 2024
### What changes were proposed in this pull request?
Divide PythonStreamingDataSourceSuite into PythonStreamingDataSourceSuite, PythonStreamingDataSourceSimpleSuite and PythonStreamingDataSourceWriteSuite

### Why are the changes needed?
It is reported that PythonStreamingDataSourceSuite runs too long. We need to divide it to allow better parallelism.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Watch CI to cover the three new tests and pass.

### Was this patch authored or co-authored using generative AI tooling?
No.

Closes apache#47479 from siying/divide_python_stream_suite.

Authored-by: Siying Dong <siying.dong@databricks.com>
Signed-off-by: allisonwang-db <allison.wang@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants