blaginin opened a new pull request, #14636:
URL: https://github.com/apache/datafusion/pull/14636

   ## Which issue does this PR close?
   
   Related to https://github.com/apache/datafusion/issues/14563 (probably more 
prs to come)
   
   
   ## Rationale for this change
   
   Now, when normalizing the column, we always generate `plan.using_columns()` 
which is recursive and very expensive - and [may not be 
needed](https://github.com/apache/datafusion/blob/980931c5a70b8b08c918195803145f64f1ec901f/datafusion/common/src/column.rs#L224)
 if column is already normalized 
   
   ## What changes are included in this PR?
   
   Exit early if column already has a relation set. Also, set the relation when 
`with_column_renamed` is called
   
   ## Are these changes tested?
   
   Extended a test to assert references
   
   ## Are there any user-facing changes?
   
   No


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