Uploaded image for project: 'Pig'
  1. Pig
  2. PIG-4260

SpillableMemoryManager.spill should revert spill on all exception

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.17.0, 0.16.1
    • impl
    • None
    • Reviewed

    Description

      Found by Rohini when working on PIG-4250.

      If there is a exception during spill() called by SpillableMemoryManager it will be just ignored. We do not track that there was an exception during spill and throw that back when the bag is accessed next time

      Attachments

        1. PIG-4260-3-nowhitespacechanges.patch
          8 kB
          Rohini Palaniswamy
        2. PIG-4260-3.patch
          32 kB
          Rohini Palaniswamy
        3. PIG-4260-2-nowhitespacechanges.patch
          8 kB
          Rohini Palaniswamy
        4. PIG-4260-2.patch
          32 kB
          Rohini Palaniswamy
        5. PIG-4260-1.patch
          6 kB
          Rohini Palaniswamy

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

          People

            rohini Rohini Palaniswamy
            daijy Daniel Dai
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment