[GitHub] activemq-artemis issue #1439: ARTEMIS-1317 Expire messages that got expiredA...

2017-08-03 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1439 @jdanekrh that's totally fine.. I just wanted to know if I should run the tests before merging.. will merge it now then.. thanks --- If your project is set up for it, you can reply

[GitHub] activemq-artemis issue #1439: ARTEMIS-1317 Expire messages that got expiredA...

2017-08-03 Thread jdanekrh
Github user jdanekrh commented on the issue: https://github.com/apache/activemq-artemis/pull/1439 Yes, I run `mvn -Ptests -Pextra-tests -Popenwire-tests test` which includes integration-tests and activemq5-openwire tests you mentioned I should run. The fails are either fails t

[GitHub] activemq-artemis issue #1439: ARTEMIS-1317 Expire messages that got expiredA...

2017-08-03 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1439 @jdanekrh did you run the testsuite? --- 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 th