Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-4179

Remove RowBatch::MarkNeedsDeepCopy() memory management API

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: Impala 2.8.0
    • Fix Version/s: None
    • Component/s: Backend

      Description

      We should remove this API as a simplification of the memory transfer model.

      It was used in various places to manage lifetime of BufferHandles, but those can be replaced with attaching the BufferHandle + MarkFlushResources().

      It is also used to work around the fact that some memory is never returned from ExecNodes and is freed in Close(). Part of the solution is probably to add a way to attach all resources to a RowBatch before Close() and during Reset().

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                tarmstrong Tim Armstrong
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated: