Bikramjeet Vig has posted comments on this change. Change subject: IMPALA-5104: Fix Query Admission Rule ......................................................................
Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7401/1//COMMIT_MSG Commit Message: PS1, Line 7: Fix Query Admission Rule > Vague subject line, which is often the only thing shown (e.g. git log --one Done http://gerrit.cloudera.org:8080/#/c/7401/1/tests/custom_cluster/test_admission_controller.py File tests/custom_cluster/test_admission_controller.py: PS1, Line 323: PROC_MEM_TEST_LIMIT > move this constant to just above this method since it's only used here Done PS1, Line 326: requested > the requested Done PS1, Line 326: query > a query Done PS1, Line 326: process > the process Done -- To view, visit http://gerrit.cloudera.org:8080/7401 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I197648f4162f2057141517b4b42ab5196884a65a Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com> Gerrit-HasComments: Yes