Riza Suminto has uploaded this change for review. ( http://gerrit.cloudera.org:8080/23140
Change subject: IMPALA-14207: Hook log4j to Glog in AdmissionD ...................................................................... IMPALA-14207: Hook log4j to Glog in AdmissionD log4j logs does not shows up in dedicated AdmissionD service. This is because GlogAppender.Install is not called anywhere. It is called in constructor of JniCatalog.java and JniFrontend.java, but AdmissionD does not utilize either of them. AdmissionD has call path to JniRequestPoolService.java. This patch adds function ReintallGlogAppender there to call GlogAppender.Install and hook up log4j to Glog. It is written as separate method so that Executors/CatalogD does not call GlogAppender.Install twice. Testing: Run TestAdmissionControllerWithACService and confirm log4j logs appears in admissiond.INFO Change-Id: Ie28ec017f70c79d6ffdc56daa5aaeb9efcb8bb79 --- M be/src/scheduling/admissiond-env.cc M be/src/scheduling/request-pool-service.cc M be/src/scheduling/request-pool-service.h M fe/src/main/java/org/apache/impala/util/JniRequestPoolService.java M fe/src/main/java/org/apache/impala/util/RequestPoolService.java 5 files changed, 29 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/40/23140/1 -- To view, visit http://gerrit.cloudera.org:8080/23140 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ie28ec017f70c79d6ffdc56daa5aaeb9efcb8bb79 Gerrit-Change-Number: 23140 Gerrit-PatchSet: 1 Gerrit-Owner: Riza Suminto <[email protected]>
