dengliming opened a new pull request, #7064: URL: https://github.com/apache/shenyu/pull/7064
<!-- Describe your PR here; e.g. Fixes #issueNo --> <!-- Thank you for proposing a pull request. This template will guide you through the essential steps necessary for a pull request. --> Make sure that: - [ ] You have read the [contribution guidelines](https://shenyu.apache.org/community/contributor-guide). - [ ] You submit test cases (unit or integration tests) that back your changes. - [ ] Your local test passed `./mvnw clean install -Dmaven.javadoc.skip=true`. ## Summary Make `MqttBootstrapServer.shutdown()` deterministic by closing the server channel before shutting down Netty event loop groups, and waiting for each operation to complete. ## Motivation The CI failure was in: - `org.apache.shenyu.protocol.mqtt.MqttBootstrapServerTest` - `startAndShutdownShouldReleaseChannelAndEventLoops` The test expected the channel to be inactive immediately after `shutdown()` returns, but the previous implementation could return before the channel was fully closed. ## Changes - close the channel first - wait synchronously for channel close completion - gracefully shut down boss and worker event loops - wait for event loop shutdown completion - keep null-safe shutdown behavior -- 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]
