[kudu-CR](gh-pages) Add weekly update for 07/25

2016-07-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 07/25 .. Patch Set 4: Code-Review+2 Verified+1 Gonna assume Todd and Mike are fine with my latest edit. -- To view, visit http://ger

[kudu-CR](gh-pages) Add weekly update for 07/25

2016-07-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add weekly update for 07/25 .. Add weekly update for 07/25 Change-Id: I95de597ca5e33c05ec4f820315952310f07983a0 Reviewed-on: http://gerrit.cloudera

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-07-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1358 (part 3): new multi-master stress test .. Patch Set 13: Verified+1 Overriding Jenkins, hit KUDU-1527 again. -- To view, visit http://gerrit.cloudera.org

[kudu-CR] Fix zip command in make site.sh

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix zip command in make_site.sh .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2659/ -- To view, visit http://gerrit.cloudera.org:8080/3780

[kudu-CR] Fix zip command in make site.sh

2016-07-26 Thread Mike Percy (Code Review)
Hello Adar Dembo, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3780 to review the following change. Change subject: Fix zip command in make_site.sh .. Fix zip command in ma

[kudu-CR] Fix a couple bugs in make site.sh

2016-07-26 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new patch set (#2). Change subject: Fix a couple bugs in make_site.sh .. Fix a couple bugs in make_site.sh This patch fixes a couple minor script bugs: * zip command should not quote arguments * doxyge

[kudu-CR] Fix a couple bugs in make site.sh

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Fix a couple bugs in make_site.sh .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2660/ -- To view, visit http://gerrit.cloudera.org:8080/37

[kudu-CR] Fix a couple bugs in make site.sh

2016-07-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Fix a couple bugs in make_site.sh .. Patch Set 2: Code-Review+2 So you'll need to modify Kudu-Docs to pass --no-doxygen until we get doxygen installed in those bui

[kudu-CR] Fix a couple bugs in make site.sh

2016-07-26 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Fix a couple bugs in make_site.sh .. Patch Set 2: > So you'll need to modify Kudu-Docs to pass --no-doxygen until we > get doxygen installed in those build images,

[kudu-CR] Fix a couple bugs in make site.sh

2016-07-26 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Fix a couple bugs in make_site.sh .. Fix a couple bugs in make_site.sh This patch fixes a couple minor script bugs: * zip command should not quote argumen

[kudu-CR] Import CallbackList from chromium-base

2016-07-26 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: Import CallbackList from chromium-base .. Abandoned we didn't end up needing this stuff. also, now that Kudu supports C++11, we could import the variadic template version i

[kudu-CR] Changes to import CallbackList from chromium-base

2016-07-26 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: Changes to import CallbackList from chromium-base .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/998 To unsubscribe, visit http://gerrit.cloudera.org:8080/s

[kudu-CR] Add support to Env for creation of a temp dir

2016-07-26 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: Add support to Env for creation of a temp dir .. Abandoned didn't end up needing this -- To view, visit http://gerrit.cloudera.org:8080/996 To unsubscribe, visit http://ge

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-07-26 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new change for review. http://gerrit.cloudera.org:8080/3786 Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. KUDU-526: use on-disk cmeta when loading existing master s

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2661/ -- To view, visit http:

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-07-26 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 1: Code-Review+2 (1 comment) This patch looks fine to me. It obviously converts the gfl

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-07-26 Thread Adar Dembo (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3786 to look at the new patch set (#2). Change subject: KUDU-526: use on-disk cmeta when loading existing master state ..

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2662/ -- To view, visit http:

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-07-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3786/1/src/kudu/integration-tests/ma

[kudu-CR] add apply retry time limit

2016-07-26 Thread song bruce zhang (Code Review)
song bruce zhang has uploaded a new change for review. http://gerrit.cloudera.org:8080/3787 Change subject: add apply retry time limit .. add apply retry time limit Change-Id: Iacd33cdc5316e294e613d1b2273ef12e6b1cf687 --- M j

[kudu-CR] add apply retry time limit

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: add apply retry time limit .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2663/ -- To view, visit http://gerrit.cloudera.org:8080/3787 To u

[kudu-CR] add apply retry time limit

2016-07-26 Thread song bruce zhang (Code Review)
song bruce zhang has abandoned this change. Change subject: add apply retry time limit .. Abandoned invalid commit message format. -- To view, visit http://gerrit.cloudera.org:8080/3787 To unsubscribe, visit http://gerrit.clo

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-07-26 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3786 To unsubscribe,

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread song bruce zhang (Code Review)
song bruce zhang has uploaded a new patch set (#2). Change subject: KUDU-1542: in some cases apply will hang. .. KUDU-1542: in some cases apply will hang. In current implementation, apply will retry forever when met failed. It w

[kudu-CR] add apply retry time limit

2016-07-26 Thread song bruce zhang (Code Review)
song bruce zhang has restored this change. Change subject: add apply retry time limit .. Restored -- To view, visit http://gerrit.cloudera.org:8080/3787 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Me

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1542: in some cases apply will hang. .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2664/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] KUDU-526: use on-disk cmeta when loading existing master state

2016-07-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-526: use on-disk cmeta when loading existing master state .. Patch Set 2: Verified+1 Overriding Jenkins, the isolate server died mid-run and several tests fail

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1542: in some cases apply will hang. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3787/2/java/kudu-client/src/main/java/org/apache/

[kudu-CR] docs: design for handling permanent master failures

2016-07-26 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3393 to look at the new patch set (#3). Change subject: docs: design for handling permanent master failures .. docs: des

[kudu-CR] docs: design for handling permanent master failures

2016-07-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: docs: design for handling permanent master failures .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3393/2/docs/design-docs/master-perm-failure-1.

[kudu-CR] docs: design for handling permanent master failures

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: docs: design for handling permanent master failures .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2665/ -- To view, visit http://gerrit.cl

[kudu-CR] docs: design for handling permanent master failures

2016-07-26 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: docs: design for handling permanent master failures .. Patch Set 2: The doc has been changed significantly. The "real" approach now requires DNS cnames and thus do

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread song bruce zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3787 to look at the new patch set (#3). Change subject: KUDU-1542: in some cases apply will hang. .. KUDU-1542: in some

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1542: in some cases apply will hang. .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2666/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread song bruce zhang (Code Review)
song bruce zhang has posted comments on this change. Change subject: KUDU-1542: in some cases apply will hang. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3787/2/java/kudu-client/src/main/java/org/apache/ku

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1542: in some cases apply will hang. .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2667/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] KUDU-1542: in some cases apply will hang.

2016-07-26 Thread song bruce zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3787 to look at the new patch set (#4). Change subject: KUDU-1542: in some cases apply will hang. .. KUDU-1542: in some