srh commented on PR #17185:
URL: https://github.com/apache/datafusion/pull/17185#issuecomment-3187389901

   I guess this got fixed differently, a couple days ago, by #17129.  That 
change does leave MemorySourceConfig stripped of its sort information, while 
the DataSourceExec above it remembers it from the cache.
   
   I am not familiar with the whole story of equivalence properties maintenance 
there (and I guess the DataSourceExec change works with every input 
DataSource).  So, I will leave this PR open so that you see it, and you can 
decide what you want to do, but as far as I know, you *could* just close 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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to