Issue about delivery.dropped hook and metric

[tigercl]

What happened?

Issue 1

The delivery.dropped hook is not called when the delivery.dropped.too_large count is incremented.

Issue 2

The definition of message.delivered is the number of messages forwarded to subscribers internally in EMQX.

The definition of these metrics: delivery.dropped.expired, delivery.dropped.no_local, is the number of messages dropped before being actually sent to the physical subscriber after being forwarded to the subscriber process.

But now the growth of these metrics such as delivery.dropped.expired occurs before message.delivered, and only delivery.dropped.too_large occurs after message.delivered. This is inconsistent with the definition.

What did you expect to happen?

1. Call delivery.dropped hook when the delivery.dropped.too_large count is incremented.
2. All increments of delivery.dropped.* metrics should be placed after message.delivered

How can we reproduce it (as minimally and precisely as possible)?

No response

Anything else we need to know?

No response

EMQX version

EMQX v5.4.1

OS version

macOS 13.4

Log files