You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@kudu.apache.org by "Yingchun Lai (Code Review)" <ge...@cloudera.org> on 2024/04/01 02:11:41 UTC

[kudu-CR] [codegen] use std::atomic instead of AtomicInt

Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/21225 )

Change subject: [codegen] use std::atomic instead of AtomicInt
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/21225/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/21225/1//COMMIT_MSG@14
PS1, Line 14: are
Missing "no"?



-- 
To view, visit http://gerrit.cloudera.org:8080/21225
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3271ef420163a6d996645bb60aa8d475c7925f92
Gerrit-Change-Number: 21225
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yifan Zhang <ch...@163.com>
Gerrit-Reviewer: Yingchun Lai <la...@apache.org>
Gerrit-Comment-Date: Mon, 01 Apr 2024 02:11:41 +0000
Gerrit-HasComments: Yes