You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by mtaylor <gi...@git.apache.org> on 2016/02/01 12:04:07 UTC

[GitHub] activemq-artemis pull request: Fix Delete TX on JDBCJournalImpl

GitHub user mtaylor opened a pull request:

    https://github.com/apache/activemq-artemis/pull/364

    Fix Delete TX on JDBCJournalImpl

    The JDBC in memory Tx representation was using the Tx ID instead of the
    Record ID which was causing deletes to fail.  Also the Store line up
    should be set to true as default.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mtaylor/activemq-artemis FixDeleteTxRecords

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/364.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 #364
    
----
commit d4e809bf4b1b07b899e62a2f3a42beb0112b85a7
Author: Martyn Taylor <mt...@redhat.com>
Date:   2016-02-01T11:00:23Z

    Fix Delete TX on JDBCJournalImpl
    
    The JDBC in memory Tx representation was using the Tx ID instead of the
    Record ID which was causing deletes to fail.  Also the Store line up
    should be set to true as default.

----


---
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: Fix Delete TX on JDBCJournalImpl

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/364


---
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: Fix Delete TX on JDBCJournalImpl

Posted by clebertsuconic <gi...@git.apache.org>.
Github user clebertsuconic commented on the pull request:

    https://github.com/apache/activemq-artemis/pull/364#issuecomment-178008882
  
    Can you rebase it please to be merged on top please?


---
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.
---