Skip to content

Conversation

@jonmeredith
Copy link
Contributor

@jonmeredith jonmeredith force-pushed the C17252-connection-limit-handler-may-leak-per-ip-count-4.0 branch from 68829a0 to f214615 Compare January 12, 2022 00:54
…n drops

Keep the IP address used to increment as an attribute of the connection
so that it can be decremented when the channel becomes inactive.

Patch by Jon Meredith; reviewed by ? for CASSANDRA-17252
@jonmeredith jonmeredith force-pushed the C17252-connection-limit-handler-may-leak-per-ip-count-4.0 branch from f214615 to 4adbfea Compare January 12, 2022 16:54
@smiklosovic smiklosovic changed the title ConnectionLimitHandler may leaks connection count if remote connection drops CASSANDRA-17252 ConnectionLimitHandler may leaks connection count if remote connection drops 4.0 Mar 16, 2022
@jonmeredith
Copy link
Contributor Author

Committed.

michaeljmarshall pushed a commit to michaeljmarshall/cassandra that referenced this pull request Nov 6, 2024
…pache#1386)

* CNDB-11434: Refactor TrieMemtableIndexTest

* CNDB-11434: Refactor TinySegmentFlushingFailureTest

* CNDB-11434: Make all tests in TrieMemtableIndexTest to be run with both offheap and onheap to improve test coverage
blambov pushed a commit to blambov/cassandra that referenced this pull request Feb 20, 2025
…pache#1386)

* CNDB-11434: Refactor TrieMemtableIndexTest

* CNDB-11434: Refactor TinySegmentFlushingFailureTest

* CNDB-11434: Make all tests in TrieMemtableIndexTest to be run with both offheap and onheap to improve test coverage
@jonmeredith jonmeredith deleted the C17252-connection-limit-handler-may-leak-per-ip-count-4.0 branch April 22, 2025 22:21
michaelsembwever pushed a commit to thelastpickle/cassandra that referenced this pull request Jan 7, 2026
…pache#1386)

* CNDB-11434: Refactor TrieMemtableIndexTest

* CNDB-11434: Refactor TinySegmentFlushingFailureTest

* CNDB-11434: Make all tests in TrieMemtableIndexTest to be run with both offheap and onheap to improve test coverage
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