alamb opened a new pull request, #5814: URL: https://github.com/apache/arrow-datafusion/pull/5814
# Which issue does this PR close? related to #5637 , but really a Friday afternoon project that has bothered me for a whle # Rationale for this change The PlannerContext in the SQL planner has a map of `LogicalPlans` and it is cloned during planning (which deep clones the values 😱 ). Also, the LogicalPlan is cloned on use anyways. # What changes are included in this PR? 1. Make the fields of `PlannerContext` non pub 2. # Are these changes tested? Covered by existing tests # 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
