HeartSaVioR edited a comment on issue #23609: [SPARK-26379][SS] Fix issue on adding current_timestamp/current_date to streaming query URL: https://github.com/apache/spark/pull/23609#issuecomment-456998940 > If the fix is to resolve the nodes, then this should be fine. If the fix were to change how v2 works and avoid some call to get the schema of a plan, then I would be concerned. This patch is to resolve the nodes prior to get a schema (via having temporary plan), while it doesn't touch the actual plan so that it would work as before. I agree DSv2 is a moving target, but the bug is affecting not only for master branch but also for 2.4/2.3 as well. So I hope we get it resolved first, and fix it (again) when new DSv2 is concerning.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
