You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Kurt Deschler (Code Review)" <ge...@cloudera.org> on 2021/03/01 13:58:07 UTC

[Impala-ASF-CR] IMPALA-10546: Add ImpalaServer interface to retrieve BackendConfig from impalad

Hello Thomas Tauber-Marshall, Joe McDonnell, John Sherman, Impala Public Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/17116

to look at the new patch set (#10).

Change subject: IMPALA-10546: Add ImpalaServer interface to retrieve BackendConfig from impalad
......................................................................

IMPALA-10546: Add ImpalaServer interface to retrieve BackendConfig from impalad

This patch add a new interface ImpalaServer::GetBackendConfig() that
returns the current TBackendGflags from impalad.

Testing:
Called new interface from external frontend. Verified that
TBackendGflags were populated correctly.

Reviewed-by: John Sherman <jf...@cloudera.com>
Change-Id: I14a3cee29f1fc91f4431b7ea89053bb3fbfa5e69
---
M be/src/catalog/catalog.cc
M be/src/rpc/hs2-http-test.cc
M be/src/service/frontend.cc
M be/src/service/impala-hs2-server.cc
M be/src/service/impala-server.h
M be/src/util/backend-gflag-util.cc
M be/src/util/backend-gflag-util.h
M be/src/util/logging-support.cc
M common/thrift/ImpalaService.thrift
M tests/hs2/test_hs2.py
10 files changed, 67 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/16/17116/10
-- 
To view, visit http://gerrit.cloudera.org:8080/17116
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I14a3cee29f1fc91f4431b7ea89053bb3fbfa5e69
Gerrit-Change-Number: 17116
Gerrit-PatchSet: 10
Gerrit-Owner: Kurt Deschler <kd...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <jo...@cloudera.com>
Gerrit-Reviewer: John Sherman <jf...@cloudera.com>
Gerrit-Reviewer: Kurt Deschler <kd...@cloudera.com>
Gerrit-Reviewer: Thomas Tauber-Marshall <tm...@cloudera.com>