matthewgapp commented on issue #462:
URL: 
https://github.com/apache/arrow-datafusion/issues/462#issuecomment-1724805767

   I've continued to clean up the rough edges and test more use cases. I have a 
couple priorities before requesting additional feedback 
   - [] stop optimizer from inserting `RepartitionExec` as a decedent of the 
`RecursiveQueryExec`
   - [] implement more robust validation in the `Query` to `LogicalPlan` 
transform step. We should try to catch the obvious cases of infinite recursion, 
etc. 
   - [] perform more testing and perhaps write some tests that capture this 
behavior. So far I've manually tested the trivial case, selects from another 
table in the static term, joins in the static term and joins in the recursive 
term. 
   - [] outline obvious performance improvements (like [optionally] eliminating 
metric in the recursive tree since these need to be rebuilt every iteration 
because recursive queries behave very much unlike regular queries). 


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

Reply via email to