Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9888 )
Change subject: Followup to bae6170c77 to fix build on macos ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/9888/2/src/kudu/util/env.h File src/kudu/util/env.h: http://gerrit.cloudera.org:8080/#/c/9888/2/src/kudu/util/env.h@322 PS2, Line 322: If there is : // no limit, returns RLIM_INFINITY which is guaranteed to be positive. > Alternatively, maybe we can just remove this sentence? AFAICT callers can Yeah that's an option too; I guess I was concerned that callers may know that rlimits on UNIX have a concept of "no limit" and would go hunting for it if it wasn't explicitly talked about. But it's probably not a real issue. -- To view, visit http://gerrit.cloudera.org:8080/9888 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I13780d86d4a286710c1bcf8a5f4266fb7fe162c7 Gerrit-Change-Number: 9888 Gerrit-PatchSet: 2 Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Mon, 02 Apr 2018 19:23:42 +0000 Gerrit-HasComments: Yes
