LukaZdravic commented on PR #59134: URL: https://github.com/apache/spark/pull/59134#issuecomment-5934904120
> This rebuild here is not needed, we can compare the original operands directly and we would avoid making this fix, because Spark already compares struct by positions and handles whole and nested NULLs. > > We still need to keep the fix for sorting, because NULL starts needs to come before a non-NULL structs whose fields are all NULL. Great catch! Will work on removing rebuild all together -- 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]
