[GitHub] activemq-artemis pull request #2063: [ART] Fix opening SSL connection in IBM...

2018-05-03 Thread rstancel
GitHub user rstancel opened a pull request: https://github.com/apache/activemq-artemis/pull/2063 [ART] Fix opening SSL connection in IBM JDK in tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/rstancel/jboss-activemq

[GitHub] activemq-artemis pull request #1990: [ARTEMIS-1770] Log warning instead of s...

2018-04-05 Thread rstancel
Github user rstancel commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1990#discussion_r179361706 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientSessionImpl.java --- @@ -1539,6 +1540,8

[GitHub] activemq-artemis pull request #1995: [ARTEMIS-1770] Log warning when connect...

2018-04-05 Thread rstancel
GitHub user rstancel opened a pull request: https://github.com/apache/activemq-artemis/pull/1995 [ARTEMIS-1770] Log warning when connection is closed Jira: https://issues.apache.org/jira/browse/ARTEMIS-1770 Master PR: https://github.com/apache/activemq-artemis/pull/1990

[GitHub] activemq-artemis pull request #1975: [ARTEMIS-1761] Log warning if cluster i...

2018-04-09 Thread rstancel
Github user rstancel commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1975#discussion_r180048729 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/impl/BridgeImpl.java --- @@ -930,9 +930,7 @@ protected

[GitHub] activemq-artemis issue #1964: [ARTEMIS-1761] Log one warning if cluster coul...

2018-03-26 Thread rstancel
Github user rstancel commented on the issue: https://github.com/apache/activemq-artemis/pull/1964 @michaelandrepearce Done. Can you please check? Thanks ---

[GitHub] activemq-artemis issue #1975: [ARTEMIS-1761] Log warning if cluster is not a...

2018-03-26 Thread rstancel
Github user rstancel commented on the issue: https://github.com/apache/activemq-artemis/pull/1975 @clebertsuconic Clebert, can you please merge also this PR? thanks ---

[GitHub] activemq-artemis pull request #1975: [ARTEMIS-1761] Log warning if cluster i...

2018-03-26 Thread rstancel
GitHub user rstancel opened a pull request: https://github.com/apache/activemq-artemis/pull/1975 [ARTEMIS-1761] Log warning if cluster is not able to connect at startup Jira: https://issues.apache.org/jira/browse/ARTEMIS-1761 master commit: https://github.com/apache/activemq

[GitHub] activemq-artemis issue #1964: [ARTEMIS-1761] Log one warning if cluster coul...

2018-03-26 Thread rstancel
Github user rstancel commented on the issue: https://github.com/apache/activemq-artemis/pull/1964 @clebertsuconic yeap, I will do that once it will be approved ---

[GitHub] activemq-artemis pull request #1990: [ARTEMIS-1770] Log warning instead of s...

2018-04-03 Thread rstancel
Github user rstancel commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1990#discussion_r178928320 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientSessionImpl.java --- @@ -29,12 +29,7

[GitHub] activemq-artemis issue #1990: [ARTEMIS-1770] Log warning instead of stacktra...

2018-04-03 Thread rstancel
Github user rstancel commented on the issue: https://github.com/apache/activemq-artemis/pull/1990 While clear shutdown it makes no sence to throw any exception. It should be sufficient just to log message about whats happening. ---

[GitHub] activemq-artemis pull request #1990: [ARTEMIS-1770] Log warning instead of s...

2018-04-03 Thread rstancel
GitHub user rstancel opened a pull request: https://github.com/apache/activemq-artemis/pull/1990 [ARTEMIS-1770] Log warning instead of stacktrace while shutdown Jira: https://issues.apache.org/jira/browse/ARTEMIS-1770 You can merge this pull request into a Git repository by running

[GitHub] activemq-artemis issue #1990: [ARTEMIS-1770] Log warning instead of stacktra...

2018-04-04 Thread rstancel
Github user rstancel commented on the issue: https://github.com/apache/activemq-artemis/pull/1990 Can someone please run retest? I have the same issue on Travis as this PR: https://github.com/apache/activemq-artemis/pull/1985 and I think it has nothing to do with my PR ---

[GitHub] activemq-artemis pull request #1964: [ARTEMIS-1761] Log one warning if clust...

2018-03-21 Thread rstancel
GitHub user rstancel opened a pull request: https://github.com/apache/activemq-artemis/pull/1964 [ARTEMIS-1761] Log one warning if cluster could not be formed because… … it's not possible to connect to other cluster node Jira: https://issues.apache.org/jira/browse

[GitHub] activemq-artemis pull request #1964: [ARTEMIS-1761] Log one warning if clust...

2018-03-23 Thread rstancel
Github user rstancel commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1964#discussion_r176724023 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/impl/BridgeImpl.java --- @@ -927,9 +927,7 @@ protected

[GitHub] activemq-artemis pull request #1975: [ARTEMIS-1761] Log warning if cluster i...

2018-12-17 Thread rstancel
Github user rstancel closed the pull request at: https://github.com/apache/activemq-artemis/pull/1975 ---

[GitHub] activemq-artemis issue #1975: [ARTEMIS-1761] Log warning if cluster is not a...

2018-12-17 Thread rstancel
Github user rstancel commented on the issue: https://github.com/apache/activemq-artemis/pull/1975 Closing this PR as requested by Justin Bertram @jbertram ---