You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "mingmwang (via GitHub)" <gi...@apache.org> on 2023/03/14 01:52:54 UTC

[GitHub] [arrow-datafusion] mingmwang commented on pull request #5570: Add Analyzer phase to DataFusion , add basic validation logic to Subquery Plans and Expressions

mingmwang commented on PR #5570:
URL: https://github.com/apache/arrow-datafusion/pull/5570#issuecomment-1467223087

   > Thanks @mingmwang -- I think this PR's structure looks great.
   > 
   > Do you plan to move the https://github.com/apache/arrow-datafusion/blob/main/datafusion/optimizer/src/type_coercion.rs logic into the analyzer as well?
   > 
   > I had some suggestions on how to reduce replication which I think would be nice but not strictly required.
   
   Sure, I can work on this in future. But recently I plan to address the subquery related gaps first.
   
   https://github.com/apache/arrow-datafusion/issues/5483
   


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

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