[IMPALA-5611] KuduPartitionExpr holds onto memory unnecessarily

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: Impala 2.9.0
• Fix Version/s: Impala 2.10.0
• Component/s: Backend
• Epic Color: ghx-label-9
• Assignee: Thomas Tauber-Marshall
• Reporter: Thomas Tauber-Marshall
• Created: 30/Jun/17 18:41
• Updated: 06/Jul/17 21:50
• Resolved: 06/Jul/17 21:50

Description

IMPALA-3742 introduced KuduPartitionExpr, which takes a row and passes it to the Kudu client to determine what partition it belongs to.

KuduPartitionExpr never calls ScalarExprEvaluator::FreeLocalAllocations, causing it to hang on to memory longer than it needs it.

Since we only need the value of the row for the call into the Kudu client, we can call FreeLocalAllocations after that.

Issue Links

• is broken by: IMPALA-3742 INSERTs into Kudu tables should partition and sort