You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2018/08/01 18:14:44 UTC

[native-toolchain-CR] IMPALA-7364: Add RapidJson 1.1.0

Hello Quanlong Huang, 

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

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

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

Change subject: IMPALA-7364: Add RapidJson 1.1.0
......................................................................

IMPALA-7364: Add RapidJson 1.1.0

Also copy over license to built tarball.

Testing:
Built both rapidjson version locally with:

  ./build.sh rapidjson 0.11
  ./build.sh rapidjson 1.1.0

Will run build on a range of operating systems but I don't anticipate
any problems since this "build" is so trivial.

Change-Id: Ie98462fde43c97101e95d0761be1594bc2636859
---
M buildall.sh
M source/rapidjson/build.sh
2 files changed, 4 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/native-toolchain refs/changes/92/11092/2
-- 
To view, visit http://gerrit.cloudera.org:8080/11092
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie98462fde43c97101e95d0761be1594bc2636859
Gerrit-Change-Number: 11092
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <hu...@gmail.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>