Skip to content
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

demos/tcp_sctp_server_demo: Modernize with seastar::async and proper teardown #2685

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tchaikov
Copy link
Contributor

@tchaikov tchaikov commented Mar 9, 2025

Replace deprecated at_exit() with the recommended seastar::async + defer() pattern. This change:

  • Captures background task future as member variable for proper cleanup
  • Implements complete service teardown sequence (abort accept → wait for task)
  • Upgrades to app.run() from deprecated app.run_deprecated()
  • Uses stop_signal for clean shutdown handling

The demo now follows current Seastar best practices for initialization, resource management, and graceful termination.
stop using at_exit().

Ref eefe659. which deprecated at_exit().

…teardown

Replace deprecated at_exit() with the recommended seastar::async + defer()
pattern. This change:

- Captures background task future as member variable for proper cleanup
- Implements complete service teardown sequence (abort accept → wait for task)
- Upgrades to app.run() from deprecated app.run_deprecated()
- Uses stop_signal for clean shutdown handling

The demo now follows current Seastar best practices for initialization,
resource management, and graceful termination.
stop using at_exit().

Ref eefe659. which deprecated `at_exit()`.

Signed-off-by: Kefu Chai <kefu.chai@scylladb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant