You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@impala.apache.org by "Bharath Vissapragada (Code Review)" <ge...@cloudera.org> on 2018/12/12 18:03:22 UTC

[Impala-ASF-CR] IMPALA-7867 (Part 2): ArrayList cleanup in analyzer

Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/11995 )

Change subject: IMPALA-7867 (Part 2): ArrayList cleanup in analyzer
......................................................................


Patch Set 3: Code-Review+1

Fredy, could you +2 when you get a chance? 

Paul, this probably needs a rebase.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7c0b5f40a0504fc2d324055bd2a962e35f8e744a
Gerrit-Change-Number: 11995
Gerrit-PatchSet: 3
Gerrit-Owner: Paul Rogers <pa...@yahoo.com>
Gerrit-Reviewer: Bharath Vissapragada <bh...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Paul Rogers <pa...@yahoo.com>
Gerrit-Comment-Date: Wed, 12 Dec 2018 18:03:22 +0000
Gerrit-HasComments: No