Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4160: Remove Llama support. ......................................................................
Patch Set 4: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/4445/1/be/src/scheduling/query-schedule.cc File be/src/scheduling/query-schedule.cc: PS1, Line 112: bool ignored; > I don't think we can or should remove the query option now, as that would b can we print "deprecated" somewhere? http://gerrit.cloudera.org:8080/#/c/4445/4/common/thrift/Frontend.thrift File common/thrift/Frontend.thrift: Line 390: // TODO: Remove this and associated code in Planner. also add a todo in Planner.java, it's too easy to overlook here -- To view, visit http://gerrit.cloudera.org:8080/4445 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Icfb14209e31f6608bb7b8a33789e00411a6447ef Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Henry Robinson <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Marcel Kornacker <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]> Gerrit-HasComments: Yes
