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
