[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. Change subject: [thirdparty]: added include-what-you-use .. [thirdparty]: added include-what-you-use Build the include-what-you-use utility along with the LLVM toolchai

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-14 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [thirdparty]: added include-what-you-use .. Patch Set 6: Verified+1 Unrelated flake in org.apache.kudu.client.TestScannerMultiTablet.org.apache.kudu.client.Test

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [thirdparty]: added include-what-you-use .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7593 To unsubscribe, visit http://gerrit.c

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-14 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7593 to look at the new patch set (#6). Change subject: [thirdparty]: added include-what-you-use .. [thirdparty]: added

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-14 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7593 to look at the new patch set (#4). Change subject: [thirdparty]: added include-what-you-use .. [thirdparty]: added

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [thirdparty]: added include-what-you-use .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7593/3/thirdparty/patches/llvm-iwyu-nocurses.patch File

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [thirdparty]: added include-what-you-use .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/7593/3/thirdparty/download-thirdparty.sh File thirdpar

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-11 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [thirdparty]: added include-what-you-use .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7593/3/thirdparty/patches/llvm-iwyu-nocurses.patch File th

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-11 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [thirdparty]: added include-what-you-use .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7593/3/thirdparty/download-thirdparty.sh File thirdparty/d

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: [thirdparty]: added include-what-you-use .. Patch Set 3: (2 comments) any idea what the incremental compilation time cost is for thirdparty on this? http://gerri

[kudu-CR] [thirdparty]: added include-what-you-use

2017-08-07 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7593 to look at the new patch set (#3). Change subject: [thirdparty]: added include-what-you-use .. [thirdparty]: added