Skip to content

Back-pressure handler in NettyHelixIPCService#4

Merged
asfgit merged 1 commit intoapache:masterfrom
brandtg:master
Sep 16, 2014
Merged

Back-pressure handler in NettyHelixIPCService#4
asfgit merged 1 commit intoapache:masterfrom
brandtg:master

Conversation

@brandtg
Copy link
Copy Markdown

@brandtg brandtg commented Sep 16, 2014

Removes hack in which Netty Channel was periodically closed to allow Netty objects to be GC'ed in favor of properly using channel writability changed events.

Before calling Channel#writeAndFlush, we call BackPressureHandler#waitUntilWritable, which will block if the channel is not writable until #channelWritabilityChanged is called.

Ran BenchmarkDriver locally to verify that heap usage stays relatively low and constant when NIC saturated.

@kanakb
Copy link
Copy Markdown
Member

kanakb commented Sep 16, 2014

Verified build and mvn test. LGTM.

@asfgit asfgit merged commit 6c0f13b into apache:master Sep 16, 2014
kaisun2000 referenced this pull request in kaisun2000/helix Oct 2, 2020
# This is the 1st commit message:

09/28 fix race condition t1

# This is the commit message #2:

09/28 fix race condition t2

# This is the commit message #3:

09/28 fix race condition t3

# This is the commit message #4:

09/28 fix race condition t4

# This is the commit message #5:

09/28 fix race condition t5

# This is the commit message apache#6:

09/28 fix race condition t6
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.

3 participants