[GitHub] activemq-artemis pull request #551: some minimal trace logs

2016-06-01 Thread clebertsuconic
GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/551 some minimal trace logs You can merge this pull request into a Git repository by running: $ git pull https://github.com/clebertsuconic/activemq-artemis traces Alternatively y

[GitHub] activemq-artemis pull request #550: Jdbc refactor post

2016-06-01 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/550#discussion_r65450813 --- Diff: artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java --- @@ -0,0 +1,129 @@

[GitHub] activemq-artemis pull request #550: Jdbc refactor post

2016-06-01 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/550#discussion_r65450589 --- Diff: artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/drivers/AbstractJDBCDriver.java --- @@ -0,0 +1,129 @@

[GitHub] activemq-artemis pull request #550: Jdbc refactor post

2016-06-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/550 --- 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 featu

[GitHub] activemq-artemis issue #550: Jdbc refactor post

2016-06-01 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/550 I will merge it.. but we should refactor logging / tracing. --- 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 pr

[GitHub] activemq-artemis issue #550: Jdbc refactor post

2016-06-01 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/550 I don't know in which commit it was changed. but JDBCJournalImpl has a Logger which is not final. it should be the first line. we are using the loggers just for trace and deb

[GitHub] activemq-artemis pull request #550: Jdbc refactor post

2016-06-01 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/550#discussion_r65445835 --- Diff: artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/file/JDBCSequentialFile.java --- @@ -75,33 +57,30 @@

[GitHub] activemq-artemis pull request #550: Jdbc refactor post

2016-06-01 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/550#discussion_r65445360 --- Diff: artemis-jdbc-store/src/main/java/org/apache/activemq/artemis/jdbc/store/file/JDBCSequentialFile.java --- @@ -75,33 +57,30 @@

[GitHub] activemq-artemis pull request #550: Jdbc refactor post

2016-06-01 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/550#discussion_r65409448 --- Diff: artemis-jdbc-store/pom.xml --- @@ -63,6 +63,13 @@ org.postgresql postgresql + pr

[GitHub] activemq-artemis issue #548: Executor in JournalImpl for append operations

2016-06-01 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/548 lets defer merging this until the release is done. quite a big change now at the last minute.. better to wait next release. --- If your project is set up for it, you can rep

[GitHub] activemq-artemis pull request #:

2016-06-01 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/commit/927edb76f323885a7855a7c6fddc1e557a004023#commitcomment-17699805 Yes, it should. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] activemq-artemis pull request #:

2016-06-01 Thread arunpselvam
Github user arunpselvam commented on the pull request: https://github.com/apache/activemq-artemis/commit/927edb76f323885a7855a7c6fddc1e557a004023#commitcomment-17699665 Shouldn't the shebang be fixed in " artemis-cli/src/main/resources/org/apache/activemq/artemis/cli/commands/bin/arte

REST API with .NET WCF - POST error

2016-06-01 Thread RDESCHAMPS
We are implementing an integration from our middleware (Biztalk) to active MQ using the WCF-HTTP adapter. from biztalk we are sending messages to a queue using the MQ REST API by doing HTTP-POST. The message makes it to the queue, but in biztalk we are getting following error on the response side

[GitHub] activemq-artemis pull request: Jdbc refactor post

2016-06-01 Thread mtaylor
GitHub user mtaylor opened a pull request: https://github.com/apache/activemq-artemis/pull/550 Jdbc refactor post Refactor the JDBC storage layer. Adds Postgres and MySQL drivers. You can merge this pull request into a Git repository by running: $ git pull https://github.com/m