[prev in list] [next in list] [prev in thread] [next in thread] 

List:       ws-general
Subject:    Re: Removing Qpid from JMS unit tests
From:       robertlazarski <robertlazarski () gmail ! com>
Date:       2022-03-09 3:11:41
Message-ID: CABpPLBU1Sr1Dn3xT3LjvJGGeaGdf=nUnpxKX53B1qH7dDGbzqQ () mail ! gmail ! com
[Download RAW message or body]

Wrong list sorry - this discussion is Axis related.

On Tue, Mar 8, 2022 at 4:54 PM robertlazarski <robertlazarski@gmail.com>
wrote:

> We support two different ways to run our JMS unit tests - active mq, and
> qpid.
>
> Qpid is a problem because these tests were written many years ago and the
> ancient version used has a log4j 1.x dependency that we've been asked to
> remove in AXIS2-5857.
>
> Qpid went through huge changes in the last 10 years and our unit tests
> didn't keep up.
>
> At this point, the Qpid changes are far too great to refactor the test. I
> tried updating the code to the latest dep and it was far from easy as the
> changes to the project are huge.
>
> To be clear, Qpid and JMS together in unit tests are not trivial. I don't
> recall any Axis2 users of Qpid.
>
> So I am looking at removing Qpid from the unit tests. Just thought I'd
> bring it up for discussion before I take action.
>
>
>
>

[Attachment #3 (text/html)]

<div dir="ltr">Wrong list sorry - this discussion is Axis related. <br></div><br><div \
class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 8, 2022 at 4:54 PM \
robertlazarski &lt;<a \
href="mailto:robertlazarski@gmail.com">robertlazarski@gmail.com</a>&gt; \
wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px \
0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>We \
support two different ways to run our JMS unit tests - active mq, and qpid. \
<br></div><div><br></div><div>Qpid is a problem because these tests were written many \
years ago and the ancient version used has a log4j 1.x dependency that we&#39;ve been \
asked to remove in AXIS2-5857. <br></div><div><br></div><div>Qpid went through huge \
changes in the last 10 years and our unit tests didn&#39;t keep up. \
<br></div><div><br></div><div>At this point, the Qpid changes are far too great to \
refactor the test. I tried updating the code to the latest dep and it was far from \
easy as the changes to the project are huge. <br></div><div><br></div><div>To be \
clear, Qpid and JMS together in unit tests are not trivial. I don&#39;t recall any \
Axis2 users of Qpid. <br></div><div><br></div><div>So I am looking at removing Qpid \
from the unit tests. Just thought I&#39;d bring it up for discussion before I take \
action. <br></div><div><br></div><div><br></div><div><br></div></div> \
</blockquote></div>



[prev in list] [next in list] [prev in thread] [next in thread]