[kudu-CR] cache: switch to std::atomic
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10209 ) Change subject: cache: switch to std::atomic .. cache: switch to std::atomic A previous commit added some use of std::atomic, so this patch makes the file self-consistent. Change-Id: I359f273af9ba7130d230ef639a40e12a8ac51a06 Reviewed-on: http://gerrit.cloudera.org:8080/10209 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon--- M src/kudu/util/cache.cc 1 file changed, 11 insertions(+), 12 deletions(-) Approvals: Kudu Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/10209 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I359f273af9ba7130d230ef639a40e12a8ac51a06 Gerrit-Change-Number: 10209 Gerrit-PatchSet: 5 Gerrit-Owner: Todd Lipcon Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: David Ribeiro Alves Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] cache: switch to std::atomic
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10209 ) Change subject: cache: switch to std::atomic .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/10209 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I359f273af9ba7130d230ef639a40e12a8ac51a06 Gerrit-Change-Number: 10209 Gerrit-PatchSet: 3 Gerrit-Owner: Todd LipconGerrit-Reviewer: Adar Dembo Gerrit-Reviewer: David Ribeiro Alves Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Thu, 26 Apr 2018 17:42:43 + Gerrit-HasComments: No
[kudu-CR] cache: switch to std::atomic
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10209 ) Change subject: cache: switch to std::atomic .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/10209 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I359f273af9ba7130d230ef639a40e12a8ac51a06 Gerrit-Change-Number: 10209 Gerrit-PatchSet: 1 Gerrit-Owner: Todd LipconGerrit-Reviewer: Adar Dembo Gerrit-Reviewer: David Ribeiro Alves Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Wed, 25 Apr 2018 21:47:54 + Gerrit-HasComments: No
[kudu-CR] cache: switch to std::atomic
Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10209 to review the following change. Change subject: cache: switch to std::atomic .. cache: switch to std::atomic A previous commit added some use of std::atomic, so this patch makes the file self-consistent. Change-Id: I359f273af9ba7130d230ef639a40e12a8ac51a06 --- M src/kudu/util/cache.cc 1 file changed, 11 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/09/10209/1 -- To view, visit http://gerrit.cloudera.org:8080/10209 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I359f273af9ba7130d230ef639a40e12a8ac51a06 Gerrit-Change-Number: 10209 Gerrit-PatchSet: 1 Gerrit-Owner: Todd LipconGerrit-Reviewer: David Ribeiro Alves