Skip to content

Conversation

@WweiL
Copy link
Contributor

@WweiL WweiL commented Apr 6, 2023

What changes were proposed in this pull request?

This PR adds the orc, parquet, and text APIs in connect's DataStreamReader

Why are the changes needed?

Part of Streaming Connect project.

Does this PR introduce any user-facing change?

Yes, now the three APIs are enabled. But everything is pretty much still under developed so far.

How was this patch tested?

Manually tested, unit tests will be added in SPARK-43031 as a follow-up PR #40691.

Copy link

@rangadi rangadi left a comment

Choose a reason for hiding this comment

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

LGTM.

@rangadi
Copy link

rangadi commented Apr 10, 2023

cc: @HyukjinKwon please merge.

@HyukjinKwon
Copy link
Member

Merged to master.

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