impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dan Hecht (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-5788: Fix agg node crash when grouping by nondeterministic exprs
Date Tue, 22 Aug 2017 21:21:25 GMT
Dan Hecht has posted comments on this change.

Change subject: IMPALA-5788: Fix agg node crash when grouping by nondeterministic exprs

Patch Set 3:

File be/src/exec/

Line 1160:   // partition index.
> Just to add to this, we would also have to make the same effort for hash jo
i agree it's best to not adding a check to the fast path. We can go forward with this fix,
but please do add that comment referencing this jira to the header file.

Why isn't this a problem in the join?  Is it because we repartition into the stream first,
and then build hashtables over a single stream in the join case? whereas in agg we're repartitioning
and building the table in the same pass?

Ultimately I think we should eventually reject these queries during analysis (e.g. IMPALA-4605),
but that's obviously out of scope for now.
File be/src/exec/partitioned-aggregation-node.h:

PS3, Line 338: all pointers in this vector will point to a single
             :   /// in-memory partition
> You are absolutely right. This is very specific to our bug fix. would it ma
I think adding a reference to the JIRA makes sense in this case since it's so specific to

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibdb09239577b3f0a19d710b0d148e882b0b73e23
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Bikramjeet Vig <>
Gerrit-Reviewer: Bikramjeet Vig <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Tim Armstrong <>
Gerrit-HasComments: Yes

View raw message