Skip to content

Comments

DISPATCH-911 - Added global delivery counts#247

Closed
ganeshmurthy wants to merge 1 commit intoapache:masterfrom
ganeshmurthy:DISPATCH-911
Closed

DISPATCH-911 - Added global delivery counts#247
ganeshmurthy wants to merge 1 commit intoapache:masterfrom
ganeshmurthy:DISPATCH-911

Conversation

@ganeshmurthy
Copy link
Contributor

No description provided.

Copy link
Contributor

@ErnieAllen ErnieAllen left a comment

Choose a reason for hiding this comment

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

These new statistics look good to me.
Questions:

  1. Using qdstat, how do I see the stats accumulated across all routers (in core?) instead of just one router?
  2. Same question when making a management call. How do I request the aggregate stats?

@ganeshmurthy
Copy link
Contributor Author

@ErnieAllen you will have to query every router in the network and aggregate the stats. thx

"deliveriesToContainer",
"deliveriesFromContainer",
"deliveriesToRouteContainer",
"deliveriesFromRouteContainer",
Copy link
Member

Choose a reason for hiding this comment

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

I find the names of these two new attributes misleading. They appear to be grouped with the former two attributes "deliveriesToContainer" and "deliveriesFromContainer" because of the similar naming. The new stats are supposed to be grouped with "deliveriesIngress" and "deliveriesEgress". I think better names would be "deliveriesIngressRouteContainer" and "deliveriesEgressRouteContainer".

ChugR pushed a commit to ChugR/qpid-dispatch that referenced this pull request Oct 13, 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.

3 participants