fix: UDP_FORWARDERを長時間起動していると新規リクエストを受け付けなくなる問題を修正#235
Draft
toku-sa-n wants to merge 9 commits intoDeNA:masterfrom
Draft
fix: UDP_FORWARDERを長時間起動していると新規リクエストを受け付けなくなる問題を修正#235toku-sa-n wants to merge 9 commits intoDeNA:masterfrom
toku-sa-n wants to merge 9 commits intoDeNA:masterfrom
Conversation
Limit active UDP forwarding connections and clean up old resources so long-running traffic no longer exhausts native threads and stops receiving new packets. Made-with: Cursor
Replace the per-connection duplex flow with a lighter datagram forwarding path so the UDP forwarder spends less time creating transient sessions during bursts. Made-with: Cursor
Revert the datagram-forwarding rewrite because it breaks UDP forwarder behavior and removes the existing duplex-based processing path. Made-with: Cursor
Shut down the per-connection receive executor when a UDPConn closes so evicted UDP forwarder connections do not leave worker threads behind. Add a regression test that fails until the cleanup path actually terminates the created thread. Made-with: Cursor
Keep the UDP leak fixes intact while removing style-only final additions on pre-existing fields. Made-with: Cursor
Keep the UDP thread leak fix focused by removing cached RawEndpoint members that do not change behavior. Made-with: Cursor
Keep the UDP thread leak fix focused by dropping the extra start-once guard that is not required by the current call flow. Made-with: Cursor
Make the UDPConn lifecycle explicit by failing fast when automatic receive is requested after the connection has already been closed. Made-with: Cursor
Extract the repeated close-and-ignore pattern so the shutdown path stays compact without changing behavior. Made-with: Cursor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
UDP_FORWARDERにおいて、接続やスレッドが増え続けてしまう問題を解決しました。