Henry Robinson has posted comments on this change. Change subject: MT: Planner for multi-threaded execution ......................................................................
Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/2846/5/common/thrift/ImpalaInternalService.thrift File common/thrift/ImpalaInternalService.thrift: Line 188: 44: optional i32 mt_num_cores = 1 > fair concern. let's solve that by having a blacklist of invisible options i Are you going to address that in a follow-on patch? I think it's important to get this done for 2.6. -- To view, visit http://gerrit.cloudera.org:8080/2846 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic3c34dd3f9190a131e6f03d901b4bfcd164a5174 Gerrit-PatchSet: 5 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Marcel Kornacker <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Marcel Kornacker <[email protected]> Gerrit-HasComments: Yes
