Skip to content

Change bufferSize back to 32#3560

Merged
fuweid merged 1 commit intocontainerd:masterfrom
keloyang:buffersize-32
Aug 20, 2019
Merged

Change bufferSize back to 32#3560
fuweid merged 1 commit intocontainerd:masterfrom
keloyang:buffersize-32

Conversation

@keloyang
Copy link
Copy Markdown
Contributor

@keloyang keloyang commented Aug 20, 2019

Shim use non-blocking send now, there is no need to set bufferSize to 2048,
it's a waste.

link to #3540

Signed-off-by: Shukui Yang [email protected]

Shim use non-blocking send now, there is no need to set bufferSize to 2048,
it's a waste.

Signed-off-by: Shukui Yang <[email protected]>
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3560 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #3560   +/-   ##
======================================
  Coverage    42.3%   42.3%           
======================================
  Files         126     126           
  Lines       13869   13869           
======================================
  Hits         5867    5867           
  Misses       7116    7116           
  Partials      886     886
Flag Coverage Δ
#linux 45.8% <ø> (ø) ⬆️
#windows 37.26% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 08061c7...21174cb. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 20, 2019

Build succeeded.

Copy link
Copy Markdown
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

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

LGTM

@fuweid fuweid merged commit cd76c41 into containerd:master Aug 20, 2019
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.

4 participants