You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Michael Smith (Code Review)" <ge...@cloudera.org> on 2022/08/03 16:13:14 UTC

[Impala-ASF-CR] IMPALA-9442: Add Ozone to minicluster

Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/18738 )

Change subject: IMPALA-9442: Add Ozone to minicluster
......................................................................


Patch Set 15:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/18738/14/be/src/scheduling/executor-group.cc
File be/src/scheduling/executor-group.cc:

http://gerrit.cloudera.org:8080/#/c/18738/14/be/src/scheduling/executor-group.cc@103
PS14, Line 103:   if (be_descs_it == executor_map_.end()) {
> Oh actually, the warning comes from fixing IMPALA-11457, so I guess I need 
Done


http://gerrit.cloudera.org:8080/#/c/18738/12/bin/impala-config.sh
File bin/impala-config.sh:

http://gerrit.cloudera.org:8080/#/c/18738/12/bin/impala-config.sh@311
PS12, Line 311: # Allow overriding of Hive source location in case we want to build Impala without
> This default will mess with our existing builds.
This will need to be addressed when merging to downstream branches.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf8b0f7b2d685d8b011df1926e12bf5434b5a2be
Gerrit-Change-Number: 18738
Gerrit-PatchSet: 15
Gerrit-Owner: Michael Smith <mi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: Michael Smith <mi...@cloudera.com>
Gerrit-Comment-Date: Wed, 03 Aug 2022 16:13:14 +0000
Gerrit-HasComments: Yes