[ 
https://issues.apache.org/jira/browse/ARTEMIS-2682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Federico Valeri updated ARTEMIS-2682:
-------------------------------------
    Description: I'm proposing theĀ {{OpenWireMessageConverter}} refactoring to 
have a similar layout as the {{AMQPConverter}}. The latter has two dedicated 
classes for inbound and outbound messages with a singleton entry point for the 
converter. This change is basically doing the same without semantics change and 
also adding a unit test for the converter. I'm also eliminating some duplicated 
code, but the bulk of the code remains untouched.  (was: I'm proposing the 
OpenWireMessageConverter refactoring to have a similar layout as the 
AMQPConverter. The latter has two dedicated classes for inbound and outbound 
messages with a singleton entry point for the converter. This change is 
basically doing the same without semantics change and also adding a unit test 
for the converter. I'm also eliminating some duplicated code, but the bulk of 
the code remains untouched.)

> OpenWireMessageConverter refactoring
> ------------------------------------
>
>                 Key: ARTEMIS-2682
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-2682
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>          Components: OpenWire
>    Affects Versions: 2.12.0
>            Reporter: Federico Valeri
>            Priority: Minor
>              Labels: openwire, refactoring
>
> I'm proposing theĀ {{OpenWireMessageConverter}} refactoring to have a similar 
> layout as the {{AMQPConverter}}. The latter has two dedicated classes for 
> inbound and outbound messages with a singleton entry point for the converter. 
> This change is basically doing the same without semantics change and also 
> adding a unit test for the converter. I'm also eliminating some duplicated 
> code, but the bulk of the code remains untouched.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to