Uploaded image for project: 'Apache Drill'
  1. Apache Drill
  2. DRILL-304

Implement destroy() method for HyperBatches

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Task
    • Status: Resolved
    • Major
    • Resolution: Invalid
    • None
    • 0.4.0
    • None
    • None

    Description

      Ideally, classes like AbstractSingleRecordBatch() should call container.clear() to decrement the reference count of the incoming ValueVectors before calling next() on the upstream operator. There is an issue with HyperBatches in this case though; the underlying ValueVectors should not be freed implicitly. That is, the operator should only clear the vectors when the entire HyperBatch has been consumed.

      It has been suggested that a destroy() method be used to handle the HyperBatch case.

      Attachments

        Activity

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

          People

            Unassigned Unassigned
            benjamin.becker Ben Becker
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment