Adar Dembo has submitted this change and it was merged. Change subject: [server] clean-up: stringstream --> ostringstream ......................................................................
[server] clean-up: stringstream --> ostringstream Replaced std::stringstream with std::ostringstream in places where only output stream functionality is required from the stream buffer class. Include iosfwd instead of stream headers for forward declarations. Minor clean-up on header files include order to be in line with the styling guide. There are not any functional changes in this changelist. Change-Id: I059c19bb287ee37f96e5ad42f4886015a1697d19 Reviewed-on: http://gerrit.cloudera.org:8080/4303 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins Reviewed-by: Will Berkeley <wdberke...@gmail.com> --- M src/kudu/client/samples/sample.cc M src/kudu/codegen/code_generator.cc M src/kudu/codegen/module_builder.cc M src/kudu/master/master-path-handlers.cc M src/kudu/master/master-path-handlers.h M src/kudu/server/default-path-handlers.cc M src/kudu/server/pprof-path-handlers.cc M src/kudu/server/rpcz-path-handler.cc M src/kudu/server/server_base.cc M src/kudu/server/tracing-path-handlers.cc M src/kudu/server/tracing-path-handlers.h M src/kudu/server/webserver-test.cc M src/kudu/server/webserver.cc M src/kudu/server/webserver.h M src/kudu/server/webui_util.cc M src/kudu/server/webui_util.h M src/kudu/tablet/tablet-test.cc M src/kudu/tools/fs_dump-tool.cc M src/kudu/tools/fs_list-tool.cc M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck_remote-test.cc M src/kudu/tserver/tablet_server-stress-test.cc M src/kudu/tserver/tablet_server-test.cc M src/kudu/tserver/tserver-path-handlers.cc M src/kudu/tserver/tserver-path-handlers.h M src/kudu/util/jsonwriter.cc M src/kudu/util/jsonwriter.h M src/kudu/util/logging.cc M src/kudu/util/mem_tracker.cc M src/kudu/util/metrics-test.cc M src/kudu/util/metrics.cc M src/kudu/util/os-util.cc M src/kudu/util/pb_util.cc M src/kudu/util/spinlock_profiling-test.cc M src/kudu/util/spinlock_profiling.cc M src/kudu/util/spinlock_profiling.h M src/kudu/util/thread.cc M src/kudu/util/thread.h M src/kudu/util/trace.cc M src/kudu/util/url-coding-test.cc M src/kudu/util/url-coding.cc M src/kudu/util/url-coding.h M src/kudu/util/web_callback_registry.h 43 files changed, 228 insertions(+), 192 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Will Berkeley: Looks good to me, but someone else must approve Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4303 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I059c19bb287ee37f96e5ad42f4886015a1697d19 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
