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

   ## Which issue does this PR close?
   
   - part of #20529 
   
   ## Rationale for this change
   
   
   While working on morselization, I found that the `PreparedAccessPlan` was in 
a different module which made it harder to understand the relationship between 
the two. Thus, I think it would be better to move PreparedAccessPlan to the 
same module as ParquetAccessPlan.
   
   
   
   ## What changes are included in this PR?
   
   1. Move PreparedAccessPlan: Relocated the PreparedAccessPlan struct and its 
implementation from datafusion/datasource-parquet/src/opener.rs to 
datafusion/datasource-parquet/src/access_plan.rs.
   2. Unify Logic: Added a prepare method to ParquetAccessPlan to create a 
PreparedAccessPlan, replacing the PreparedAccessPlan::from_access_plan static 
method.
   
   
   ## Are these changes tested?
   
   Yes by CI
   
   ## 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