Maxim Smyatkin has uploaded a new change for review. http://gerrit.cloudera.org:8080/5123
Change subject: KUDU-1634 (part 3). Changed tmp files' infix from ".tmp" to ".kudutmp" ...................................................................... KUDU-1634 (part 3). Changed tmp files' infix from ".tmp" to ".kudutmp" As these ".tmp" strings were scattered over the code, I've tried to remove this duplication as well by defining the single kTmpInfix in pb_util.h. In a couple of places it led to string concatenations, but I suppose it's not that bad for overal performance? The necessity of moving from "tmp" to "kudutmp" is backed by the idea that we don't want to remove any user's data accidentally (see https://gerrit.cloudera.org/#/c/5007/). Change-Id: I52a794cc16d008570039d276ab32e6a26829bc53 --- M src/kudu/consensus/log.cc M src/kudu/consensus/log_util.cc M src/kudu/consensus/log_util.h M src/kudu/fs/fs_manager-test.cc M src/kudu/fs/fs_manager.cc M src/kudu/fs/fs_manager.h M src/kudu/integration-tests/external_mini_cluster_fs_inspector.cc M src/kudu/tserver/tablet_copy_client.cc M src/kudu/tserver/tablet_copy_session-test.cc M src/kudu/util/pb_util.cc M src/kudu/util/pb_util.h 11 files changed, 31 insertions(+), 29 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/23/5123/1 -- To view, visit http://gerrit.cloudera.org:8080/5123 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I52a794cc16d008570039d276ab32e6a26829bc53 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Maxim Smyatkin <[email protected]>
