Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding in more statistics into the client #119

Closed
wants to merge 4 commits into from

Conversation

benrjm
Copy link

@benrjm benrjm commented Jun 12, 2019

These are extra statistics available in the Rabbit API and they are useful in monitoring how long a message has been in the queue and some additional ACK stats

@AaronEshbach
Copy link

I was about to make PR for the same changes (the ACK Rate/stats), as I have forked the repository to add this myself. This is a really useful statistic if you have subscribers doing manual acknowledgments!

Copy link
Member

@micdenny micdenny left a comment

Choose a reason for hiding this comment

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

thank you for the PR, please check the comment

@@ -4,6 +4,8 @@ namespace EasyNetQ.Management.Client.Model
{
public class MessageStats
{
public Int64 Ack { get; set; }
Copy link
Member

Choose a reason for hiding this comment

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

please use long instead of Int64

@micdenny micdenny added this to the v1.5.0 milestone Mar 16, 2021
WadeTheFade pushed a commit to WadeTheFade/EasyNetQ.Management.Client that referenced this pull request Mar 23, 2021
@Pliner Pliner mentioned this pull request Dec 11, 2022
@Pliner
Copy link
Member

Pliner commented Dec 11, 2022

Fixed in #173

@Pliner Pliner closed this Dec 11, 2022
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.

5 participants