Khazar Mammadli has posted comments on this change. ( http://gerrit.cloudera.org:8080/17555 )
Change subject: [rest] add rest implementation ...................................................................... Patch Set 50: (2 comments) http://gerrit.cloudera.org:8080/#/c/17555/50/src/kudu/integration-tests/rest_server-itest.cc File src/kudu/integration-tests/rest_server-itest.cc: http://gerrit.cloudera.org:8080/#/c/17555/50/src/kudu/integration-tests/rest_server-itest.cc@84 PS50, Line 84: vector<string> ParseResponse(const std::vector<std::string>& response) { > EasyJson uses rapidjson in the background, afaik that supports schema-less Will give it a look, main reason I have avoided using it for writing/parsing was this line in the Easy JSON header file: // Intended solely for building json objects, not writing/parsing. So if this isn't a problem I can give it a try http://gerrit.cloudera.org:8080/#/c/17555/50/src/kudu/rest/rest_server.cc File src/kudu/rest/rest_server.cc: http://gerrit.cloudera.org:8080/#/c/17555/50/src/kudu/rest/rest_server.cc@74 PS50, Line 74: .setVersion("1.15.0") > Can't you use VersionInfo? Yes, it can be, updated it to use short version info -- To view, visit http://gerrit.cloudera.org:8080/17555 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2ca3121fd7e95a1267853be45cb5f5855298c763 Gerrit-Change-Number: 17555 Gerrit-PatchSet: 50 Gerrit-Owner: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Thu, 26 May 2022 12:39:48 +0000 Gerrit-HasComments: Yes
