You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by TomRoss <gi...@git.apache.org> on 2015/12/07 19:02:42 UTC
[GitHub] activemq-artemis pull request: NullPointerException when trying to...
GitHub user TomRoss opened a pull request:
https://github.com/apache/activemq-artemis/pull/257
NullPointerException when trying to list prepared transactions as JSON
https://issues.apache.org/jira/browse/ARTEMIS-315
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/TomRoss/activemq-artemis master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/257.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #257
----
commit bc23eebb5ca9c6aa9314793908c45d335e93b2c0
Author: Tom Ross <to...@redhat.com>
Date: 2015-12-07T17:57:43Z
NullPointerException when trying to list prepared transactions as JSON
----
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] activemq-artemis pull request: Make CLI print data more friendly
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/257#discussion_r30243500
--- Diff: artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/tools/DataAbstract.java ---
@@ -72,9 +78,19 @@ public String getPaging() throws Exception
if (paging == null)
{
paging = getFileConfiguration().getPagingDirectory();
+ checkIfDirectoryExists(binding);
--- End diff --
Unless you change the server's behaviour to always create the paging directory, even before paging. it should be an easy change
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] activemq-artemis pull request: Make CLI print data more friendly
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/257#discussion_r30243429
--- Diff: artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/tools/DataAbstract.java ---
@@ -72,9 +78,19 @@ public String getPaging() throws Exception
if (paging == null)
{
paging = getFileConfiguration().getPagingDirectory();
+ checkIfDirectoryExists(binding);
--- End diff --
I would just remove the check on paging
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] activemq-artemis pull request: Make CLI print data more friendly
Posted by jbertram <gi...@git.apache.org>.
Github user jbertram closed the pull request at:
https://github.com/apache/activemq-artemis/pull/257
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] activemq-artemis pull request: NullPointerException when trying to...
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-artemis/pull/257#issuecomment-162621763
@TomRoss NM I have ammended your commit, merged it and fixed 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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] activemq-artemis pull request: NullPointerException when trying to...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/257
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] activemq-artemis pull request: NullPointerException when trying to...
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the pull request:
https://github.com/apache/activemq-artemis/pull/257#issuecomment-162617890
@TomRoss checkstyle issues: use -Pdev when building to verify
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] activemq-artemis pull request: Make CLI print data more friendly
Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/257#discussion_r30243293
--- Diff: artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/tools/DataAbstract.java ---
@@ -72,9 +78,19 @@ public String getPaging() throws Exception
if (paging == null)
{
paging = getFileConfiguration().getPagingDirectory();
+ checkIfDirectoryExists(binding);
--- End diff --
typo here...
Besides the paging might not exist for a running server if it didn't page yet
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---