Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-5557

Correctly handle interruptions of Thread.sleep()

Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Low
    • Resolution: Fixed
    • 2.0 beta 1
    • None
    • None

    Description

      There is no need to throw assertion errors or runtime exceptions when Thread.sleep() is interrupted - code can just ignore interruption (and that is correct behaviour). Also this eliminates boilerplate code. See attached patch.

      Attachments

        1. trunk-5557-v3.txt
          41 kB
          Mikhail Mazursky
        2. trunk-5557-v2.txt
          38 kB
          Mikhail Mazursky
        3. trunk-5557.txt
          36 kB
          Mikhail Mazursky

        Issue Links

        relates to

        Improvement - An improvement or enhancement to an existing feature or task. CASSANDRA-5623 Cleanup interruption handling

        • Low -
        • Resolved

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ash2k Mikhail Mazursky Assign to me
            ash2k Mikhail Mazursky
            Mikhail Mazursky
            Jonathan Ellis
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment