timsaucer commented on code in PR #1720:
URL:
https://github.com/apache/datafusion-python/pull/1720#discussion_r3979658843
##########
docs/source/user-guide/upgrade-guides.md:
##########
@@ -96,16 +96,6 @@ way `add_physical_optimizer_rule` does and returns nothing —
the query planner
lives in `SessionState`, so it belongs to the session rather than to a
particular handle on it. See {ref}`extension_planners` for the full protocol.
-If a library ships codecs *and* a planner, prefer
-`SessionContext.with_extensions(bundle)` over installing each piece by hand. It
-installs every codec before it binds any planner, so a planner cannot end up
-carrying a chain that a later `with_logical_extension_codec` call has grown.
-The library exposes a bundle object implementing
-`__datafusion_session_components__` for its codecs and
-`__datafusion_session_planner__` for its planner — the latter is handed the
-planner installed so far, so several libraries that each ship one nest instead
-of displacing each other. See {ref}`extension_bundles`.
-
Review Comment:
This is a drive by removal. There is no "upgrading" to this since this is a
new feature.
--
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]
