peter-toth commented on PR #53658:
URL: https://github.com/apache/spark/pull/53658#issuecomment-3737912700

   > @peter-toth I see that we are in agreement with transformDown.. Thank you 
for your understanding. If you still think that separating the cases for Not 
from others would not miss any un-anticipated situations , then I will not 
block .. Though I urge you to reconsider....
   
   All right, I don't see any case that would be problematic with transform 
down so let's keep it this way.
   Please fix https://github.com/apache/spark/pull/53658#discussion_r2681701087 
and leave a comment on the 2 affected cases why the extra transform down is 
beneficial there and then I'm ok to merge this PR.


-- 
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]

Reply via email to