Yingchun Lai has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12835
Change subject: [json] Add JsonFileReader to read JSON content from file ...................................................................... [json] Add JsonFileReader to read JSON content from file Introduce a new class JsonFileReader, which has the same APIs with JsonReader. In fact they are derived from the same base class JsonReaderBase, the difference is JsonReader reads JSON content from raw string, while JsonFileReader reads content from file. With JsonFileReader, it's able to read configuration from file, and configfile reader is useful in some usecases. For example, a Kudu administrator who manage several Kudu clusters is not easy to remember all master rpc addresses when use kudu CLI tool to access them. But it's easy to access these clusters by a simplfy cluster name, which will be resolved to master rpc addresses by a JSON file. I'll introduce a cluster name resolve method later. Change-Id: I7c57c3c31189578268b126ef535bf3f933867288 --- M src/kudu/util/jsonreader-test.cc M src/kudu/util/jsonreader.cc M src/kudu/util/jsonreader.h 3 files changed, 300 insertions(+), 206 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/35/12835/1 -- To view, visit http://gerrit.cloudera.org:8080/12835 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I7c57c3c31189578268b126ef535bf3f933867288 Gerrit-Change-Number: 12835 Gerrit-PatchSet: 1 Gerrit-Owner: Yingchun Lai <[email protected]>
