+1 (non binding)

Christian

2015-12-20 7:52 GMT+01:00 Claus Ibsen <[email protected]>:

> Hi
>
> +1 (binding)
>
>
> I gave the binary a quick spin, and created a broker
>
> The readme.html file does not include 1.2.0 changes. But I really
> think the readme.html should not include that details, but refer to
> the website for a changelog / migration guide etc. So the readme can
> be short and simple.
>
> Also the output of the help command in the html looks ugly. Its not
> spaced correctly.
>
> And its a bit unclear that after creating a broker you must cd into
> that directory to start it. eg some people may think you can start the
> broker from the unzip like you can do in Apache ActiveMQ. I would like
> to see the readme more clear on this. Maybe a better step by step
> showing the commands and output etc so its really clear.
>
> Also why not use a .md file instead of .html ?
>
> In JMX the broker is listed as brokerName="0.0.0.0" but I was not
> given any opportunity to name the broker when I created it from the
> command line.
>
> And I cannot see the broker name in the log when the broker start up.
> That name should IMHO be allowed to enter when creating, and maybe
> default to the directory name or something? And show the name in the
> log.
>
> Also I would like to be abel to easily enable jolokia out of the box.
> Apache ActiveMQ has that, but Artemis does not.
>
> The web console is likely a bit disappointment, its just the
> documentation. People in 2015 would expect a management console. So
> maybe point that out in the docs that there is none (yet).
>
> When you create a queue its logged at INFO level and the message is
> not with a capital letter, eg trying -> Trying
>
> [org.apache.activemq.artemis.core.server] AMQ221003: trying to deploy
> queue jms.queue.foo
>
> And is that a debug message instead of INFO ?
>
>
> And when the broker shutdown, I would like to see an INFO logging that
> says the uptime of the broker. See how we do that in ActiveMQ.
>
> ^C INFO | Apache ActiveMQ 5.13.0 (localhost,
> ID:davsclaus.air-51151-1450593630078-0:1) is shutting down
> ...
> INFO | Apache ActiveMQ 5.13.0 (localhost,
> ID:davsclaus.air-51151-1450593630078-0:1) uptime 13.012 seconds
>  INFO | Apache ActiveMQ 5.13.0 (localhost,
> ID:davsclaus.air-51151-1450593630078-0:1) is shutdown
>
> Also IMHO you should log at INFO level when the broker is being
> shutdown, so the user can see from exactly in the log when the
> shutdown starts, in case there is problems during so.
>
> eg as shown above where I started ActiveMQ in the foreground, and then
> hit CTRL+C (eg ^C).
>
>
>
>
> But great to see a new release coming out at steady pace - keep up that.
>
> Really like the protocol detection on the single port. I would love to
> see in JMX some MBeans that can list those protocols and their ports
> in a tree, so tooling is able to discover that.
>
>
>
> On Fri, Dec 18, 2015 at 6:36 PM, Martyn Taylor <[email protected]> wrote:
> > Hello all.
> >
> > I'd like to propose an Apache Artemis 1.2.0 release.
> >
> > Since 1.1.0 we've had some significant improvements to performance,
> > particularly around the persistence layer.  In addition new features have
> > been added such as LDAP and OSGi support.  There has also been more
> > improvements to the OpenWire protocol.
> >
> > The release notes can be found here:
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12333274
> >
> > The binary distributions can be found here:
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1076/org/apache/activemq/apache-artemis/1.2.0/
> >
> > The source archives can be found here:
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1076/org/apache/activemq/apache-artemis/1.2.0/
> >
> > The Maven repository is here:
> >
> https://repository.apache.org/content/repositories/orgapacheactivemq-1076/
> >
> > The source tag:
> >
> https://git-wip-us.apache.org/repos/asf?p=activemq-artemis.git;a=tag;h=refs/tags/1.2.0
> >
> > The project website for that version has been staged to:
> > http://people.apache.org/~martyntaylor/
> >
> > The vote will remain open for 72 hours.
> >
> > [ ] +1 approve the release as Apache Artemis 1.2.0
> > [ ] +0 no opinion
> > [ ] -1 disapprove (and reason why)
> >
> > Here's my (non-binding) +1
> >
> > Regards
> >
> > Martyn
>
>
>
> --
> Claus Ibsen
> -----------------
> http://davsclaus.com @davsclaus
> Camel in Action 2: https://www.manning.com/ibsen2
>



-- 
-- 
Christian Schneider
http://www.liquid-reality.de
<https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e46&URL=http%3a%2f%2fwww.liquid-reality.de>

Open Source Architect
http://www.talend.com
<https://owa.talend.com/owa/redir.aspx?C=3aa4083e0c744ae1ba52bd062c5a7e46&URL=http%3a%2f%2fwww.talend.com>

Reply via email to