[GitHub] activemq-6 pull request: Few changes around docs and dependencies

2015-03-13 Thread clebertsuconic
GitHub user clebertsuconic opened a pull request:

https://github.com/apache/activemq-6/pull/183

Few changes around docs and dependencies



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/clebertsuconic/activemq-6 master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-6/pull/183.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #183


commit 7ce776776a3492bd34eefe29c48da3fe7e2645c1
Author: Clebert Suconic 
Date:   2015-03-13T22:37:23Z

Removing extra dependencies brought by transitivity on rest

commit f47064c426550998099c087164bf315ce235ec55
Author: Clebert Suconic 
Date:   2015-03-13T22:37:28Z

A Few fixes around docs using GMF (Github Markdown Flavored) on tables

I also did some review of the huge table we have,
removed a few columns such as the datatype and
embedded the default on the description. Just trying to make easier to 
render and read




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Updated] (AMQ-5646) AMQP: Second receiver attaching to queue hangs

2015-03-13 Thread Timothy Bish (JIRA)

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

Timothy Bish updated AMQ-5646:
--
Fix Version/s: 5.12.0

> AMQP: Second receiver attaching to queue hangs
> --
>
> Key: AMQ-5646
> URL: https://issues.apache.org/jira/browse/AMQ-5646
> Project: ActiveMQ
>  Issue Type: Bug
>  Components: AMQP
>Affects Versions: 5.11.0
> Environment: ActiveMQ 5.11.1 release on Fedora 19
> Test client: http://amqpnetlite.codeplex.com/ 
> LinkTests.cs 
> http://amqpnetlite.codeplex.com/SourceControl/latest#test/Test.Amqp.NetMF/LinkTests.cs
>Reporter: Chuck Rolke
>Assignee: Timothy Bish
> Fix For: 5.12.0
>
> Attachments: amq-5.11-second-receiver-hangs.html, 
> qpidd-second-receiver-works.html
>
>
> The last test in the file, TestMethod_AdvancedLinkFlowControl() hangs. The 
> test:
> * opens a connection and session
> * open a sender and send a batch of messages
> * open a receiver with credit of 2 and no automatic credit renewal
> * receive two messages, accept them
> * open a second receiver with credit of 2 and no automatic credit renewal
> * receive a two messages
> The broker does not send the messages to the second receiver.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (AMQ-5646) AMQP: Second receiver attaching to queue hangs

2015-03-13 Thread Timothy Bish (JIRA)

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

Timothy Bish resolved AMQ-5646.
---
Resolution: Fixed

Fixed by changes in AMQ-5661

> AMQP: Second receiver attaching to queue hangs
> --
>
> Key: AMQ-5646
> URL: https://issues.apache.org/jira/browse/AMQ-5646
> Project: ActiveMQ
>  Issue Type: Bug
>  Components: AMQP
>Affects Versions: 5.11.0
> Environment: ActiveMQ 5.11.1 release on Fedora 19
> Test client: http://amqpnetlite.codeplex.com/ 
> LinkTests.cs 
> http://amqpnetlite.codeplex.com/SourceControl/latest#test/Test.Amqp.NetMF/LinkTests.cs
>Reporter: Chuck Rolke
>Assignee: Timothy Bish
> Attachments: amq-5.11-second-receiver-hangs.html, 
> qpidd-second-receiver-works.html
>
>
> The last test in the file, TestMethod_AdvancedLinkFlowControl() hangs. The 
> test:
> * opens a connection and session
> * open a sender and send a batch of messages
> * open a receiver with credit of 2 and no automatic credit renewal
> * receive two messages, accept them
> * open a second receiver with credit of 2 and no automatic credit renewal
> * receive a two messages
> The broker does not send the messages to the second receiver.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Resolved] (AMQ-5661) AMQP: Honor receiver flow as the true prefetch value on a link.

2015-03-13 Thread Timothy Bish (JIRA)

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

Timothy Bish resolved AMQ-5661.
---
Resolution: Fixed

Link credit now configures subscription prefetch.  Old prefetch value on the 
AMQP transport connector is now unused and deprecated and will be removed in a 
future release.  

> AMQP: Honor receiver flow as the true prefetch value on a link.
> ---
>
> Key: AMQ-5661
> URL: https://issues.apache.org/jira/browse/AMQ-5661
> Project: ActiveMQ
>  Issue Type: Sub-task
>  Components: AMQP
>Affects Versions: 5.11.0, 5.11.1
>Reporter: Timothy Bish
>Assignee: Timothy Bish
> Fix For: 5.12.0
>
>
> We currently don't honor the flow values sent from a receiver to indicate how 
> many messages our sender can produce (prefetch) and always maintain a fixed 
> value based either on a transport configuration option or the first non-zero 
> flow.  This is however not in keeping with how the credit model is expected 
> to work in AMQP and leads to some issues with the brokers sender accepting 
> messages and buffering them until the receiver sends a flow thereby 
> preventing those messages from being dispatched to another receiver 
> subscribed to the same destination.  
> We need to honor the link credit and only dispatch when the receiver 
> indicates it is willing to take messages by sending more link credit.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: ActiveMQ DSLs

2015-03-13 Thread David Jencks
Unfortunately your sample didn't show up on the mailing list…. I had to resort 
to looking at nabble.

I think your example is too simple to see the benefits of a fluent api. For 
instance, you haven't added an object that itself needs configuration to a 
collection (e.g. transport connector).  My experience is that they make 
configuring anything reasonably complicated 10x simpler and more pleasant :-)

That being said….. I'm not familiar with the ".end()" idiom.  Is this common? 
"standard"?  I'd think it would be less confusing if they were 
.endTransportConnectors() etc so it was clear what you were ending.

thanks
david jencks


On Mar 13, 2015, at 5:32 PM, artnaseef  wrote:

> I see "fluent is easier" as a valid argument to maintaining the project, but
> not necessarily as a reason to pull it into the ActiveMQ main code base.
> 
> Still, I am 50/50 on this myself.  I personally don't have a need for
> simpler configuration of embedded brokers.  Also, the syntax of fluent
> builders doesn't appeal to me personally, so that makes it harder for me to
> buy-in.  For example:
> 
> Seems simpler to understand than (I'm making this one up - not using the
> posted language):
> 
> With that said - I acknowledge this is my opinion and welcome some
> discussion of the benefit of including the code inside of ActiveMQ itself
> (beyond the well-understood "it will then be maintained with the core
> code").
> 
> Art
> 
> 
> 
> --
> View this message in context: 
> http://activemq.2283324.n4.nabble.com/ActiveMQ-DSLs-tp4693171p4693202.html
> Sent from the ActiveMQ - Dev mailing list archive at Nabble.com.



[jira] [Commented] (AMQ-5661) AMQP: Honor receiver flow as the true prefetch value on a link.

2015-03-13 Thread Timothy Bish (JIRA)

[ 
https://issues.apache.org/jira/browse/AMQ-5661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14361187#comment-14361187
 ] 

Timothy Bish commented on AMQ-5661:
---

Root cause for this related issue.

> AMQP: Honor receiver flow as the true prefetch value on a link.
> ---
>
> Key: AMQ-5661
> URL: https://issues.apache.org/jira/browse/AMQ-5661
> Project: ActiveMQ
>  Issue Type: Sub-task
>  Components: AMQP
>Affects Versions: 5.11.0, 5.11.1
>Reporter: Timothy Bish
>Assignee: Timothy Bish
> Fix For: 5.12.0
>
>
> We currently don't honor the flow values sent from a receiver to indicate how 
> many messages our sender can produce (prefetch) and always maintain a fixed 
> value based either on a transport configuration option or the first non-zero 
> flow.  This is however not in keeping with how the credit model is expected 
> to work in AMQP and leads to some issues with the brokers sender accepting 
> messages and buffering them until the receiver sends a flow thereby 
> preventing those messages from being dispatched to another receiver 
> subscribed to the same destination.  
> We need to honor the link credit and only dispatch when the receiver 
> indicates it is willing to take messages by sending more link credit.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (AMQ-5661) AMQP: Honor receiver flow as the true prefetch value on a link.

2015-03-13 Thread Timothy Bish (JIRA)
Timothy Bish created AMQ-5661:
-

 Summary: AMQP: Honor receiver flow as the true prefetch value on a 
link.
 Key: AMQ-5661
 URL: https://issues.apache.org/jira/browse/AMQ-5661
 Project: ActiveMQ
  Issue Type: Sub-task
  Components: AMQP
Affects Versions: 5.11.1, 5.11.0
Reporter: Timothy Bish
Assignee: Timothy Bish
 Fix For: 5.12.0


We currently don't honor the flow values sent from a receiver to indicate how 
many messages our sender can produce (prefetch) and always maintain a fixed 
value based either on a transport configuration option or the first non-zero 
flow.  This is however not in keeping with how the credit model is expected to 
work in AMQP and leads to some issues with the brokers sender accepting 
messages and buffering them until the receiver sends a flow thereby preventing 
those messages from being dispatched to another receiver subscribed to the same 
destination.  

We need to honor the link credit and only dispatch when the receiver indicates 
it is willing to take messages by sending more link credit.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: ActiveMQ DSLs

2015-03-13 Thread artnaseef
I see "fluent is easier" as a valid argument to maintaining the project, but
not necessarily as a reason to pull it into the ActiveMQ main code base.

Still, I am 50/50 on this myself.  I personally don't have a need for
simpler configuration of embedded brokers.  Also, the syntax of fluent
builders doesn't appeal to me personally, so that makes it harder for me to
buy-in.  For example:

Seems simpler to understand than (I'm making this one up - not using the
posted language):

With that said - I acknowledge this is my opinion and welcome some
discussion of the benefit of including the code inside of ActiveMQ itself
(beyond the well-understood "it will then be maintained with the core
code").

Art



--
View this message in context: 
http://activemq.2283324.n4.nabble.com/ActiveMQ-DSLs-tp4693171p4693202.html
Sent from the ActiveMQ - Dev mailing list archive at Nabble.com.


Re: Active MQ implemenation from multiple clients/devices

2015-03-13 Thread artnaseef
Look at JMSReplyTo - it was intended specifically for that purpose.

The requestor tells the processor where it wants the response delivered. 
This page[1] might help

[1]
http://activemq.apache.org/how-should-i-implement-request-response-with-jms.html



--
View this message in context: 
http://activemq.2283324.n4.nabble.com/Active-MQ-implemenation-from-multiple-clients-devices-tp4693149p4693200.html
Sent from the ActiveMQ - Dev mailing list archive at Nabble.com.


[jira] [Commented] (AMQ-5660) Dequeue count for Topics does not increase for durable subscribers

2015-03-13 Thread Arthur Naseef (JIRA)

[ 
https://issues.apache.org/jira/browse/AMQ-5660?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14361127#comment-14361127
 ] 

Arthur Naseef commented on AMQ-5660:


Because of the nature of topics (being a broadcast) dequeue count is a little 
tricky.  For example, how many times do you count a message consumed by 
multiple subscribers?  Once after they all have consumed it (and what if one 
subscriber drops before acking the message)?  Once per subscriber (leading to 
dequeue growing faster than enqueue)?

And what about counting dequeue on non-durable subscriptions?  For inflight 
messages or only acked messages?  Messages published to a Topic with no 
consumers?

In other words, the meaning of dequeue on a Topic itself is hard to nail down, 
and even if it were nailed down, it would be hard to use reliably.  Also keep 
in mind that ActiveMQ does not store messages on Topics, but instead stores 
them on the subscriptions.

Therefore, I find the best method of watching enqueue and dequeue counts for 
Topics is looking at individual subscriptions.

> Dequeue count for Topics does not increase for durable subscribers
> --
>
> Key: AMQ-5660
> URL: https://issues.apache.org/jira/browse/AMQ-5660
> Project: ActiveMQ
>  Issue Type: Bug
>Affects Versions: 5.6.0
> Environment: 
>Reporter: Jessy
>
> I upgraded my ActiveMQ from 5.2.1 to 5.6.0.
> I have durable topic subscribers .
> I see the "subcribers" tab in activemqConsole and i find that the these 
> subscribers are able to consume the messages from the topic . 
> The "subcribers" tab shows up all the counter's correctly - the dispatched 
> counter equals enqueue counter and dequeue counters.
> However, When i check the topic's enque and dequeue count in "Topics" tab, I 
> see the dequeue count is always zero and not increased ,even thought the 
> subcribers are reported to consume them as seen in the "subcribers" tab.
> I did not face this issue with my previous version of ActiveMQ - version 
> 5.2.1. 
> Version ActiveMQ 5.2.1 works very good in showing up the correct dequeue 
> count for durable topic subscribers.
> My subscribers are on different versions of ActiveMQ.
> Please advise the steps that i need to consider get the dequeue counter match 
> with the enqueue counter .
> I came accross exactly the same issue on Jira for version 5.8.0.  -- 
> https://issues.apache.org/jira/browse/AMQ-4965
> If this is a bug to be fixed, please help us get a fix for this.
> Thanks 
> Jessy



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: ActiveMQ DSLs

2015-03-13 Thread David Jencks
Speaking as a non-user of AMQ, I would say that if this fluent approach is 
easier to use than the existing way of configuring and starting a broker in 
java, then it should be added to AMQ.  

thanks
david jencks

On Mar 13, 2015, at 4:13 PM, artnaseef  wrote:

> I'm 50/50 on this.  Like Hadrian said, it's nice syntax, but does it really
> add value?  I don't see anything there that cannot be done already via java
> code - is that right?
> 
> I think this is a good project to maintain, but not sure it should go into
> ActiveMQ itself.
> 
> 
> 
> --
> View this message in context: 
> http://activemq.2283324.n4.nabble.com/ActiveMQ-DSLs-tp4693171p4693187.html
> Sent from the ActiveMQ - Dev mailing list archive at Nabble.com.



Re: ActiveMQ DSLs

2015-03-13 Thread artnaseef
I'm 50/50 on this.  Like Hadrian said, it's nice syntax, but does it really
add value?  I don't see anything there that cannot be done already via java
code - is that right?

I think this is a good project to maintain, but not sure it should go into
ActiveMQ itself.



--
View this message in context: 
http://activemq.2283324.n4.nabble.com/ActiveMQ-DSLs-tp4693171p4693187.html
Sent from the ActiveMQ - Dev mailing list archive at Nabble.com.


Re: ActiveMQ DSLs

2015-03-13 Thread Hadrian Zbarcea
Good job, but I don't think it's a good idea. Sure, interesting 
syntactic sugar, but what problem does it solve? Who would use it? Json 
or YAML would be way more interesting imho.


Hadrian

On 03/13/2015 12:13 PM, Jakub Korab wrote:

Hi All,

In working on a separate project, I found the need to instantiate
multiple different ActiveMQ configurations. Rather than writing up a lot
of different XML configs or messing around with assembling BrokerService
instances, I wrote a DSL that mimics the ActiveMQ config schema via a
directed builder pattern, much like Camel. Here's an example:

BrokerContext context = new
BrokerContext(ActiveMQBrokers.broker("embeddedBroker").useJmx(false).persistent(false)

 .transportConnectors()
 .transportConnector("openwire", "tcp://0.0.0.0:61616").end()
 .end()
 .plugins()
 .simpleAuthenticationPlugin()
 .users()
.authenticationUser("user").password("pass").groups("guests").end()
 .end()
 .end()
 .end()));

context.start();
...
context.end();

Unlike Camel, the DSL itself defines a model that is used to instantiate
a BrokerService. Camel has an intermediate set of VOs
(org.apache.camel.model) between the Java DSL and the implementation. I
had a go at generating a similar model via JAXB from the ActiveMQ
schema, to abstract away the DSL/XBean config, but the generated classes
were hideous and I abandoned the approach.

I also developed a testing DSL for JUnit that allows you to spin up
these configurations, and rely on the JUnit runner to start up and shut
down the broker. The test DSL also allows for the definition of proxies
(using the ActiveMQ SocketProxy class) to easily simulate network outages.

A full writeup and source code are available at

 https://github.com/jkorab/activemq-dsls

As it stands, the DSL is a work in progress that supports my own code,
but I think it would be useful as something provided by ActiveMQ itself.
I would like to contribute what I have written so far, and develop it
further (perhaps cleaning up a bunch of ActiveMQ tests to use it in an
"eating your own dogfood" way). I was initially thinking that it could
be part of activemq-broker in the src/test tree, and once it gets
fleshed out, moved to src/main. The set of config options in ActiveMQ is
huge, and retrofitting tests would be an ideal way of evolving it and
validating that everything works before making it public.

I think that this is something that would be useful, as I have seen
ActiveMQ deployments where the broker was defined in Java. This is
particularly useful where there are is a broker network with a bunch of
network connectors from a broker, each with dynamically or statically
included or excluded destinations, used in conjuction with composite
destinations to route messages in "just the right way". That sort of
thing is much better expressed in code rather than huge slabs of XML.

What do you think? Is this something that would be worthwhile to have in
ActiveMQ?

Jakub



[jira] [Resolved] (AMQ-5602) AMQP: Create simple Proton based client to be used in testing

2015-03-13 Thread Timothy Bish (JIRA)

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

Timothy Bish resolved AMQ-5602.
---
Resolution: Fixed

added new proton-j based test client on master along some initial work on 
validation tests for AMQP conformance.  

> AMQP: Create simple Proton based client to be used in testing
> -
>
> Key: AMQ-5602
> URL: https://issues.apache.org/jira/browse/AMQ-5602
> Project: ActiveMQ
>  Issue Type: Sub-task
>  Components: AMQP
>Affects Versions: 5.11.0, 5.11.1
>Reporter: Timothy Bish
>Assignee: Timothy Bish
> Fix For: 5.12.0
>
>
> The current AMQP implementation depends on testing using the prototype JMS 
> 1.0 client from QPid.  This present a few difficulties when it comes to unit 
> testing the AMQP bits in that it is quite difficult to test the AMQP 
> functionality directly as the client attempts to map AMQP to JMS but that is 
> not the only valid type of client interaction nor is it compliant with the 
> developing JMS mapping work.
> We should develop a thin proton wrapper client that lets us more easily 
> recreate various client interactions with the broker and test compliance with 
> AMQP 1.0 expected behaviors.  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: ActiveMQ DSLs

2015-03-13 Thread Jean-Baptiste Onofré

I second Tim there. Good job !

Regards
JB

On 03/13/2015 07:08 PM, Timothy Bish wrote:

On 03/13/2015 12:13 PM, Jakub Korab wrote:

Hi All,

In working on a separate project, I found the need to instantiate
multiple different ActiveMQ configurations. Rather than writing up a
lot of different XML configs or messing around with assembling
BrokerService instances, I wrote a DSL that mimics the ActiveMQ config
schema via a directed builder pattern, much like Camel. Here's an
example:

BrokerContext context = new
BrokerContext(ActiveMQBrokers.broker("embeddedBroker").useJmx(false).persistent(false)

.transportConnectors()
.transportConnector("openwire", "tcp://0.0.0.0:61616").end()
.end()
.plugins()
.simpleAuthenticationPlugin()
.users()
.authenticationUser("user").password("pass").groups("guests").end()
.end()
.end()
.end()));

context.start();
...
context.end();

Unlike Camel, the DSL itself defines a model that is used to
instantiate a BrokerService. Camel has an intermediate set of VOs
(org.apache.camel.model) between the Java DSL and the implementation.
I had a go at generating a similar model via JAXB from the ActiveMQ
schema, to abstract away the DSL/XBean config, but the generated
classes were hideous and I abandoned the approach.

I also developed a testing DSL for JUnit that allows you to spin up
these configurations, and rely on the JUnit runner to start up and
shut down the broker. The test DSL also allows for the definition of
proxies (using the ActiveMQ SocketProxy class) to easily simulate
network outages.

A full writeup and source code are available at

https://github.com/jkorab/activemq-dsls

As it stands, the DSL is a work in progress that supports my own code,
but I think it would be useful as something provided by ActiveMQ
itself. I would like to contribute what I have written so far, and
develop it further (perhaps cleaning up a bunch of ActiveMQ tests to
use it in an "eating your own dogfood" way). I was initially thinking
that it could be part of activemq-broker in the src/test tree, and
once it gets fleshed out, moved to src/main. The set of config options
in ActiveMQ is huge, and retrofitting tests would be an ideal way of
evolving it and validating that everything works before making it public.

I think that this is something that would be useful, as I have seen
ActiveMQ deployments where the broker was defined in Java. This is
particularly useful where there are is a broker network with a bunch
of network connectors from a broker, each with dynamically or
statically included or excluded destinations, used in conjuction with
composite destinations to route messages in "just the right way". That
sort of thing is much better expressed in code rather than huge slabs
of XML.

What do you think? Is this something that would be worthwhile to have
in ActiveMQ?

Jakub


Cool stuff, definitely seems like something we ought to consider adding.



--
Jean-Baptiste Onofré
jbono...@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com


Re: ActiveMQ DSLs

2015-03-13 Thread Timothy Bish

On 03/13/2015 12:13 PM, Jakub Korab wrote:

Hi All,

In working on a separate project, I found the need to instantiate 
multiple different ActiveMQ configurations. Rather than writing up a 
lot of different XML configs or messing around with assembling 
BrokerService instances, I wrote a DSL that mimics the ActiveMQ config 
schema via a directed builder pattern, much like Camel. Here's an 
example:


BrokerContext context = new 
BrokerContext(ActiveMQBrokers.broker("embeddedBroker").useJmx(false).persistent(false)

.transportConnectors()
.transportConnector("openwire", "tcp://0.0.0.0:61616").end()
.end()
.plugins()
.simpleAuthenticationPlugin()
.users()
.authenticationUser("user").password("pass").groups("guests").end()
.end()
.end()
.end()));

context.start();
...
context.end();

Unlike Camel, the DSL itself defines a model that is used to 
instantiate a BrokerService. Camel has an intermediate set of VOs 
(org.apache.camel.model) between the Java DSL and the implementation. 
I had a go at generating a similar model via JAXB from the ActiveMQ 
schema, to abstract away the DSL/XBean config, but the generated 
classes were hideous and I abandoned the approach.


I also developed a testing DSL for JUnit that allows you to spin up 
these configurations, and rely on the JUnit runner to start up and 
shut down the broker. The test DSL also allows for the definition of 
proxies (using the ActiveMQ SocketProxy class) to easily simulate 
network outages.


A full writeup and source code are available at

https://github.com/jkorab/activemq-dsls

As it stands, the DSL is a work in progress that supports my own code, 
but I think it would be useful as something provided by ActiveMQ 
itself. I would like to contribute what I have written so far, and 
develop it further (perhaps cleaning up a bunch of ActiveMQ tests to 
use it in an "eating your own dogfood" way). I was initially thinking 
that it could be part of activemq-broker in the src/test tree, and 
once it gets fleshed out, moved to src/main. The set of config options 
in ActiveMQ is huge, and retrofitting tests would be an ideal way of 
evolving it and validating that everything works before making it public.


I think that this is something that would be useful, as I have seen 
ActiveMQ deployments where the broker was defined in Java. This is 
particularly useful where there are is a broker network with a bunch 
of network connectors from a broker, each with dynamically or 
statically included or excluded destinations, used in conjuction with 
composite destinations to route messages in "just the right way". That 
sort of thing is much better expressed in code rather than huge slabs 
of XML.


What do you think? Is this something that would be worthwhile to have 
in ActiveMQ?


Jakub


Cool stuff, definitely seems like something we ought to consider adding.

--
Tim Bish
Sr Software Engineer | RedHat Inc.
tim.b...@redhat.com | www.redhat.com
skype: tabish121 | twitter: @tabish121
blog: http://timbish.blogspot.com/



[jira] [Commented] (AMQ-5026) Replicated LevelDB Store getting EOF exception

2015-03-13 Thread gbsnaker (JIRA)

[ 
https://issues.apache.org/jira/browse/AMQ-5026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14360786#comment-14360786
 ] 

gbsnaker commented on AMQ-5026:
---

I hava the same problem in my clusters. do you resove this problem?

> Replicated LevelDB Store getting EOF exception
> --
>
> Key: AMQ-5026
> URL: https://issues.apache.org/jira/browse/AMQ-5026
> Project: ActiveMQ
>  Issue Type: Bug
>  Components: activemq-leveldb-store
>Affects Versions: 5.10.0
>Reporter: kal patel
>
> 3 brokers setup for ReplicatedLevelDB master/slave conf. 
> running producer and slow consumer and doing failover by stopping master 
> broker. 
> After few failovers, the leveldb throws EOF exception while reading record 
> from the store.  
> using the latest source code from activemq 5.10 branch. 
> The file that has the issue: 
> [root@vmx0568 data]# ls -l /solidstate/ldbkp/1f4022e4.log 
> -rw-r--r--. 1 root root 91500224 Jan 17 14:44 
> /solidstate/ldbkp/1f4022e4.log 
> [root@vmx0568 data]# 
> Client | main 
> 2014-01-17 14:44:19,118 | INFO  | locator read: DataLocator(24b41aac, 2283) | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | Seq read: 260217 | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | locator read: DataLocator(24b423ee, 2283) | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | Seq read: 260218 | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | locator read: DataLocator(24b42d02, 2283) | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | Seq read: 260219 | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | locator read: DataLocator(24b43628, 2283) | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | Seq read: 260220 | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | locator read: DataLocator(24b43f6a, 2283) | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | Seq read: 260221 | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | locator read: DataLocator(24b4487e, 2283) | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | Seq read: 260222 | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,118 | INFO  | locator read: DataLocator(24b451a4, 2283) | 
> org.apache.activemq.leveldb.LevelDBClient | main 
> 2014-01-17 14:44:19,124 | INFO  | No IOExceptionHandler registered, ignoring 
> IO exception | org.apache.activemq.broker.BrokerService | LevelDB IOException 
> handler. 
> java.io.EOFException: File '/solidstate/ldbkp/1f4022e4.log' offset: 
> 91500234 
> at 
> org.apache.activemq.leveldb.RecordLog$LogReader.read(RecordLog.scala:299) 
> at 
> org.apache.activemq.leveldb.RecordLog$$anonfun$read$2.apply(RecordLog.scala:564)
>  
> at 
> org.apache.activemq.leveldb.RecordLog$$anonfun$read$2.apply(RecordLog.scala:564)
>  
> at 
> org.apache.activemq.leveldb.RecordLog.get_reader(RecordLog.scala:554) 
> at org.apache.activemq.leveldb.RecordLog.read(RecordLog.scala:564) 
> at 
> org.apache.activemq.leveldb.LevelDBClient.getMessage(LevelDBClient.scala:1318)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$$anonfun$queueCursor$1.apply(LevelDBClient.scala:1257)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$$anonfun$queueCursor$1.apply(LevelDBClient.scala:1252)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$$anonfun$collectionCursor$1$$anonfun$apply$mcV$sp$12.apply(LevelDBClient.scala:1342)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$$anonfun$collectionCursor$1$$anonfun$apply$mcV$sp$12.apply(LevelDBClient.scala:1341)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$RichDB.check$4(LevelDBClient.scala:323)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$RichDB.cursorRange(LevelDBClient.scala:325)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$$anonfun$collectionCursor$1.apply$mcV$sp(LevelDBClient.scala:1341)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$$anonfun$collectionCursor$1.apply(LevelDBClient.scala:1341)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$$anonfun$collectionCursor$1.apply(LevelDBClient.scala:1341)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient.usingIndex(LevelDBClient.scala:1019)
>  
> at 
> org.apache.activemq.leveldb.LevelDBClient$$anonfun$might_fail_using_index$1.apply(LevelDBClient.scala:1025)
>  
> at 
> org.apache.activemq.leveldb.Lev

Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Robbie Gemmell
The source archive and/or tag (depending on who you ask) is the
ultimate release at the end of the day. The binaries are only
conveniences, but as we do ship them are expected to be treated
similaly in terms of license/notice handling, and they often tend to
be treated as component parts of the release process as a result.
Certainly I would be surprisd to find anything in the releases area of
repository.apache.org that hadnt been voted on in some way, even
though only by proxy of the matching source artifact release.

The apache parent pom adds the tooling leading to creation of the
LICENCE and NOTICE files in the jar files. It has always been my
understanding they do need to be there, mainly a result of seeing
other projects pulled up on not having them.
http://apache.org/legal/src-headers.html#faq mentions:

Does the policy apply to binary/object files, such as executables or JAR files?

Yes. Even if there are no source files within the release, the LICENSE
file and NOTICE file are still both required within every ASF
distribution -- whether the unit of distribution is a .jar, .msi,
.tar/.?gz, .zip, .exe installer, or any other file format used for
distributions. For example, Windows .exe files must not be used as a
unit of distribution unless they are installers and include the
LICENSE and NOTICE files in their installation.

The interesting bit there would be deciding whether something (a .jar)
was not 'a unit of distribution' because it was included inside
something else that was (a tar.gz). Including the files seems
easier/safer :-)

Robbie

On 13 March 2015 at 17:09, Gary Tully  wrote:
> The apache release process covers the artifacts in apache/dist[1]
> The jars in maven are outside the process from what I understand and
> as such they do not need a LICENSE or NOTICE file.
>
> However, I see some value in putting them in there, but I also see a
> potential downside, because it may suggest that they are 'released'
> artifacts that have been voted on by a PMC, which will not be the
> case. It may be a case of maven tech adoption going ahead of explicit
> policy.
> What is an apache release may indeed, in the future, extend to what is
> in maven, and we will be ahead of the curve with the inclusion of
> these two files :-)
>
> [1] http://www.apache.org/dev/release.html#where-do-releases-go
>
> On 13 March 2015 at 16:15, Robbie Gemmell  wrote:
>> On 13 March 2015 at 15:59, Martyn Taylor  wrote:
>>>
>>> On 13/03/15 15:16, Robbie Gemmell wrote:

 I downloaded the src+bin tar.gz files, and gave the src build a run
 through and kicked the tyres on the binary. The NPE from RC1 when
 using an AMQP client is now fixed. In the docs the links to the
 configuration index now work, and the table there now links to the
 .html files rather than the .md files.

 For the docs, the project-info page near the start is still 404's.
>>>
>>> Thanks.


 Looking at some of the diff since RC1, I noticed that some JSON
 related source files had their 'JSON licence' restored to them. They
 should be mentioned in the distro LICENCE files as a result, either
 including the license or pointing to it in the tree.
>>>
>>> Given that the activemq-core-client jar covers these source files it it's
>>> LICENSE, am I correct in assuming there is no need to add this again in the
>>> binary distribution? i.e. apache-activemq-6.0.0-bin i.e. we should only
>>> need to add this to the LICENSE file in the source distribution?

>>
>> I think it needs to be there (anyone else know otherwise?), and I
>> would personally add it since the LICENSE file in the distribution
>> root ultimately lets people know what terms the various included bits
>> are under. The ones in the jars also need to be there because jars are
>> also distributions themselves and people can/do use them in isolation
>> (e.g from a maven repo).
>>
 The
 activemq-core-client jar has the ASLv2 included twice in its
 META-INF/LICENCE file, likely due to the appended-resources file for
 adding the licence for the JSON bits.
>>>
>>> Yes.  The build appears to append the license rather than overwrite as we
>>> had thought.


 There was a base64.txt licence file previously that went away; is that
 code no longer included?
>>>
>>> Clebert already replied to this.  But yes this is no longer included.
>>>

 Robbie

 On 13 March 2015 at 14:05, Gary Tully  wrote:
>
> Thanks for the fast turnaround.
>
> NOTICE and LICENCE files look correct in both source and binary distros.
>
> source builds, tests run, resulting binary looks great.
>
> However, I found one Cat-x transitive dep that breaks the build.
> I added an exclusion and the source builds find, so the dependency is
> not required but it needs to be removed.
>
> Detail:
> [ERROR] Failed to execute goal on project activemq-rest: Could not
> resolve dependencies for project
>

Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Gary Tully
The apache release process covers the artifacts in apache/dist[1]
The jars in maven are outside the process from what I understand and
as such they do not need a LICENSE or NOTICE file.

However, I see some value in putting them in there, but I also see a
potential downside, because it may suggest that they are 'released'
artifacts that have been voted on by a PMC, which will not be the
case. It may be a case of maven tech adoption going ahead of explicit
policy.
What is an apache release may indeed, in the future, extend to what is
in maven, and we will be ahead of the curve with the inclusion of
these two files :-)

[1] http://www.apache.org/dev/release.html#where-do-releases-go

On 13 March 2015 at 16:15, Robbie Gemmell  wrote:
> On 13 March 2015 at 15:59, Martyn Taylor  wrote:
>>
>> On 13/03/15 15:16, Robbie Gemmell wrote:
>>>
>>> I downloaded the src+bin tar.gz files, and gave the src build a run
>>> through and kicked the tyres on the binary. The NPE from RC1 when
>>> using an AMQP client is now fixed. In the docs the links to the
>>> configuration index now work, and the table there now links to the
>>> .html files rather than the .md files.
>>>
>>> For the docs, the project-info page near the start is still 404's.
>>
>> Thanks.
>>>
>>>
>>> Looking at some of the diff since RC1, I noticed that some JSON
>>> related source files had their 'JSON licence' restored to them. They
>>> should be mentioned in the distro LICENCE files as a result, either
>>> including the license or pointing to it in the tree.
>>
>> Given that the activemq-core-client jar covers these source files it it's
>> LICENSE, am I correct in assuming there is no need to add this again in the
>> binary distribution? i.e. apache-activemq-6.0.0-bin i.e. we should only
>> need to add this to the LICENSE file in the source distribution?
>>>
>
> I think it needs to be there (anyone else know otherwise?), and I
> would personally add it since the LICENSE file in the distribution
> root ultimately lets people know what terms the various included bits
> are under. The ones in the jars also need to be there because jars are
> also distributions themselves and people can/do use them in isolation
> (e.g from a maven repo).
>
>>> The
>>> activemq-core-client jar has the ASLv2 included twice in its
>>> META-INF/LICENCE file, likely due to the appended-resources file for
>>> adding the licence for the JSON bits.
>>
>> Yes.  The build appears to append the license rather than overwrite as we
>> had thought.
>>>
>>>
>>> There was a base64.txt licence file previously that went away; is that
>>> code no longer included?
>>
>> Clebert already replied to this.  But yes this is no longer included.
>>
>>>
>>> Robbie
>>>
>>> On 13 March 2015 at 14:05, Gary Tully  wrote:

 Thanks for the fast turnaround.

 NOTICE and LICENCE files look correct in both source and binary distros.

 source builds, tests run, resulting binary looks great.

 However, I found one Cat-x transitive dep that breaks the build.
 I added an exclusion and the source builds find, so the dependency is
 not required but it needs to be removed.

 Detail:
 [ERROR] Failed to execute goal on project activemq-rest: Could not
 resolve dependencies for project
 org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
 system is offline but the artifact

 org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final
 ..

 Resolution:
 
org.jboss.resteasy
resteasy-jaxrs


 org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec

   
 

 So this is another -1.

 However, with this issue fixed and no regressions - I am ready to +1
 this release.

 Shout out to other pmc members - even if you don't intend to vote, can
 you take a few moments to cast your eyes over the artifacts.
 As this is a first release, we need many eyes to validate there are no
 glaring omissions.



 On 12 March 2015 at 21:30, Martyn Taylor  wrote:
>
> Hello all.
>
> I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing
> the
> initial RC feedback from community members.
>
> This is a first release of the HornetQ code donation with support for
> AMQP,
> STOMP, CORE and OPENWIRE
>
>
> The release notes can be found here:
>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953
>
> The binary distributions can be found here:
>
> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>
> The source archives can be found here:
>
> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>
> The Maven repository is here:
>
> https://re

Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Martyn Taylor


On 13/03/15 16:15, Robbie Gemmell wrote:

On 13 March 2015 at 15:59, Martyn Taylor  wrote:

On 13/03/15 15:16, Robbie Gemmell wrote:

I downloaded the src+bin tar.gz files, and gave the src build a run
through and kicked the tyres on the binary. The NPE from RC1 when
using an AMQP client is now fixed. In the docs the links to the
configuration index now work, and the table there now links to the
.html files rather than the .md files.

For the docs, the project-info page near the start is still 404's.

Thanks.


Looking at some of the diff since RC1, I noticed that some JSON
related source files had their 'JSON licence' restored to them. They
should be mentioned in the distro LICENCE files as a result, either
including the license or pointing to it in the tree.

Given that the activemq-core-client jar covers these source files it it's
LICENSE, am I correct in assuming there is no need to add this again in the
binary distribution? i.e. apache-activemq-6.0.0-bin i.e. we should only
need to add this to the LICENSE file in the source distribution?

I think it needs to be there (anyone else know otherwise?), and I
would personally add it since the LICENSE file in the distribution
root ultimately lets people know what terms the various included bits
are under. The ones in the jars also need to be there because jars are
also distributions themselves and people can/do use them in isolation
(e.g from a maven repo).
Thanks Robbie.  We'll include them in the source and binary distribution 
(as well as the jars).



The
activemq-core-client jar has the ASLv2 included twice in its
META-INF/LICENCE file, likely due to the appended-resources file for
adding the licence for the JSON bits.

Yes.  The build appears to append the license rather than overwrite as we
had thought.


There was a base64.txt licence file previously that went away; is that
code no longer included?

Clebert already replied to this.  But yes this is no longer included.


Robbie

On 13 March 2015 at 14:05, Gary Tully  wrote:

Thanks for the fast turnaround.

NOTICE and LICENCE files look correct in both source and binary distros.

source builds, tests run, resulting binary looks great.

However, I found one Cat-x transitive dep that breaks the build.
I added an exclusion and the source builds find, so the dependency is
not required but it needs to be removed.

Detail:
[ERROR] Failed to execute goal on project activemq-rest: Could not
resolve dependencies for project
org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
system is offline but the artifact

org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final
..

Resolution:

org.jboss.resteasy
resteasy-jaxrs


org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec

   


So this is another -1.

However, with this issue fixed and no regressions - I am ready to +1
this release.

Shout out to other pmc members - even if you don't intend to vote, can
you take a few moments to cast your eyes over the artifacts.
As this is a first release, we need many eyes to validate there are no
glaring omissions.



On 12 March 2015 at 21:30, Martyn Taylor  wrote:

Hello all.

I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing
the
initial RC feedback from community members.

This is a first release of the HornetQ code donation with support for
AMQP,
STOMP, CORE and OPENWIRE


The release notes can be found here:

https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953

The binary distributions can be found here:

https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/

The source archives can be found here:

https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/

The Maven repository is here:

https://repository.apache.org/content/repositories/orgapacheactivemq-1037/

The source tag:

https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
[ ] +0 no opinion
[ ] -1 disapprove (and reason why)

Here's my (non-binding) +1

Regards

Martyn






Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Robbie Gemmell
On 13 March 2015 at 15:59, Martyn Taylor  wrote:
>
> On 13/03/15 15:16, Robbie Gemmell wrote:
>>
>> I downloaded the src+bin tar.gz files, and gave the src build a run
>> through and kicked the tyres on the binary. The NPE from RC1 when
>> using an AMQP client is now fixed. In the docs the links to the
>> configuration index now work, and the table there now links to the
>> .html files rather than the .md files.
>>
>> For the docs, the project-info page near the start is still 404's.
>
> Thanks.
>>
>>
>> Looking at some of the diff since RC1, I noticed that some JSON
>> related source files had their 'JSON licence' restored to them. They
>> should be mentioned in the distro LICENCE files as a result, either
>> including the license or pointing to it in the tree.
>
> Given that the activemq-core-client jar covers these source files it it's
> LICENSE, am I correct in assuming there is no need to add this again in the
> binary distribution? i.e. apache-activemq-6.0.0-bin i.e. we should only
> need to add this to the LICENSE file in the source distribution?
>>

I think it needs to be there (anyone else know otherwise?), and I
would personally add it since the LICENSE file in the distribution
root ultimately lets people know what terms the various included bits
are under. The ones in the jars also need to be there because jars are
also distributions themselves and people can/do use them in isolation
(e.g from a maven repo).

>> The
>> activemq-core-client jar has the ASLv2 included twice in its
>> META-INF/LICENCE file, likely due to the appended-resources file for
>> adding the licence for the JSON bits.
>
> Yes.  The build appears to append the license rather than overwrite as we
> had thought.
>>
>>
>> There was a base64.txt licence file previously that went away; is that
>> code no longer included?
>
> Clebert already replied to this.  But yes this is no longer included.
>
>>
>> Robbie
>>
>> On 13 March 2015 at 14:05, Gary Tully  wrote:
>>>
>>> Thanks for the fast turnaround.
>>>
>>> NOTICE and LICENCE files look correct in both source and binary distros.
>>>
>>> source builds, tests run, resulting binary looks great.
>>>
>>> However, I found one Cat-x transitive dep that breaks the build.
>>> I added an exclusion and the source builds find, so the dependency is
>>> not required but it needs to be removed.
>>>
>>> Detail:
>>> [ERROR] Failed to execute goal on project activemq-rest: Could not
>>> resolve dependencies for project
>>> org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
>>> system is offline but the artifact
>>>
>>> org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final
>>> ..
>>>
>>> Resolution:
>>> 
>>>org.jboss.resteasy
>>>resteasy-jaxrs
>>>
>>>
>>> org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec
>>>
>>>   
>>> 
>>>
>>> So this is another -1.
>>>
>>> However, with this issue fixed and no regressions - I am ready to +1
>>> this release.
>>>
>>> Shout out to other pmc members - even if you don't intend to vote, can
>>> you take a few moments to cast your eyes over the artifacts.
>>> As this is a first release, we need many eyes to validate there are no
>>> glaring omissions.
>>>
>>>
>>>
>>> On 12 March 2015 at 21:30, Martyn Taylor  wrote:

 Hello all.

 I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing
 the
 initial RC feedback from community members.

 This is a first release of the HornetQ code donation with support for
 AMQP,
 STOMP, CORE and OPENWIRE


 The release notes can be found here:

 https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953

 The binary distributions can be found here:

 https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/

 The source archives can be found here:

 https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/

 The Maven repository is here:

 https://repository.apache.org/content/repositories/orgapacheactivemq-1037/

 The source tag:

 https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
 [ ] +0 no opinion
 [ ] -1 disapprove (and reason why)

 Here's my (non-binding) +1

 Regards

 Martyn
>
>


Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Martyn Taylor


On 13/03/15 15:59, Martyn Taylor wrote:


On 13/03/15 15:16, Robbie Gemmell wrote:

I downloaded the src+bin tar.gz files, and gave the src build a run
through and kicked the tyres on the binary. The NPE from RC1 when
using an AMQP client is now fixed. In the docs the links to the
configuration index now work, and the table there now links to the
.html files rather than the .md files.

For the docs, the project-info page near the start is still 404's.

Thanks.


Looking at some of the diff since RC1, I noticed that some JSON
related source files had their 'JSON licence' restored to them. They
should be mentioned in the distro LICENCE files as a result, either
including the license or pointing to it in the tree.
Given that the activemq-core-client jar covers these source files it 
it's LICENSE, am I correct in assuming there is no need to add this 
again in the binary distribution? i.e. apache-activemq-6.0.0-bin i.e. 
we should only  need to add this to the LICENSE file in the source 
distribution?
Nevermind... reading the documentation, it is implied that the license 
will be needed in both places: 
http://www.apache.org/dev/licensing-howto.html#alv2-dep.  We'll update.  
Thanks.

The
activemq-core-client jar has the ASLv2 included twice in its
META-INF/LICENCE file, likely due to the appended-resources file for
adding the licence for the JSON bits.
Yes.  The build appears to append the license rather than overwrite as 
we had thought.


There was a base64.txt licence file previously that went away; is that
code no longer included?

Clebert already replied to this.  But yes this is no longer included.


Robbie

On 13 March 2015 at 14:05, Gary Tully  wrote:

Thanks for the fast turnaround.

NOTICE and LICENCE files look correct in both source and binary 
distros.


source builds, tests run, resulting binary looks great.

However, I found one Cat-x transitive dep that breaks the build.
I added an exclusion and the source builds find, so the dependency is
not required but it needs to be removed.

Detail:
[ERROR] Failed to execute goal on project activemq-rest: Could not
resolve dependencies for project
org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
system is offline but the artifact
org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final 


..

Resolution:

   org.jboss.resteasy
   resteasy-jaxrs
   
org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec
   
  


So this is another -1.

However, with this issue fixed and no regressions - I am ready to +1
this release.

Shout out to other pmc members - even if you don't intend to vote, can
you take a few moments to cast your eyes over the artifacts.
As this is a first release, we need many eyes to validate there are no
glaring omissions.



On 12 March 2015 at 21:30, Martyn Taylor  wrote:

Hello all.

I've cut a second release candidate of Apache ActiveMQ 6.0.0 
addressing the

initial RC feedback from community members.

This is a first release of the HornetQ code donation with support 
for AMQP,

STOMP, CORE and OPENWIRE


The release notes can be found here:
https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953 



The binary distributions can be found here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/ 



The source archives can be found here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/ 



The Maven repository is here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/ 



The source tag:
https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
[ ] +0 no opinion
[ ] -1 disapprove (and reason why)

Here's my (non-binding) +1

Regards

Martyn






ActiveMQ DSLs

2015-03-13 Thread Jakub Korab

Hi All,

In working on a separate project, I found the need to instantiate 
multiple different ActiveMQ configurations. Rather than writing up a lot 
of different XML configs or messing around with assembling BrokerService 
instances, I wrote a DSL that mimics the ActiveMQ config schema via a 
directed builder pattern, much like Camel. Here's an example:


BrokerContext context = new 
BrokerContext(ActiveMQBrokers.broker("embeddedBroker").useJmx(false).persistent(false)

.transportConnectors()
.transportConnector("openwire", "tcp://0.0.0.0:61616").end()
.end()
.plugins()
.simpleAuthenticationPlugin()
.users()
.authenticationUser("user").password("pass").groups("guests").end()
.end()
.end()
.end()));

context.start();
...
context.end();

Unlike Camel, the DSL itself defines a model that is used to instantiate 
a BrokerService. Camel has an intermediate set of VOs 
(org.apache.camel.model) between the Java DSL and the implementation. I 
had a go at generating a similar model via JAXB from the ActiveMQ 
schema, to abstract away the DSL/XBean config, but the generated classes 
were hideous and I abandoned the approach.


I also developed a testing DSL for JUnit that allows you to spin up 
these configurations, and rely on the JUnit runner to start up and shut 
down the broker. The test DSL also allows for the definition of proxies 
(using the ActiveMQ SocketProxy class) to easily simulate network outages.


A full writeup and source code are available at

https://github.com/jkorab/activemq-dsls

As it stands, the DSL is a work in progress that supports my own code, 
but I think it would be useful as something provided by ActiveMQ itself. 
I would like to contribute what I have written so far, and develop it 
further (perhaps cleaning up a bunch of ActiveMQ tests to use it in an 
"eating your own dogfood" way). I was initially thinking that it could 
be part of activemq-broker in the src/test tree, and once it gets 
fleshed out, moved to src/main. The set of config options in ActiveMQ is 
huge, and retrofitting tests would be an ideal way of evolving it and 
validating that everything works before making it public.


I think that this is something that would be useful, as I have seen 
ActiveMQ deployments where the broker was defined in Java. This is 
particularly useful where there are is a broker network with a bunch of 
network connectors from a broker, each with dynamically or statically 
included or excluded destinations, used in conjuction with composite 
destinations to route messages in "just the right way". That sort of 
thing is much better expressed in code rather than huge slabs of XML.


What do you think? Is this something that would be worthwhile to have in 
ActiveMQ?


Jakub


Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Robbie Gemmell
On 13 March 2015 at 15:30, Clebert Suconic  wrote:
>> There was a base64.txt licence file previously that went away; is that
> code no longer included?
>
>
> We are using the Base64 from here:
> http://iharder.sourceforge.net/current/java/base64/
>
>
> Where he states you can license it anyway you wanted, for that we
> removed the base64 license.
>
> Do you see any issues with it?
>

He says you dont need to worry about matching it up to another
license, just use it as its public domain, but that he will license it
to you under any reasonable license if contacted to ask for that. He
does say do anything you want with it, which possibly includes simply
using it with a different licence by yourself but I'm not sure on that
at all, IANAL. Asking on legal@ (or seeing if somoene else previosuly
did) might be the way to go.

I would personally just call out the that files were released into
public domain, as per
http://www.apache.org/legal/resolved.html#can-works-placed-in-the-public-domain-be-included-in-apache-products.

>
> Regarding the docs:
>
> I guess the website doc wasn't updated with the release, only on the
> distribution. We will update it before the final upload though.
>

It was the distribution that I tried, I'll admit I hadn't even got as
far down the email to notice the link to the website thus far :)

> I'm doing some work on that part now to convert everything to
> Markdown. So even the PDF format will have the links fixed.
>
>
>
> We will take a look at the licenses.. thanks again!
>
>
>
>
> On Fri, Mar 13, 2015 at 11:16 AM, Robbie Gemmell
>  wrote:
>> I downloaded the src+bin tar.gz files, and gave the src build a run
>> through and kicked the tyres on the binary. The NPE from RC1 when
>> using an AMQP client is now fixed. In the docs the links to the
>> configuration index now work, and the table there now links to the
>> .html files rather than the .md files.
>>
>> For the docs, the project-info page near the start is still 404's.
>>
>> Looking at some of the diff since RC1, I noticed that some JSON
>> related source files had their 'JSON licence' restored to them. They
>> should be mentioned in the distro LICENCE files as a result, either
>> including the license or pointing to it in the tree. The
>> activemq-core-client jar has the ASLv2 included twice in its
>> META-INF/LICENCE file, likely due to the appended-resources file for
>> adding the licence for the JSON bits.
>>
>> There was a base64.txt licence file previously that went away; is that
>> code no longer included?
>>
>> Robbie
>>
>> On 13 March 2015 at 14:05, Gary Tully  wrote:
>>> Thanks for the fast turnaround.
>>>
>>> NOTICE and LICENCE files look correct in both source and binary distros.
>>>
>>> source builds, tests run, resulting binary looks great.
>>>
>>> However, I found one Cat-x transitive dep that breaks the build.
>>> I added an exclusion and the source builds find, so the dependency is
>>> not required but it needs to be removed.
>>>
>>> Detail:
>>> [ERROR] Failed to execute goal on project activemq-rest: Could not
>>> resolve dependencies for project
>>> org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
>>> system is offline but the artifact
>>> org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final
>>> ..
>>>
>>> Resolution:
>>> 
>>>   org.jboss.resteasy
>>>   resteasy-jaxrs
>>>   
>>>
>>> org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec
>>>   
>>>  
>>> 
>>>
>>> So this is another -1.
>>>
>>> However, with this issue fixed and no regressions - I am ready to +1
>>> this release.
>>>
>>> Shout out to other pmc members - even if you don't intend to vote, can
>>> you take a few moments to cast your eyes over the artifacts.
>>> As this is a first release, we need many eyes to validate there are no
>>> glaring omissions.
>>>
>>>
>>>
>>> On 12 March 2015 at 21:30, Martyn Taylor  wrote:
 Hello all.

 I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing the
 initial RC feedback from community members.

 This is a first release of the HornetQ code donation with support for AMQP,
 STOMP, CORE and OPENWIRE


 The release notes can be found here:
 https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953

 The binary distributions can be found here:
 https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/

 The source archives can be found here:
 https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/

 The Maven repository is here:
 https://repository.apache.org/content/repositories/orgapacheactivemq-1037/

 The source tag:
 https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.0

 The project website for that version has been staged to:
 http://people.apache.org/~martynt

Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Martyn Taylor


On 13/03/15 15:16, Robbie Gemmell wrote:

I downloaded the src+bin tar.gz files, and gave the src build a run
through and kicked the tyres on the binary. The NPE from RC1 when
using an AMQP client is now fixed. In the docs the links to the
configuration index now work, and the table there now links to the
.html files rather than the .md files.

For the docs, the project-info page near the start is still 404's.

Thanks.


Looking at some of the diff since RC1, I noticed that some JSON
related source files had their 'JSON licence' restored to them. They
should be mentioned in the distro LICENCE files as a result, either
including the license or pointing to it in the tree.
Given that the activemq-core-client jar covers these source files it 
it's LICENSE, am I correct in assuming there is no need to add this 
again in the binary distribution? i.e. apache-activemq-6.0.0-bin i.e. we 
should only  need to add this to the LICENSE file in the source 
distribution?

The
activemq-core-client jar has the ASLv2 included twice in its
META-INF/LICENCE file, likely due to the appended-resources file for
adding the licence for the JSON bits.
Yes.  The build appears to append the license rather than overwrite as 
we had thought.


There was a base64.txt licence file previously that went away; is that
code no longer included?

Clebert already replied to this.  But yes this is no longer included.


Robbie

On 13 March 2015 at 14:05, Gary Tully  wrote:

Thanks for the fast turnaround.

NOTICE and LICENCE files look correct in both source and binary distros.

source builds, tests run, resulting binary looks great.

However, I found one Cat-x transitive dep that breaks the build.
I added an exclusion and the source builds find, so the dependency is
not required but it needs to be removed.

Detail:
[ERROR] Failed to execute goal on project activemq-rest: Could not
resolve dependencies for project
org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
system is offline but the artifact
org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final
..

Resolution:

   org.jboss.resteasy
   resteasy-jaxrs
   

org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec
   
  


So this is another -1.

However, with this issue fixed and no regressions - I am ready to +1
this release.

Shout out to other pmc members - even if you don't intend to vote, can
you take a few moments to cast your eyes over the artifacts.
As this is a first release, we need many eyes to validate there are no
glaring omissions.



On 12 March 2015 at 21:30, Martyn Taylor  wrote:

Hello all.

I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing the
initial RC feedback from community members.

This is a first release of the HornetQ code donation with support for AMQP,
STOMP, CORE and OPENWIRE


The release notes can be found here:
https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953

The binary distributions can be found here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/

The source archives can be found here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/

The Maven repository is here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/

The source tag:
https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
[ ] +0 no opinion
[ ] -1 disapprove (and reason why)

Here's my (non-binding) +1

Regards

Martyn




Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Clebert Suconic
> There was a base64.txt licence file previously that went away; is that
code no longer included?


We are using the Base64 from here:
http://iharder.sourceforge.net/current/java/base64/


Where he states you can license it anyway you wanted, for that we
removed the base64 license.

Do you see any issues with it?


Regarding the docs:

I guess the website doc wasn't updated with the release, only on the
distribution. We will update it before the final upload though.

I'm doing some work on that part now to convert everything to
Markdown. So even the PDF format will have the links fixed.



We will take a look at the licenses.. thanks again!




On Fri, Mar 13, 2015 at 11:16 AM, Robbie Gemmell
 wrote:
> I downloaded the src+bin tar.gz files, and gave the src build a run
> through and kicked the tyres on the binary. The NPE from RC1 when
> using an AMQP client is now fixed. In the docs the links to the
> configuration index now work, and the table there now links to the
> .html files rather than the .md files.
>
> For the docs, the project-info page near the start is still 404's.
>
> Looking at some of the diff since RC1, I noticed that some JSON
> related source files had their 'JSON licence' restored to them. They
> should be mentioned in the distro LICENCE files as a result, either
> including the license or pointing to it in the tree. The
> activemq-core-client jar has the ASLv2 included twice in its
> META-INF/LICENCE file, likely due to the appended-resources file for
> adding the licence for the JSON bits.
>
> There was a base64.txt licence file previously that went away; is that
> code no longer included?
>
> Robbie
>
> On 13 March 2015 at 14:05, Gary Tully  wrote:
>> Thanks for the fast turnaround.
>>
>> NOTICE and LICENCE files look correct in both source and binary distros.
>>
>> source builds, tests run, resulting binary looks great.
>>
>> However, I found one Cat-x transitive dep that breaks the build.
>> I added an exclusion and the source builds find, so the dependency is
>> not required but it needs to be removed.
>>
>> Detail:
>> [ERROR] Failed to execute goal on project activemq-rest: Could not
>> resolve dependencies for project
>> org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
>> system is offline but the artifact
>> org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final
>> ..
>>
>> Resolution:
>> 
>>   org.jboss.resteasy
>>   resteasy-jaxrs
>>   
>>
>> org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec
>>   
>>  
>> 
>>
>> So this is another -1.
>>
>> However, with this issue fixed and no regressions - I am ready to +1
>> this release.
>>
>> Shout out to other pmc members - even if you don't intend to vote, can
>> you take a few moments to cast your eyes over the artifacts.
>> As this is a first release, we need many eyes to validate there are no
>> glaring omissions.
>>
>>
>>
>> On 12 March 2015 at 21:30, Martyn Taylor  wrote:
>>> Hello all.
>>>
>>> I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing the
>>> initial RC feedback from community members.
>>>
>>> This is a first release of the HornetQ code donation with support for AMQP,
>>> STOMP, CORE and OPENWIRE
>>>
>>>
>>> The release notes can be found here:
>>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953
>>>
>>> The binary distributions can be found here:
>>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>>>
>>> The source archives can be found here:
>>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>>>
>>> The Maven repository is here:
>>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/
>>>
>>> The source tag:
>>> https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
>>> [ ] +0 no opinion
>>> [ ] -1 disapprove (and reason why)
>>>
>>> Here's my (non-binding) +1
>>>
>>> Regards
>>>
>>> Martyn



-- 
Clebert Suconic
http://community.jboss.org/people/clebert.suco...@jboss.com
http://clebertsuconic.blogspot.com


Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Robbie Gemmell
I downloaded the src+bin tar.gz files, and gave the src build a run
through and kicked the tyres on the binary. The NPE from RC1 when
using an AMQP client is now fixed. In the docs the links to the
configuration index now work, and the table there now links to the
.html files rather than the .md files.

For the docs, the project-info page near the start is still 404's.

Looking at some of the diff since RC1, I noticed that some JSON
related source files had their 'JSON licence' restored to them. They
should be mentioned in the distro LICENCE files as a result, either
including the license or pointing to it in the tree. The
activemq-core-client jar has the ASLv2 included twice in its
META-INF/LICENCE file, likely due to the appended-resources file for
adding the licence for the JSON bits.

There was a base64.txt licence file previously that went away; is that
code no longer included?

Robbie

On 13 March 2015 at 14:05, Gary Tully  wrote:
> Thanks for the fast turnaround.
>
> NOTICE and LICENCE files look correct in both source and binary distros.
>
> source builds, tests run, resulting binary looks great.
>
> However, I found one Cat-x transitive dep that breaks the build.
> I added an exclusion and the source builds find, so the dependency is
> not required but it needs to be removed.
>
> Detail:
> [ERROR] Failed to execute goal on project activemq-rest: Could not
> resolve dependencies for project
> org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
> system is offline but the artifact
> org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final
> ..
>
> Resolution:
> 
>   org.jboss.resteasy
>   resteasy-jaxrs
>   
>
> org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec
>   
>  
> 
>
> So this is another -1.
>
> However, with this issue fixed and no regressions - I am ready to +1
> this release.
>
> Shout out to other pmc members - even if you don't intend to vote, can
> you take a few moments to cast your eyes over the artifacts.
> As this is a first release, we need many eyes to validate there are no
> glaring omissions.
>
>
>
> On 12 March 2015 at 21:30, Martyn Taylor  wrote:
>> Hello all.
>>
>> I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing the
>> initial RC feedback from community members.
>>
>> This is a first release of the HornetQ code donation with support for AMQP,
>> STOMP, CORE and OPENWIRE
>>
>>
>> The release notes can be found here:
>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953
>>
>> The binary distributions can be found here:
>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>>
>> The source archives can be found here:
>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>>
>> The Maven repository is here:
>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/
>>
>> The source tag:
>> https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
>> [ ] +0 no opinion
>> [ ] -1 disapprove (and reason why)
>>
>> Here's my (non-binding) +1
>>
>> Regards
>>
>> Martyn


[GitHub] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread mastanik
Github user mastanik commented on the pull request:

https://github.com/apache/activemq-6/pull/182#issuecomment-79029355
  
Sure, I will update it as well


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: [CANCEL][VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Clebert Suconic
Even though we are canceling the vote we would appreciate feedback
from other PMCs as Gary Mentioned.

On Fri, Mar 13, 2015 at 10:35 AM, Martyn Taylor  wrote:
> Cancelling vote.  Another RC will follow addressing the Cat X issue as
> described by Gary Tully.
>
> On 12/03/15 21:30, Martyn Taylor wrote:
>>
>> Hello all.
>>
>> I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing
>> the initial RC feedback from community members.
>>
>> This is a first release of the HornetQ code donation with support for
>> AMQP, STOMP, CORE and OPENWIRE
>>
>>
>> The release notes can be found here:
>>
>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953
>>
>> The binary distributions can be found here:
>>
>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>>
>> The source archives can be found here:
>>
>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>>
>> The Maven repository is here:
>> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/
>>
>> The source tag:
>>
>> https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
>> [ ] +0 no opinion
>> [ ] -1 disapprove (and reason why)
>>
>> Here's my (non-binding) +1
>>
>> Regards
>>
>> Martyn
>
>



-- 
Clebert Suconic
http://community.jboss.org/people/clebert.suco...@jboss.com
http://clebertsuconic.blogspot.com


[GitHub] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread clebertsuconic
Github user clebertsuconic commented on the pull request:

https://github.com/apache/activemq-6/pull/182#issuecomment-79020971
  
If you have a chance also, add a paragraph to the docs :) I will do it 
right away if you don't have time to do it.. but please advise if that's the 
case so I will do the docs.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread clebertsuconic
Github user clebertsuconic commented on the pull request:

https://github.com/apache/activemq-6/pull/182#issuecomment-79020069
  
I will hold the PR open until you do it then


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[CANCEL][VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Martyn Taylor
Cancelling vote.  Another RC will follow addressing the Cat X issue as 
described by Gary Tully.


On 12/03/15 21:30, Martyn Taylor wrote:

Hello all.

I've cut a second release candidate of Apache ActiveMQ 6.0.0 
addressing the initial RC feedback from community members.


This is a first release of the HornetQ code donation with support for 
AMQP, STOMP, CORE and OPENWIRE



The release notes can be found here:
https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953 



The binary distributions can be found here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/ 



The source archives can be found here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/ 



The Maven repository is here:
https://repository.apache.org/content/repositories/orgapacheactivemq-1037/ 



The source tag:
https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
[ ] +0 no opinion
[ ] -1 disapprove (and reason why)

Here's my (non-binding) +1

Regards

Martyn




[GitHub] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread mastanik
Github user mastanik commented on the pull request:

https://github.com/apache/activemq-6/pull/182#issuecomment-78994222
  
Yes, sure, I will update this pr when it is done


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: [VOTE] Apache ActiveMQ 6.0.0 (RC2)

2015-03-13 Thread Gary Tully
Thanks for the fast turnaround.

NOTICE and LICENCE files look correct in both source and binary distros.

source builds, tests run, resulting binary looks great.

However, I found one Cat-x transitive dep that breaks the build.
I added an exclusion and the source builds find, so the dependency is
not required but it needs to be removed.

Detail:
[ERROR] Failed to execute goal on project activemq-rest: Could not
resolve dependencies for project
org.apache.activemq.rest:activemq-rest:jar:6.0.0: The repository
system is offline but the artifact
org.jboss.spec.javax.annotation:jboss-annotations-api_1.1_spec:jar:1.0.1.Final
..

Resolution:

  org.jboss.resteasy
  resteasy-jaxrs
  
   
org.jboss.spec.javax.annotationjboss-annotations-api_1.1_spec
  
 


So this is another -1.

However, with this issue fixed and no regressions - I am ready to +1
this release.

Shout out to other pmc members - even if you don't intend to vote, can
you take a few moments to cast your eyes over the artifacts.
As this is a first release, we need many eyes to validate there are no
glaring omissions.



On 12 March 2015 at 21:30, Martyn Taylor  wrote:
> Hello all.
>
> I've cut a second release candidate of Apache ActiveMQ 6.0.0 addressing the
> initial RC feedback from community members.
>
> This is a first release of the HornetQ code donation with support for AMQP,
> STOMP, CORE and OPENWIRE
>
>
> The release notes can be found here:
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315920&version=12328953
>
> The binary distributions can be found here:
> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>
> The source archives can be found here:
> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/org/apache/activemq/apache-activemq/6.0.0/
>
> The Maven repository is here:
> https://repository.apache.org/content/repositories/orgapacheactivemq-1037/
>
> The source tag:
> https://git-wip-us.apache.org/repos/asf?p=activemq-6.git;a=tag;h=refs/tags/6.0.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 ActiveMQ 6.0.0
> [ ] +0 no opinion
> [ ] -1 disapprove (and reason why)
>
> Here's my (non-binding) +1
>
> Regards
>
> Martyn


[GitHub] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread clebertsuconic
Github user clebertsuconic commented on the pull request:

https://github.com/apache/activemq-6/pull/182#issuecomment-78967583
  
Nicely done... +1 on me for a merge


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Updated] (AMQ-5659) Add safety measure against infinite loop when store exception prevents message removal

2015-03-13 Thread metatech (JIRA)

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

metatech updated AMQ-5659:
--
Attachment: purge_queue_abort_loop_v3.patch

V3 of the patch fixes the incorrect threshold

> Add safety measure against infinite loop when store exception prevents 
> message removal
> --
>
> Key: AMQ-5659
> URL: https://issues.apache.org/jira/browse/AMQ-5659
> Project: ActiveMQ
>  Issue Type: Improvement
>  Components: Broker
>Affects Versions: 5.7.0
> Environment: ServiceMix 4.5.3
>Reporter: metatech
> Attachments: purge_queue_abort_loop.patch, 
> purge_queue_abort_loop_v2.patch, purge_queue_abort_loop_v3.patch
>
>
> When the broker is configured with a database store, the "purge" operation 
> enters an infinite loop when the message removal operation fails, for 
> instance when the broker datasource is being restarted (see example stack 
> trace below). 
> Here is a patch which adds a safety measure, in case the "dequeue" count of 
> the queue does not increase between 2 messages removal operations.  The check 
> is not garanteed to detect the problem on the next iteration, because a 
> business consumer might also be dequeuing messages from the queue.  But the 
> "purge" is probably much faster than the business consumer, so if it fails to 
> remove 2 messages in a row, it is enough to detect the problem and abort the 
> infinite loop.
> {code}
> 2015-03-05 15:38:30,353 | WARN  | 14571659-2202099 |  | 
> JDBCPersistenceAdapter   | Could not get JDBC connection: Data source 
> is closed
> java.sql.SQLException: Data source is closed
>   at 
> org.apache.commons.dbcp.BasicDataSource.createDataSource(BasicDataSource.java:1362)
>   at 
> org.apache.commons.dbcp.BasicDataSource.getConnection(BasicDataSource.java:1044)
>   at 
> org.apache.activemq.store.jdbc.TransactionContext.getConnection(TransactionContext.java:58)
>   at 
> org.apache.activemq.store.jdbc.adapter.DefaultJDBCAdapter.getStoreSequenceId(DefaultJDBCAdapter.java:285)
>   at 
> org.apache.activemq.store.jdbc.JDBCPersistenceAdapter.getStoreSequenceIdForMessageId(JDBCPersistenceAdapter.java:787)
>   at 
> org.apache.activemq.store.jdbc.JDBCMessageStore.removeMessage(JDBCMessageStore.java:194)
>   at 
> org.apache.activemq.store.memory.MemoryTransactionStore.removeMessage(MemoryTransactionStore.java:358)
>   at 
> org.apache.activemq.store.memory.MemoryTransactionStore$1.removeAsyncMessage(MemoryTransactionStore.java:166)
>   at org.apache.activemq.broker.region.Queue.acknowledge(Queue.java:846)
>   at 
> org.apache.activemq.broker.region.Queue.removeMessage(Queue.java:1602)
>   at 
> org.apache.activemq.broker.region.Queue.removeMessage(Queue.java:1594)
>   at 
> org.apache.activemq.broker.region.Queue.removeMessage(Queue.java:1579)
>   at org.apache.activemq.broker.region.Queue.purge(Queue.java:1158)
>   at org.apache.activemq.broker.jmx.QueueView.purge(QueueView.java:54)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread HornetQBot
Github user HornetQBot commented on the pull request:

https://github.com/apache/activemq-6/pull/182#issuecomment-78891438
  
Can one of the admins verify this patch by saying "ok to test"?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] activemq-6 pull request: Added possibility to intercept stomp fram...

2015-03-13 Thread mastanik
GitHub user mastanik opened a pull request:

https://github.com/apache/activemq-6/pull/182

Added possibility to intercept stomp frames

Initially I was trying to log all the incoming stomp frames from the client 
on a server side, but could not find any easy way of doing this. This pr adds 
possibility to intercept stomp frames via interceptor section in 
activemq-configuration.xml. 

Interceptor list was already passed as a parameter to StompProtocolManager 
constructor, but was not used in any way. I decided to introduce new abstract 
class which implements Interceptor interface as the simplest solution and then 
invoke its intercept method like it is done with the regular interceptors. 

Clients need to extend this class and put it under activemq class path, 
just as a regular jms interceptor.

Looking forward to your comments, and feel free to reject it in case it 
does not make any sense.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mastanik/activemq-6 master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-6/pull/182.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #182


commit 05b469589dc7fe5b4f9e58654437e3501b6bc2f9
Author: nberdikov 
Date:   2015-03-13T09:29:33Z

Added possibility to intercept stomp frames




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] activemq-6 pull request:

2015-03-13 Thread mtaylor
Github user mtaylor commented on the pull request:


https://github.com/apache/activemq-6/commit/1e9ac1bbc34a3ee028806d35b4c05f09b256e81f#commitcomment-10181666
  
In docs/user-manual/en/configuration-index.md:
In docs/user-manual/en/configuration-index.md on line 48:
@clebertsuconic Actually you're right.  The HTML elements are legitimate in 
markdown syntax, I presume this was left over from before we did the 
conversion.  I'll follow up with a fix to use proper md links.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] activemq-6 pull request:

2015-03-13 Thread mtaylor
Github user mtaylor commented on the pull request:


https://github.com/apache/activemq-6/commit/1e9ac1bbc34a3ee028806d35b4c05f09b256e81f#commitcomment-10181597
  
In docs/user-manual/en/configuration-index.md:
In docs/user-manual/en/configuration-index.md on line 48:
@clebertsuconic This file should be fully markdown.  We need to go through 
the docs and ensure that they have all been converted from html to markdown.  
This fixes the broken links in html for the time being.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Commented] (AMQ-5659) Add safety measure against infinite loop when store exception prevents message removal

2015-03-13 Thread metatech (JIRA)

[ 
https://issues.apache.org/jira/browse/AMQ-5659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14360133#comment-14360133
 ] 

metatech commented on AMQ-5659:
---

Gary, breaking out of the "for" loop is not enough. We really want to break the 
2 nested loops + skip the reset of the queue count performed at the end of the 
method.

> Add safety measure against infinite loop when store exception prevents 
> message removal
> --
>
> Key: AMQ-5659
> URL: https://issues.apache.org/jira/browse/AMQ-5659
> Project: ActiveMQ
>  Issue Type: Improvement
>  Components: Broker
>Affects Versions: 5.7.0
> Environment: ServiceMix 4.5.3
>Reporter: metatech
> Attachments: purge_queue_abort_loop.patch, 
> purge_queue_abort_loop_v2.patch
>
>
> When the broker is configured with a database store, the "purge" operation 
> enters an infinite loop when the message removal operation fails, for 
> instance when the broker datasource is being restarted (see example stack 
> trace below). 
> Here is a patch which adds a safety measure, in case the "dequeue" count of 
> the queue does not increase between 2 messages removal operations.  The check 
> is not garanteed to detect the problem on the next iteration, because a 
> business consumer might also be dequeuing messages from the queue.  But the 
> "purge" is probably much faster than the business consumer, so if it fails to 
> remove 2 messages in a row, it is enough to detect the problem and abort the 
> infinite loop.
> {code}
> 2015-03-05 15:38:30,353 | WARN  | 14571659-2202099 |  | 
> JDBCPersistenceAdapter   | Could not get JDBC connection: Data source 
> is closed
> java.sql.SQLException: Data source is closed
>   at 
> org.apache.commons.dbcp.BasicDataSource.createDataSource(BasicDataSource.java:1362)
>   at 
> org.apache.commons.dbcp.BasicDataSource.getConnection(BasicDataSource.java:1044)
>   at 
> org.apache.activemq.store.jdbc.TransactionContext.getConnection(TransactionContext.java:58)
>   at 
> org.apache.activemq.store.jdbc.adapter.DefaultJDBCAdapter.getStoreSequenceId(DefaultJDBCAdapter.java:285)
>   at 
> org.apache.activemq.store.jdbc.JDBCPersistenceAdapter.getStoreSequenceIdForMessageId(JDBCPersistenceAdapter.java:787)
>   at 
> org.apache.activemq.store.jdbc.JDBCMessageStore.removeMessage(JDBCMessageStore.java:194)
>   at 
> org.apache.activemq.store.memory.MemoryTransactionStore.removeMessage(MemoryTransactionStore.java:358)
>   at 
> org.apache.activemq.store.memory.MemoryTransactionStore$1.removeAsyncMessage(MemoryTransactionStore.java:166)
>   at org.apache.activemq.broker.region.Queue.acknowledge(Queue.java:846)
>   at 
> org.apache.activemq.broker.region.Queue.removeMessage(Queue.java:1602)
>   at 
> org.apache.activemq.broker.region.Queue.removeMessage(Queue.java:1594)
>   at 
> org.apache.activemq.broker.region.Queue.removeMessage(Queue.java:1579)
>   at org.apache.activemq.broker.region.Queue.purge(Queue.java:1158)
>   at org.apache.activemq.broker.jmx.QueueView.purge(QueueView.java:54)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)