The GitHub Actions job "Run Integration Tests" on 
rocketmq.git/perf/metrics-getmessagetype-map-overload has failed.
Run started by GitHub user wang-jiahua (triggered by wang-jiahua).

Head commit for run:
cb4ff2ecaea5cba29bcdaae606ac4e00af342597 / wangjiahua.wjh 
<[email protected]>
[ISSUE #10486] Add getMessageType(Map) overload to eliminate redundant 
properties decode

BrokerMetricsManager.getMessageType(SendMessageRequestHeader) is called
once per send to classify the message. It internally decodes the
properties String into a HashMap, but the typical caller
(SendMessageProcessor) has already decoded the same String moments
before. The result is a redundant decode allocation per send (one
HashMap + ~14 String substrings + one Node[]).

This commit adds a public overload getMessageType(Map<String, String>)
that lets callers pass an already-decoded Map and reuse it. The
existing SendMessageRequestHeader overload now delegates to the new
overload; behavior is unchanged for callers that don't have a decoded
Map. Downstream callers (e.g. SendMessageProcessor) can switch to the
new overload in a separate broker-layer commit.

Report URL: https://github.com/apache/rocketmq/actions/runs/27587659573

With regards,
GitHub Actions via GitBox

Reply via email to