[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [thirdparty] Make Boost a regular dependency .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5818 To unsubscribe, visit http://ger

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-07 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: [thirdparty] Make Boost a regular dependency .. [thirdparty] Make Boost a regular dependency For the new TIMESTAMP type we must mimic Impala's TimestampVal

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-07 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: [thirdparty] Make Boost a regular dependency .. Patch Set 10: Code-Review+1 Ah, nice to see you found the magical incantation for jam. :) -- To view, visit http:/

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5818 to look at the new patch set (#10). Change subject: [thirdparty] Make Boost a regular dependency .. [thirdparty] Ma

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5818 to look at the new patch set (#9). Change subject: [thirdparty] Make Boost a regular dependency .. [thirdparty] Mak

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [thirdparty] Make Boost a regular dependency .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/5818/5//COMMIT_MSG Commit Message: PS5, Lin

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [thirdparty] Make Boost a regular dependency .. Patch Set 8: (3 comments) Looks good, just a couple doc requests. http://gerrit.cloudera.org:8080/#/c/5818/5//COMM

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [thirdparty] Make Boost a regular dependency .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/5818/1/CMakeLists.txt File CMakeLists.txt:

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-06 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5818 to look at the new patch set (#8). Change subject: [thirdparty] Make Boost a regular dependency .. [thirdparty] Mak

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [thirdparty] Make Boost a regular dependency .. Patch Set 5: (8 comments) http://gerrit.cloudera.org:8080/#/c/5818/5//COMMIT_MSG Commit Message: PS5, Lin

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-06 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5818 to look at the new patch set (#7). Change subject: [thirdparty] Make Boost a regular dependency .. [thirdparty] Mak

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-06 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5818 to look at the new patch set (#6). Change subject: [thirdparty] Make Boost a regular dependency .. [thirdparty] Mak

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-03 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [thirdparty] Make Boost a regular dependency .. Patch Set 5: (9 comments) http://gerrit.cloudera.org:8080/#/c/5818/5//COMMIT_MSG Commit Message: PS5, Line 9: Howe

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [thirdparty] Make Boost a regular dependency .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5818/4/thirdparty/build-definitions.sh File t

[kudu-CR] [thirdparty] Make Boost a regular dependency

2017-02-03 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5818 to look at the new patch set (#5). Change subject: [thirdparty] Make Boost a regular dependency .. [thirdparty] Mak