spark icon indicating copy to clipboard operation
spark copied to clipboard
verified publisher icon apache

[SPARK-49062][SQL] Migrate XML to File Data Source V2

Open wayneguow opened this issue 1 year ago • 4 comments

What changes were proposed in this pull request?

This PR aims to Migrate XML to File Data Source V2.

Why are the changes needed?

Add v2 support for XML.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA and transform XmlSuite.

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

No.

wayneguow avatar Jul 30 '24 18:07 wayneguow

cc @HyukjinKwon @cloud-fan thanks.

wayneguow avatar Jul 31 '24 06:07 wayneguow

cc @sandip-db WDYT?

HyukjinKwon avatar Aug 02 '24 00:08 HyukjinKwon

cc @sandip-db WDYT?

It looks ok to me. Although, I have not come across anyone asking for it.

sandip-db avatar Aug 02 '24 19:08 sandip-db

cc @sandip-db WDYT?

It looks ok to me. Although, I have not come across anyone asking for it.

Well, the reason why I proposed this PR is that other common data sources have v1 and v2 implementations, so I want to add v2 for xml. WDYT @cloud-fan , we need your help when you have time.

wayneguow avatar Aug 03 '24 06:08 wayneguow

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

github-actions[bot] avatar Nov 23 '24 00:11 github-actions[bot]