Hi,
I have deployed a live - backup cluster. sending/reception all working when
clients are connected to same udp://231.7.7.7:9876. Failover and all are
working. My concern is that what if another cluster, created by some one
else, also get connection to the same group ip:port
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/672
I will run tests and merge this tomorrow.
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/695
---
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
GitHub user clebertsuconic opened a pull request:
https://github.com/apache/activemq-artemis/pull/695
improving fallocate check on libaio
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/clebertsuconic/activemq-artemis libaio
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/694
Merit here to @johnament. I only pushed my version as it would be easier
than ask him to change his. since I wanted to run my tests on some of my
servers.
---
If your project is
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/691
@johnament thanks for pointing this out. I only pushed it because you
pointed this was not needed.
---
If your project is set up for it, you can reply to this email and have your
Github user johnament closed the pull request at:
https://github.com/apache/activemq-artemis/pull/691
---
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
Github user johnament commented on the issue:
https://github.com/apache/activemq-artemis/pull/691
Closing out since Clebert pushed his own version.
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/694
---
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
Github user johnament commented on the issue:
https://github.com/apache/activemq-artemis/pull/694
If that comes in, I'll bring in a separate dependency from geronimo.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/694
@johnament tests are good here. can we merge this? or you have other plans
to this dependency on your next work on CDI?
---
If your project is set up for it, you can reply to this
Github user clebertsuconic commented on the issue:
https://github.com/apache/activemq-artemis/pull/691
I just opened #694. And I'm running a few tests on it. It seems good so far.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user clebertsuconic opened a pull request:
https://github.com/apache/activemq-artemis/pull/694
ARTEMIS-659 Remove unneeded CDI dependency
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/clebertsuconic/activemq-artemis cdi
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/691#discussion_r73035839
--- Diff: pom.xml ---
@@ -349,16 +349,9 @@
-javax.enterprise
-cdi-api
Github user johnament commented on the issue:
https://github.com/apache/activemq-artemis/pull/672
Looks good to me.
---
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
Github user johnament commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/691#discussion_r73035248
--- Diff: pom.xml ---
@@ -349,16 +349,9 @@
-javax.enterprise
-cdi-api
-
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/691#discussion_r73031975
--- Diff: pom.xml ---
@@ -349,16 +349,9 @@
-javax.enterprise
-cdi-api
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/691#discussion_r73031897
--- Diff: pom.xml ---
@@ -349,16 +349,9 @@
-javax.enterprise
-cdi-api
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/693
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/activemq/pull/193
---
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
GitHub user clebertsuconic opened a pull request:
https://github.com/apache/activemq-artemis/pull/693
ARTEMIS-565 another JSON fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/clebertsuconic/activemq-artemis json
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/692
---
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
GitHub user jbertram opened a pull request:
https://github.com/apache/activemq-artemis/pull/692
JSON fixes
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jbertram/activemq-artemis JSONfixes
Alternatively you can review and
Hi guys,
I’m heading out for vacation in an hour, so I won’t be able to take a look
at it immediately. I’ll dig into it as soon as I’m back. But, yeah this is
a completely new integration, so it won’t affect any of the existing stuff.
Regards
--
Dejan Bosanac
http://sensatic.net/about
On Mon,
Github user bayern39 commented on the issue:
https://github.com/apache/activemq-artemis/pull/672
@clebertsuconic I have done it !
---
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
That test looks like its focused on Karaf (OSGi) integration. Definitely
something that should be there. There's one guy I know who does OSGi and
CDI really well, but the market just isn't there. Both leverage JSR-330
(AtInject) but ultimately are two different service models.
This being its
Looks really nice.. Very well done.
There is a test Dejan wrote for CDI called
org.apache.activemq.artemis.tests.integration.kara.ArtemisFeatureTest
Can you verify the test still works?
And how this would affect that Karaf integration?
@Dejan can you take a look given you did a lot of work
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/672#discussion_r72982711
--- Diff:
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/RemotingConnectionImpl.java
---
@@ -387,4
Github user bayern39 commented on the issue:
https://github.com/apache/activemq-artemis/pull/672
@johnament OK,I have done it!
---
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
Github user johnament commented on the issue:
https://github.com/apache/activemq-artemis/pull/672
@clebertsuconic think we can add the version aware stuff to the hacking
guide? Would be useful for those not very familiar to know how to deal with it.
---
If your project is set up for
Github user johnament commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/672#discussion_r72964880
--- Diff:
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/RemotingConnectionImpl.java
---
@@ -387,4
Github user johnament commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/672#discussion_r72964784
--- Diff:
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/ActiveMQSessionContext.java
---
@@ -162,6
Github user bayern39 commented on the issue:
https://github.com/apache/activemq-artemis/pull/672
@johnament @clebertsuconic Hi,I have fixed the mistake!Thanks!
---
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
33 matches
Mail list logo