Github user ted-ross commented on a diff in the pull request:

    https://github.com/apache/qpid-dispatch/pull/247#discussion_r167675656
  
    --- Diff: src/router_core/agent_address.c ---
    @@ -54,6 +56,8 @@ const char *qdr_address_columns[] =
          "deliveriesTransit",
          "deliveriesToContainer",
          "deliveriesFromContainer",
    +     "deliveriesToRouteContainer",
    +     "deliveriesFromRouteContainer",
    --- End diff --
    
    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".


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to