Hello Csaba Ringhofer, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16449
to look at the new patch set (#7).
Change subject: IMPALA-10168: Expose JSON catalog objects in catalogd's debug
page
......................................................................
IMPALA-10168: Expose JSON catalog objects in catalogd's debug page
Catalogd has a debug page at '/catalog_object' showing catalog objects
in thrift debug strings. It's inconvenient for tests to parse the thrift
string and get interested infos.
This patch extends this page to support returning JSON results, which
eases tests to extract complex infos from the catalog objects, e.g.
partition ids of a hdfs table.
The builtin method of thrift, ThriftJSONString, is used to get the JSON
strings. Thrift structs are represented as JSON objects, with the field
ID as the key, and the field value represented as a JSON object with a
single key-value pair. The key is a short string identifier for that
type, followed by the value. See the class comment for more details:
https://github.com/apache/thrift/blob/0.11.0/lib/cpp/src/thrift/protocol/TJSONProtocol.h
Tests:
- Add tests to verified json fields existence.
Change-Id: I15f256b4e3f5206c7140746694106e03b0a4ad92
---
M be/src/catalog/catalog-server.cc
M tests/webserver/test_web_pages.py
2 files changed, 67 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/49/16449/7
--
To view, visit http://gerrit.cloudera.org:8080/16449
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I15f256b4e3f5206c7140746694106e03b0a4ad92
Gerrit-Change-Number: 16449
Gerrit-PatchSet: 7
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
