[GitHub] [arrow-datafusion] alamb commented on issue #4943: `EnforceSorting` resorts the inout of UnionExec unnecessarily

2023-01-18 Thread GitBox


alamb commented on issue #4943:
URL: 
https://github.com/apache/arrow-datafusion/issues/4943#issuecomment-1386852191

   Yes I am quite pleased with how sophisticated the sorting based 
optimizations are becoming 


-- 
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...@arrow.apache.org

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



[GitHub] [arrow-datafusion] alamb commented on issue #4943: `EnforceSorting` resorts the inout of UnionExec unnecessarily

2023-01-17 Thread GitBox


alamb commented on issue #4943:
URL: 
https://github.com/apache/arrow-datafusion/issues/4943#issuecomment-1385622282

   proposed PR: https://github.com/apache/arrow-datafusion/pull/4946


-- 
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...@arrow.apache.org

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



[GitHub] [arrow-datafusion] alamb commented on issue #4943: `EnforceSorting` resorts the inout of UnionExec unnecessarily

2023-01-17 Thread GitBox


alamb commented on issue #4943:
URL: 
https://github.com/apache/arrow-datafusion/issues/4943#issuecomment-1385388526

   Thank you @mustafasrepo  -- I will attempt to write a reproducer for this 
and try our your proposed fix


-- 
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...@arrow.apache.org

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



[GitHub] [arrow-datafusion] alamb commented on issue #4943: `EnforceSorting` resorts the inout of UnionExec unnecessarily

2023-01-17 Thread GitBox


alamb commented on issue #4943:
URL: 
https://github.com/apache/arrow-datafusion/issues/4943#issuecomment-1385291332

   cc @mustafasrepo and @mingmwang  who I think have expertise in this area


-- 
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...@arrow.apache.org

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