adriangb commented on code in PR #26565:
URL: https://github.com/apache/datafusion/pull/26565#discussion_r4383203037
##########
docs/source/library-user-guide/upgrading/56.0.0.md:
##########
@@ -1185,9 +1182,7 @@ A session carries one registry. Each library adds its
decoders to it with `Sessi
```rust,ignore
// In each library that ships extension types:
pub fn register(config: &mut SessionConfig) -> Result<()> {
- config.update_extension(|registry: &mut ProtoDecoderRegistry| {
- register_execution_plan::<MyExec>(registry)
- })
+ config.update_extension(MyExec::register)
}
Review Comment:
Good point, renamed to `register_proto` (on both `ExtensionPlanFromProto`
and `ExtensionExprFromProto`) in b21cae945. None of this API has shipped in a
release yet (it's all new since 55.2.0), so the rename doesn't break anything.
--
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]
