shrirangmhalgi commented on PR #55892:
URL: https://github.com/apache/spark/pull/55892#issuecomment-4513055622
Thanks for the suggestion! I tried `case e if !e.resolved => e` first - it
broke `5 posexplode alias-chaining tests + 1 AnalysisSuite test`. The issue is
that `posexplode(col).as("a").as("b")` creates
`Alias(Alias(unresolved_generator, "a"), "b")` where the inner alias is
unresolved but still needs trimming for chained-alias handling to work
correctly.
Narrowed it to `case u: UnresolvedFunction => u` - this targets the actual
problem class (unresolved function calls like `struct(...)` whose alias
children carry field names for `CreateStruct.apply)` without affecting alias
trimming on other unresolved expressions. Reverted the local `ExtractGenerator`
workaround - the fix now lives entirely in `AliasHelper.trimAliases`. Also
addressed the ArrayType import nit. Pushed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
