-
Notifications
You must be signed in to change notification settings - Fork 28k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SPARK-48300][SQL] Codegen Support for from_xml
#46609
Conversation
@@ -104,7 +103,7 @@ case class XmlToStructs( | |||
if (mode != PermissiveMode && mode != FailFastMode) { | |||
throw QueryCompilationErrors.parseModeUnsupportedError("from_xml", mode) | |||
} | |||
val (parserSchema, actualSchema) = nullableSchema match { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The variable actualSchema
is not used anywhere
from_xml
from_xml
& remove some redundant codes
@panbingkun Thanks for submitting the PR. Can you please separate the |
|
After the PR above is merged, I will rebase the PR again just for |
from_xml
& remove some redundant codesfrom_xml
It has rebase the master. At present, this PR is only for |
sql/core/src/test/scala/org/apache/spark/sql/XmlFunctionsSuite.scala
Outdated
Show resolved
Hide resolved
Merged to master. Thank you @panbingkun |
What changes were proposed in this pull request?
The PR aims to add
Codegen Support
forfrom_xml
Why are the changes needed?
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Was this patch authored or co-authored using generative AI tooling?
No.