Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@varshith257
Copy link
Contributor

@varshith257 varshith257 commented Jun 2, 2025

/claim #9913
Fixes #9913

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the ZIOApp documentation by introducing a new section on graceful shutdown timeout behavior.

  • Added a new documentation section "Graceful Shutdown Timeout" explaining the default behavior and how to override it.
  • Included a Scala code snippet demonstrating how to set a shutdown timeout.
Comments suppressed due to low confidence (1)

docs/reference/core/zioapp.md:138

  • The error message refers to 'shutdownTimeout' rather than 'gracefulShutdownTimeout'. Please update the method name in the message for consistency with the documented override.
Timed out waiting for ZIO application to shut down after 30 seconds. You can adjust your application's shutdown timeout by overriding the `shutdownTimeout` method

@varshith257 varshith257 requested a review from kyri-petrou June 4, 2025 06:44
@varshith257
Copy link
Contributor Author

@kyri-petrou looks like there is an unrelated test failure in CI.
You can see it here: https://github.com/zio/zio/actions/runs/15444074136/job/43468853375#step:6:5854

@varshith257 varshith257 requested a review from kyri-petrou June 4, 2025 14:23
aadarsh-nagrath added a commit to aadarsh-nagrath/zio that referenced this pull request Jun 6, 2025
zio#9923) - Add comprehensive documentation for gracefulShutdownTimeout method - Improve note styling with concise, clear formatting - Document platform-specific behavior (JVM, Scala Native, Scala.js) - Add examples for common use cases (database, HTTP server) - Include best practices for timeout configuration and resource management
@kyri-petrou kyri-petrou merged commit 46178b5 into zio:series/2.x Jun 7, 2025
18 checks passed
@varshith257 varshith257 deleted the add-docs branch June 7, 2025 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document ZIOApp#gracefulShutdownTimeout

2 participants