Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/2103
---
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/2101
---
Github user franz1981 closed the pull request at:
https://github.com/apache/activemq-artemis/pull/2097
---
It does not. This is the classic ActiveMQ 5.x range that just had release.
ActiveMq Artemis has its own versioning currently 2.6.0 just released this week
also.
You can find artemis latest and historic versions here:
https://activemq.apache.org/artemis/download.html
Sent from my Samsung Galaxy
GitHub user JiriOndrusek opened a pull request:
https://github.com/apache/activemq-artemis/pull/2099
[ARTEMIS-1879] Warnings in log when failover is executed after clean â¦
â¦shut down
Issue: https://issues.apache.org/jira/browse/ARTEMIS-1879
Test is not
GitHub user michaelandrepearce opened a pull request:
https://github.com/apache/activemq-artemis/pull/2105
ARTEMIS-1872 Fixing address security checks
Ensure CREATE_ADDRESS is honored and behavior is consistent across
protocols.
You can merge this pull request into a Git
GitHub user clebertsuconic opened a pull request:
https://github.com/apache/activemq-artemis/pull/2104
ARTEMIS-1872 Fixing Security test
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/clebertsuconic/activemq-artemis
GitHub user thisthat opened a pull request:
https://github.com/apache/activemq/pull/284
Avoid overflow errors with timestamps
Some comparisons with timestamp values are not safe. This comparisons can
trigger errors that were found in other Apache projects, e.g. KAFKA-4290.
I
Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2103#discussion_r190438892
--- Diff:
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/server/SecureConfigurationTest.java
---
Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2103#discussion_r190437791
--- Diff:
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/server/SecureConfigurationTest.java
---
Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2103#discussion_r190437930
--- Diff: tests/integration-tests/src/test/resources/multicast_topic.xml ---
@@ -85,13 +85,13 @@ under the License.
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2102#discussion_r190442460
--- Diff:
tests/compatibility-tests/src/main/resources/servers/artemisServer.groovy ---
@@ -44,7 +44,7 @@ try {
if
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/2104
---
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2093
@michaelandrepearce Maybe I'm being slow.. but why do you need to check for
version here?
shouldn't this be a simple fix on checking the address before?
from
GitHub user clebertsuconic opened a pull request:
https://github.com/apache/activemq-artemis/pull/2100
Amending fixes from #2093
Amending fixes from #2093
You can merge this pull request into a Git repository by running:
$ git pull
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2093
that would mean older clients will still hit the bug if you keep the
version check
---
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2093
compatibility tests are passing with this.. do you know any cases where
this would brea compatibility and the current tests have missed it? if that's
the case we need to add a test
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2093
the more I digg.. the more I'm convinced there's no need for the version
check... if we can speak on IRC before we merge this?
The version check was basically for wire
GitHub user gaohoward opened a pull request:
https://github.com/apache/activemq-artemis/pull/2101
ARTEMIS-1853 Adding Netty OpenSSL provider test
Added an example to demonstrate how to configure and use openssl
Moved/Added netty-tcnative dependency to artemis-distribution
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
Could you send PR to my branch so i can see what youve changed from my
intial solution i think we may jusy have cross wires but i cant figure the
change out. Im out today and
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
Thanks for sending pr to my private fork, looks fine and just cross wires /
my misunderstanding of what you meant. Im happy,
---
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
So as i said im away from a computer so i cant test it but as long as the
test passes i added im happy.
---
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
@michaelandrepearce I was just trying to understand if you really needed
the semantic difference on the client.. but I couldn't find a reason.. if you
are happy on just reverting
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2093
Ill close this if you merge yours, im happy with yours.
---
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
Eg +1 Im happy with what you have i think its all fine
---
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
@michaelandrepearce this is removing the version bumping check. I don't
think we need it.
If we merge this PR, #2093 would be closed automatically by the apache bot.
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
Version check was because after the version its fixed client side.
---
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
Re test its to run the test i added with old client. I did this manually, i
couldnt figure how automatically
---
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
The only change I saw on the client was a queueQuery done first... the
server's change would still be compatible with older clients from what I saw.
If we really need a
GitHub user jbertram opened a pull request:
https://github.com/apache/activemq-artemis/pull/2102
ARTEMIS-1882 verify PKCS12 keystores work
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-1882
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2093
I am merging your changes.. however I'm reverting the portion of the
version properties check. we may revisit this later if you see any other issues.
To make eventual
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/2100
#2093 is merged. .this is no longer needed.
---
Github user clebertsuconic closed the pull request at:
https://github.com/apache/activemq-artemis/pull/2100
---
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/2093
---
GitHub user clebertsuconic opened a pull request:
https://github.com/apache/activemq-artemis/pull/2103
ARTEMIS-1872 Fixing some extra bits on AMQP + improving the test
You can merge this pull request into a Git repository by running:
$ git pull
35 matches
Mail list logo