Skip to content

Conversation

russcam
Copy link
Contributor

@russcam russcam commented Sep 7, 2020

This commit adds the ability to enable TCP and ThreadPool
statistics to be collected when making a request, exposing
these statistics on the IApiCallDetails on the response, as
well as including in the debug information.

Add documentation explaining how to include these statistics
either on a global or per request basis.

This commit adds the ability to enable TCP and ThreadPool
statistics to be collected when making a request, exposing
these statistics on the IApiCallDetails on the response, as
well as including in the debug information.

Add documentation explaining how to include these statistics
either on a global or per request basis.
@russcam
Copy link
Contributor Author

russcam commented Sep 7, 2020

Thoughts on including this in 7.9.1, or 7.10.0, @Mpdreamz ?

@russcam russcam requested a review from Mpdreamz September 7, 2020 08:54
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

Change looks good and much welcomed! Definitely can go out in 7.9.1

I wonder if it might be good to create include these types under https://github.com/elastic/elasticsearch-net/tree/master/src/Elasticsearch.Net/Diagnostics and in the Diagnostics namespace?

debugCompletedRequestHandler.Invoke(d);
};
});
_completedRequestHandler += v ?? (d => Debug.WriteLine(d.DebugInformation)));
Copy link
Member

Choose a reason for hiding this comment

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

👍 💯

@russcam
Copy link
Contributor Author

russcam commented Sep 7, 2020

I wonder if it might be good to create include these types under https://github.com/elastic/elasticsearch-net/tree/master/src/Elasticsearch.Net/Diagnostics and in the Diagnostics namespace?

I was initially thinking of putting them there, then wondered if that should strictly be for System.Diagnostics related types. I'll move them there 👍

@russcam
Copy link
Contributor Author

russcam commented Sep 8, 2020

Failures are related to transforms; opened #5004 to address

@russcam russcam merged commit bf1768b into master Sep 8, 2020
@russcam russcam deleted the feature/tcp-threadpool-stats branch September 8, 2020 06:10
github-actions bot pushed a commit that referenced this pull request Sep 8, 2020
This commit adds the ability to enable TCP and ThreadPool
statistics to be collected when making a request, exposing
these statistics on the IApiCallDetails on the response, as
well as including in the debug information.

Add documentation explaining how to include these statistics
either on a global or per request basis.
github-actions bot pushed a commit that referenced this pull request Sep 8, 2020
This commit adds the ability to enable TCP and ThreadPool
statistics to be collected when making a request, exposing
these statistics on the IApiCallDetails on the response, as
well as including in the debug information.

Add documentation explaining how to include these statistics
either on a global or per request basis.
russcam added a commit that referenced this pull request Sep 8, 2020
This commit adds the ability to enable TCP and ThreadPool
statistics to be collected when making a request, exposing
these statistics on the IApiCallDetails on the response, as
well as including in the debug information.

Add documentation explaining how to include these statistics
either on a global or per request basis.

Co-authored-by: Russ Cam <russ.cam@elastic.co>
russcam added a commit that referenced this pull request Sep 8, 2020
* Enable TCP states and ThreadPool stats (#5003)

This commit adds the ability to enable TCP and ThreadPool
statistics to be collected when making a request, exposing
these statistics on the IApiCallDetails on the response, as
well as including in the debug information.

Add documentation explaining how to include these statistics
either on a global or per request basis.

Co-authored-by: Russ Cam <russ.cam@elastic.co>
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