jbertram commented on PR #5689: URL: https://github.com/apache/artemis/pull/5689#issuecomment-3886674849
After thinking about this for awhile I believe this problem would be better solved with a new identifier that could be used in a consumer's [filter](https://artemis.apache.org/components/artemis/documentation/latest/filter-expressions.html#filter-expressions). Currently we have `AMQSize`, but that works on the message's "encoded" size (see [here](https://github.com/apache/artemis/blob/a7ac3ab386b4e4cdc58e1716f26c5ae288e46508/artemis-server/src/main/java/org/apache/activemq/artemis/core/filter/impl/FilterImpl.java#L178)) which doesn't include the body of a large message. Adding something like `AMQFullSize` that used `org.apache.activemq.artemis.api.core.Message#getWholeMessageSize` would solve this problem. Other benefits include: - The message wouldn't even be sent across the wire. This would save bandwidth & time. - It would work on all protocols & clients that support filtering instead of only on the Core JMS client. - It would allow other consumers on the queue to get the message without having to wait for it to be rejected by another consumer. Would you be interested in modifying your PR to implement this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
