Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/23157
Change subject: [webserver] avoid memory alloc©ing for HTTP status ...................................................................... [webserver] avoid memory alloc©ing for HTTP status There is no need to create an instance of std::string out of a static string literal just to generate formatted HTTP status line: passing 'const char*' to strings::Substitute() is enough. Also, don't crash on unexpected HTTP status codes in RELEASE builds, but log an error and return '500 Internal Server Error' instead, reserving SIGABRT only for DEBUG builds. This changelist doesn't contains any functional modifications. Change-Id: I58d40bc8bf41791e53b5033ee95e981c6957faa1 --- M src/kudu/server/webserver.cc 1 file changed, 33 insertions(+), 16 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/57/23157/1 -- To view, visit http://gerrit.cloudera.org:8080/23157 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I58d40bc8bf41791e53b5033ee95e981c6957faa1 Gerrit-Change-Number: 23157 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
