[kudu-CR] thirdparty: split into dependency groups

2016-09-29 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: thirdparty: split into dependency groups .. thirdparty: split into dependency groups The monolithic thirdparty build is now quite a bit larger than it used

[kudu-CR] thirdparty: split into dependency groups

2016-09-29 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 4: Code-Review+2 Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4513 To unsubscribe, visit htt

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4513 To unsubscribe, visit http://gerrit.

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4513 to look at the new patch set (#3). Change subject: thirdparty: split into dependency groups .. thirdparty: split in

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/4513/2/thirdparty/build-thirdparty.sh File thirdparty/bui

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/4513/2/thirdparty/build-thirdparty.sh File thirdparty/bu

[kudu-CR] thirdparty: split into dependency groups

2016-09-28 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4513 to look at the new patch set (#2). Change subject: thirdparty: split into dependency groups .. thirdparty: split in

[kudu-CR] thirdparty: split into dependency groups

2016-09-27 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4513/1//COMMIT_MSG Commit Message: Line 29: "instrumente

[kudu-CR] thirdparty: split into dependency groups

2016-09-22 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4513/1/thirdparty/build-if-necessary.sh File thirdparty

[kudu-CR] thirdparty: split into dependency groups

2016-09-22 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: thirdparty: split into dependency groups .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4513/1//COMMIT_MSG Commit Message: Line 29: "instrument

[kudu-CR] thirdparty: split into dependency groups

2016-09-21 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4513 to review the following change. Change subject: thirdparty: split into dependency groups .. thirdparty: s