Alexey Serbin has submitted this change and it was merged. Change subject: [util] fix compilation breakage ......................................................................
[util] fix compilation breakage On some Linux systems (old ones), there isn't linux/sysinfo.h header file. IWYU might suggest to include linux/sysinfo.h instead of linux/kernel.h if compiling on a new system, so I added corresponding IWYU pragmas. I'll address the IWYU issue in a generic way in a separate change list (so no pragmas would be necessary). This patch is a quick fix for the compilation breakage. Change-Id: Ie9d13815052d8e4c1360db3f94e15ffd77768af5 Reviewed-on: http://gerrit.cloudera.org:8080/8065 Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Andrew Wong <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/util/env_posix.cc 1 file changed, 5 insertions(+), 4 deletions(-) Approvals: Andrew Wong: Looks good to me, but someone else must approve Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8065 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9d13815052d8e4c1360db3f94e15ffd77768af5 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot
