[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Add support for clang from XCode 8 in macOS .. Add support for clang from XCode 8 in macOS Since upgrading XCode to 8 (even in El Capitan), cmake

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 2: Code-Review+2 I agree the malloc patch should be separate. -- To view, visit

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
Hello Dan Burkert, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4482 to look at the new patch set (#2). Change subject: Add support for clang from XCode 8 in macOS .. Add support for

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 1: It seems this is missing a gperftools side update to work with tcmalloc:

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 1: -Code-Review -- To view, visit http://gerrit.cloudera.org:8080/4482 To unsubscribe, visit

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 1: Code-Review-1 Hum, turns out there's a malloc issue with the new compiler. Sorting that out

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add support for clang from XCode 8 in macOS .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4482 To unsubscribe, visit

[kudu-CR] Add support for clang from XCode 8 in macOS

2016-09-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/4482 Change subject: Add support for clang from XCode 8 in macOS .. Add support for clang from XCode 8 in macOS Since upgrading XCode