Mailing-List: contact dev-help@impala.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@impala.incubator.apache.org
Message-Id: <201605130003.u4D03NlU007547@ip-10-146-233-104.ec2.internal>
Date: Fri, 13 May 2016 00:03:23 +0000
From: "Skye Wanderman-Milne (Code Review)" <gerrit@cloudera.org>
To: impala-cr@cloudera.com, dev@impala.incubator.apache.org
CC: Dan Hecht <dhecht@cloudera.com>, Tim Armstrong <tarmstrong@cloudera.com>
Reply-To: skye@cloudera.com
Subject: =?UTF-8?Q?[Impala-CR](cdh5-trunk)_IMPALA-3311:_fix_string_data_coming_out_of_aggs_in_subplans=0A?=
In-Reply-To: <gerrit.1462223464000.Iada891504c261ba54f4eb8c9d7e4e5223668d7b9@gerrit.cloudera.org>
References: <gerrit.1462223464000.Iada891504c261ba54f4eb8c9d7e4e5223668d7b9@gerrit.cloudera.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Content-Disposition: inline
User-Agent: Gerrit/2.10-rc0
archived-at: Fri, 13 May 2016 00:03:30 -0000

Skye Wanderman-Milne has posted comments on this change.

Change subject: IMPALA-3311: fix string data coming out of aggs in subplans
......................................................................


Patch Set 6:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/2929/6/be/src/exec/partitioned-aggregation-node.cc
File be/src/exec/partitioned-aggregation-node.cc:

Line 375:   // FunctionContextImpl::AllocateLocal(), and are ultimately backed by 'agg_fn_pool_'.)
> the comment for agg_fn_ctxs_ says the allocations come from here only in th
Done


Line 380: NYI
> Not yet implemented implies that we should implement it, so is this really 
Producing collections is NYI, so this shouldn't be implemented until we do the rest of returning collections. I'll change it to "producing collections NYI"


http://gerrit.cloudera.org:8080/#/c/2929/6/be/src/exec/partitioned-aggregation-node.h
File be/src/exec/partitioned-aggregation-node.h:

Line 397: won't be cleared
> this is a bit ambiguous, eventually it will get cleared. Maybe: has a lifet
Done


-- 
To view, visit http://gerrit.cloudera.org:8080/2929
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iada891504c261ba54f4eb8c9d7e4e5223668d7b9
Gerrit-PatchSet: 6
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Skye Wanderman-Milne <skye@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhecht@cloudera.com>
Gerrit-Reviewer: Skye Wanderman-Milne <skye@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-HasComments: Yes