maropu commented on pull request #32559: URL: https://github.com/apache/spark/pull/32559#issuecomment-842720103
> Is this a bug? It looks a kind of improvements to me. >> You can consider it as an improvement, yea. Although from user perspective, it is somehow hard to distinguish them clearly. I was just wondering if we need to backport this fix or not. I think the update of CSE-related code can affect the performance of user's queries easily (e.g., , performance penalties caused by the expensive loop), so IMO it's safe to merge it into master only. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
