Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15377 )
Change subject: util: remove all gscoped_ptr ...................................................................... Patch Set 1: Code-Review+1 (2 comments) http://gerrit.cloudera.org:8080/#/c/15377/1/src/kudu/tablet/concurrent_btree.h File src/kudu/tablet/concurrent_btree.h: http://gerrit.cloudera.org:8080/#/c/15377/1/src/kudu/tablet/concurrent_btree.h@1101 PS1, Line 1101: nit: maybe, drop the extra space while you are at it? http://gerrit.cloudera.org:8080/#/c/15377/1/src/kudu/util/user.cc File src/kudu/util/user.cc: http://gerrit.cloudera.org:8080/#/c/15377/1/src/kudu/util/user.cc@60 PS1, Line 60: size_t bufsize = retval > 0 ? retval : 16384; : : unique_ptr<char[]> buf(new char[bufsize]); nit: is it possible to allocate 16K buffer on stack and then use it or it's part instead of calling new? Adding CHECK_LE(retval, 16384) for sanity check. -- To view, visit http://gerrit.cloudera.org:8080/15377 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia110feea85b6ca43ba53d2d4783f47000794e196 Gerrit-Change-Number: 15377 Gerrit-PatchSet: 1 Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 06 Mar 2020 03:52:02 +0000 Gerrit-HasComments: Yes