This is really great feedback Claus, it's exactly the type of thing we need. Thanks for taking the time to write this up.

We'll get all your comments captured in JIRA and aim to address them in the next release.

On 20/12/15 06:52, Claus Ibsen wrote:
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



Reply via email to