-
Notifications
You must be signed in to change notification settings - Fork 188
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove OperationRegistryBuilder
and Router
#1707
Labels
server
Rust server SDK
Comments
hlbarber
changed the title
Remove of
Remove of Sep 5, 2022
OperationRegistryBuilder
and Router
machineryOperationRegistryBuilder
and Router
hlbarber
changed the title
Remove of
Remove Sep 6, 2022
OperationRegistryBuilder
and Router
OperationRegistryBuilder
and Router
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The conclusion of Service Builder Improvements is the removal of the original service builder,
OperationRegistryBuilder
, along with it's output,Router
.A list of chores which should be done during the removal:
Router
associated code, and promote the protocol specific routers code to a more accessible module.from_request
/into_response
methods #1706, this can be done prior to the removal and blocks the further steps.rust-runtime/inlineable/src/server_operation_handler_trait.rs
.kotlin/software/amazon/smithy/rust/codegen/server/smithy/generators/ServerServiceGenerator.kt
and integrate it withkotlin/software/amazon/smithy/rust/codegen/server/smithy/generators/ServerServiceGeneratorV2.kt
.kotlin/software/amazon/smithy/rust/codegen/server/smithy/generators/ServerOperationHandlerGenerator.kt
.The text was updated successfully, but these errors were encountered: