Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/12643 )
Change subject: [thirdparty] Bump RapidJSON version to 1.1.0 ...................................................................... Patch Set 5: (1 comment) looks like lots of test failures from this new version in ASAN builds. I think Mike and Adar have been looking at rapidjson UBSAN problems as well http://gerrit.cloudera.org:8080/#/c/12643/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/12643/5//COMMIT_MSG@16 PS5, Line 16: output multiple JSONs, and also add an unit test for this. what's the advantage of reset vs a new one? it can reuse a buffer or something and avoid allocation if you serialize a bunch of JSON in a row? -- To view, visit http://gerrit.cloudera.org:8080/12643 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1c5e6bf6b8c6802bde28716487abb2b242a7a578 Gerrit-Change-Number: 12643 Gerrit-PatchSet: 5 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Sat, 02 Mar 2019 17:57:12 +0000 Gerrit-HasComments: Yes
