Uploaded image for project: 'Tephra'
  1. Tephra
  2. TEPHRA-152

TransactionStateCache refresh thread does not get stopped

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.4.0, 0.5.0, 0.6.0, 0.6.1, 0.6.2, 0.6.3
    • Fix Version/s: 0.12.0-incubating
    • Component/s: core
    • Labels:
      None

      Description

      TransactionStateCache starts a thread to refresh transaction state periodically. This thread runs in co-processor TransactionProcessor. When co-processor is shutdown down, this thread needs to be stopped for proper clean up of resources.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/incubator-tephra/pull/41

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/incubator-tephra/pull/41
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user poornachandra commented on the issue:

          https://github.com/apache/incubator-tephra/pull/41

          @gokulavasan Now that we have introduced class `ReferenceCountedSupplier`, it would be good to use this class in `PruneUpperBoundWriterSupplier` too. It will reduce code duplication.

          Show
          githubbot ASF GitHub Bot added a comment - Github user poornachandra commented on the issue: https://github.com/apache/incubator-tephra/pull/41 @gokulavasan Now that we have introduced class `ReferenceCountedSupplier`, it would be good to use this class in `PruneUpperBoundWriterSupplier` too. It will reduce code duplication.
          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user gokulavasan opened a pull request:

          https://github.com/apache/incubator-tephra/pull/41

          TEPHRA-152 Using ReferenceCounting for TransactionStateCache refresh …

          …thread, so that it can be stopped

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

          $ git pull https://github.com/gokulavasan/incubator-tephra feature/tephra-152-stop-thread

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

          https://github.com/apache/incubator-tephra/pull/41.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 #41


          commit c5d3378ad7cb4aab70e215e521ee1148f19bcde0
          Author: Gokul Gunasekaran <gokul@cask.co>
          Date: 2017-03-21T22:37:48Z

          TEPHRA-152 Using ReferenceCounting for TransactionStateCache refresh thread, so that it can be stopped


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user gokulavasan opened a pull request: https://github.com/apache/incubator-tephra/pull/41 TEPHRA-152 Using ReferenceCounting for TransactionStateCache refresh … …thread, so that it can be stopped You can merge this pull request into a Git repository by running: $ git pull https://github.com/gokulavasan/incubator-tephra feature/tephra-152-stop-thread Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-tephra/pull/41.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 #41 commit c5d3378ad7cb4aab70e215e521ee1148f19bcde0 Author: Gokul Gunasekaran <gokul@cask.co> Date: 2017-03-21T22:37:48Z TEPHRA-152 Using ReferenceCounting for TransactionStateCache refresh thread, so that it can be stopped

            People

            • Assignee:
              gokulavasan Gokul Gunasekaran
              Reporter:
              poornachandra Poorna Chandra
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development