Skip to content

Reduce channels in AbstractSimpleTransportTestCase (#34863) (#34880)#34915

Merged
Tim-Brooks merged 2 commits intoelastic:6.4from
Tim-Brooks:backport_reduce_channels_to_64
Oct 26, 2018
Merged

Reduce channels in AbstractSimpleTransportTestCase (#34863) (#34880)#34915
Tim-Brooks merged 2 commits intoelastic:6.4from
Tim-Brooks:backport_reduce_channels_to_64

Conversation

@Tim-Brooks
Copy link
Copy Markdown
Contributor

This is related to #30876. The AbstractSimpleTransportTestCase initiates
many tcp connections. There are normally over 1,000 connections in
TIME_WAIT at the end of the test. This is because every test opens at
least two different transports that connect to each other with 13
channel connection profiles. This commit modifies the default
connection profile used by this test to 6. One connection for each
type, except for REG which gets 2 connections.

…lastic#34880)

This is related to elastic#30876. The AbstractSimpleTransportTestCase initiates
many tcp connections. There are normally over 1,000 connections in
TIME_WAIT at the end of the test. This is because every test opens at
least two different transports that connect to each other with 13
channel connection profiles. This commit modifies the default
connection profile used by this test to 6. One connection for each
type, except for REG which gets 2 connections.
@Tim-Brooks Tim-Brooks added >test Issues or PRs that are addressing/adding tests :Distributed/Network Http and internode communication implementations labels Oct 26, 2018
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

@Tim-Brooks Tim-Brooks merged commit 3f9d17f into elastic:6.4 Oct 26, 2018
@Tim-Brooks Tim-Brooks deleted the backport_reduce_channels_to_64 branch December 18, 2019 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Network Http and internode communication implementations >test Issues or PRs that are addressing/adding tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants