New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARTEMIS-3556 show message protocol on message view page #3841
Conversation
...src/main/java/org/apache/activemq/artemis/core/message/openmbean/MessageOpenTypeFactory.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see inline comment.
@@ -123,6 +124,7 @@ protected void init() throws OpenDataException { | |||
public Map<String, Object> getFields(M m, int valueSizeLimit, int deliveryCount) throws OpenDataException { | |||
Map<String, Object> rc = new HashMap<>(); | |||
rc.put(CompositeDataConstants.MESSAGE_ID, "" + m.getMessageID()); | |||
rc.put(CompositeDataConstants.PROTOCOL, m.getClass().getSimpleName()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we could add a method on Message to return a more user friendly word?
on Message:
public String getProtocol();
then on core you return "CORE" and AMQP you return "AMQP"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I already made that change exactly as you and Michael suggest. that code is in the next commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe I should squash to prevent confusion?
see https://issues.apache.org/jira/browse/ARTEMIS-3556