Hello Dan Burkert, Kudu Jenkins,

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

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

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

Change subject: Add json builder utility
......................................................................

Add json builder utility

The EasyJson class serves as a wrapper around rapidjson
Value objects, to simplify their usage. It is intended solely
for building json objects, not writing/parsing.

Simplifies code like this:

  rapidjson::Document d;
  rapidjson::Value v;
  v.SetObject();
  rapidjson::Value list;
  list.SetArray();
  v.AddMember("list", list, d.GetAllocator());
  v["list"].PushBack(rapidjson::Value().SetString("element"), d.GetAllocator());

To this:

  EasyJson ej;
  ej["list"][0] = "element";

This commit is intended to facilitate the transition
of webserver path handlers from raw HTML to mustache
templates and json objects.

Change-Id: Ie195b06c5ba67af3043d2032c56b90c1a85f641a
---
M src/kudu/util/CMakeLists.txt
A src/kudu/util/easy_json-test.cc
A src/kudu/util/easy_json.cc
A src/kudu/util/easy_json.h
4 files changed, 412 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/87/7387/6
-- 
To view, visit http://gerrit.cloudera.org:8080/7387
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie195b06c5ba67af3043d2032c56b90c1a85f641a
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Sam Okrent <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Sam Okrent <[email protected]>
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to