[Impala-ASF-CR] IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources"
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8499 ) Change subject: IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources" .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8499 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3eec4b5a6ff350933ffda0bb80949c5960ecdf25 Gerrit-Change-Number: 8499 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Wed, 08 Nov 2017 22:03:57 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources"
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8499 ) Change subject: IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources" .. IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources" This reverts commit fe90867d890c71bfdcf8ff941f8ec51e36083f25. Change-Id: I3eec4b5a6ff350933ffda0bb80949c5960ecdf25 Reviewed-on: http://gerrit.cloudera.org:8080/8499 Reviewed-by: Thomas Tauber-Marshall Tested-by: Impala Public Jenkins --- M be/src/runtime/coordinator.cc M be/src/runtime/coordinator.h M be/src/runtime/mem-tracker.cc M be/src/runtime/mem-tracker.h M be/src/runtime/query-state.cc M be/src/scheduling/admission-controller.cc M be/src/scheduling/admission-controller.h M be/src/service/client-request-state.cc M tests/custom_cluster/test_admission_controller.py 9 files changed, 94 insertions(+), 192 deletions(-) Approvals: Thomas Tauber-Marshall: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8499 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3eec4b5a6ff350933ffda0bb80949c5960ecdf25 Gerrit-Change-Number: 8499 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Thomas Tauber-Marshall
[Impala-ASF-CR] IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources"
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8499 ) Change subject: IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources" .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8499 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3eec4b5a6ff350933ffda0bb80949c5960ecdf25 Gerrit-Change-Number: 8499 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Wed, 08 Nov 2017 19:04:39 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources"
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8499 ) Change subject: IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources" .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1448/ -- To view, visit http://gerrit.cloudera.org:8080/8499 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3eec4b5a6ff350933ffda0bb80949c5960ecdf25 Gerrit-Change-Number: 8499 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Wed, 08 Nov 2017 18:45:28 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources"
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8499 Change subject: IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources" .. IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources" This reverts commit fe90867d890c71bfdcf8ff941f8ec51e36083f25. Change-Id: I3eec4b5a6ff350933ffda0bb80949c5960ecdf25 --- M be/src/runtime/coordinator.cc M be/src/runtime/coordinator.h M be/src/runtime/mem-tracker.cc M be/src/runtime/mem-tracker.h M be/src/runtime/query-state.cc M be/src/scheduling/admission-controller.cc M be/src/scheduling/admission-controller.h M be/src/service/client-request-state.cc M tests/custom_cluster/test_admission_controller.py 9 files changed, 94 insertions(+), 192 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/99/8499/1 -- To view, visit http://gerrit.cloudera.org:8080/8499 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I3eec4b5a6ff350933ffda0bb80949c5960ecdf25 Gerrit-Change-Number: 8499 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong