[kudu-CR] Remove the LocalConsensus implementation

2016-06-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Remove the LocalConsensus implementation .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3350 To unsubscribe, visit

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add blog post about removing LocalConsensus .. Patch Set 1: (3 comments) Thanks for the review David. I updated the rendered HTML.

[kudu-CR] Remove the LocalConsensus implementation

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Remove the LocalConsensus implementation .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3350 To unsubscribe, visit

[kudu-CR] Use RaftConsensus instead of LocalConsensus in tests

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Use RaftConsensus instead of LocalConsensus in tests .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3346 To unsubscribe, visit

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Add blog post about removing LocalConsensus .. Patch Set 1: (3 comments)

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add blog post about removing LocalConsensus .. Patch Set 1: (3 comments) Thanks for the review JD. I made a couple more tweaks and updated the rendered HTML as

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new patch set (#2). Change subject: Add blog post about removing LocalConsensus .. Add blog post about removing LocalConsensus Change-Id: I571de0cbf4d73511f17bd57d374fed9e8ca302e4 --- A

[kudu-CR] Reformat raft-config-change.md to clean it up

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Reformat raft-config-change.md to clean it up .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1851/ -- To view, visit

[kudu-CR] Reformat raft-config-change.md to clean it up

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new patch set (#2). Change subject: Reformat raft-config-change.md to clean it up .. Reformat raft-config-change.md to clean it up It just needed a little bit of markdown syntax love. Change-Id:

[kudu-CR] Reformat raft-config-change.md to clean it up

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Reformat raft-config-change.md to clean it up .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1850/ -- To view, visit

[kudu-CR] Reformat raft-config-change.md to clean it up

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Reformat raft-config-change.md to clean it up .. Patch Set 1: Rendered HTML:

[kudu-CR] Reformat raft-config-change.md to clean it up

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Reformat raft-config-change.md to clean it up .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1849/ -- To view, visit

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add blog post about removing LocalConsensus .. Patch Set 1: (3 comments) I really enjoyed reading this, just minor nits.

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add blog post about removing LocalConsensus .. Patch Set 1: Rendered: http://mpercy.github.io/kudu/2016/06/17/raft-consensus-single-node.html -- To view, visit

[kudu-CR](gh-pages) Add blog post about removing LocalConsensus

2016-06-16 Thread Mike Percy (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3398 to review the following change. Change subject: Add blog post about removing LocalConsensus .. Add blog post

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add Raft remote bootstrap design doc .. Patch Set 2: Updated HTML: https://github.com/mpercy/kudu/blob/consensus-docs-3a/docs/design-docs/raft-remote-bootstrap.md

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add Raft remote bootstrap design doc .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1848/ -- To view, visit

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add Raft remote bootstrap design doc .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/3395/2/docs/design-docs/raft-remote-bootstrap.md File

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3395 to look at the new patch set (#3). Change subject: Add Raft remote bootstrap design doc .. Add Raft remote

[kudu-CR] Use RaftConsensus instead of LocalConsensus in tests

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Use RaftConsensus instead of LocalConsensus in tests .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/1846/ -- To view, visit

[kudu-CR] Use RaftConsensus instead of LocalConsensus in tests

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Use RaftConsensus instead of LocalConsensus in tests .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/3346/4/src/kudu/consensus/consensus.h File

[kudu-CR] Remove the LocalConsensus implementation

2016-06-16 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3350 to look at the new patch set (#6). Change subject: Remove the LocalConsensus implementation .. Remove the

[kudu-CR] Remove the LocalConsensus implementation

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Remove the LocalConsensus implementation .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/1847/ -- To view, visit

[kudu-CR] Use RaftConsensus instead of LocalConsensus in tests

2016-06-16 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3346 to look at the new patch set (#5). Change subject: Use RaftConsensus instead of LocalConsensus in tests .. Use

[kudu-CR] Remove the LocalConsensus implementation

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Remove the LocalConsensus implementation .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/3350/5/src/kudu/consensus/metadata.proto File

[kudu-CR] Link to raft config change design from design-docs index page

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Link to raft config change design from design-docs index page .. Link to raft config change design from design-docs index page Change-Id:

[kudu-CR] Link to raft config change design from design-docs index page

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Link to raft config change design from design-docs index page .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3394 To unsubscribe,

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add Raft remote bootstrap design doc .. Patch Set 2: (4 comments) Looks good, just a few formatting nits.

[kudu-CR] Link to raft config change design from design-docs index page

2016-06-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Link to raft config change design from design-docs index page .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3394 To unsubscribe,

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java-client] use tablet cache for locateTablet calls .. Patch Set 4: Code-Review+2 (1 comment) Feel free to punt on the nit if you want to merge right now.

[kudu-CR] Add more helpful CHECK message at master startup

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Add more helpful CHECK message at master startup .. Add more helpful CHECK message at master startup Maybe this could help debug KUDU-1488 a little bit

[kudu-CR] Add more helpful CHECK message at master startup

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add more helpful CHECK message at master startup .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3396 To unsubscribe, visit

[kudu-CR] Add more helpful CHECK message at master startup

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add more helpful CHECK message at master startup .. Patch Set 1: Jenkins in borken so +1ing myself -- To view, visit http://gerrit.cloudera.org:8080/3396 To

[kudu-CR] RaftConsensus: Trigger election at startup if single node

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: RaftConsensus: Trigger election at startup if single node .. RaftConsensus: Trigger election at startup if single node If a tablet's replication factor is

[kudu-CR] Add more helpful CHECK message at master startup

2016-06-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add more helpful CHECK message at master startup .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3396 To unsubscribe,

[kudu-CR] Add more helpful CHECK message at master startup

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add more helpful CHECK message at master startup .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1845/ -- To view, visit

[kudu-CR] Add more helpful CHECK message at master startup

2016-06-16 Thread Mike Percy (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3396 to review the following change. Change subject: Add more helpful CHECK message at master startup .. Add more

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add Raft remote bootstrap design doc .. Patch Set 2: See the rendered page at

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has uploaded a new patch set (#2). Change subject: Add Raft remote bootstrap design doc .. Add Raft remote bootstrap design doc This was ported over from a Google doc Change-Id:

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add Raft remote bootstrap design doc .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1844/ -- To view, visit

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add Raft remote bootstrap design doc .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1843/ -- To view, visit

[kudu-CR] Link to raft config change design from design-docs index page

2016-06-16 Thread Mike Percy (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3394 to review the following change. Change subject: Link to raft config change design from design-docs index page .. Link

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-16 Thread Mike Percy (Code Review)
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3395 to review the following change. Change subject: Add Raft remote bootstrap design doc .. Add Raft remote bootstrap

[kudu-CR] Remove the LocalConsensus implementation

2016-06-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Remove the LocalConsensus implementation .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/3350/5/src/kudu/consensus/metadata.proto File

[kudu-CR] Use RaftConsensus instead of LocalConsensus in tests

2016-06-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Use RaftConsensus instead of LocalConsensus in tests .. Patch Set 4: (3 comments)

[kudu-CR] RaftConsensus: Trigger election at startup if single node

2016-06-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: RaftConsensus: Trigger election at startup if single node .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3344 To

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

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: docs: informal design for handling permanent master failures .. Patch Set 1: (15 comments) This is sounding more formal, let's just call it a real design doc :)

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] use tablet cache for locateTablet calls .. Patch Set 3: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1841/ -- To view, visit

[kudu-CR] RaftConsensus: Trigger election at startup if single node

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: RaftConsensus: Trigger election at startup if single node .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/1840/ -- To view, visit

[kudu-CR] RaftConsensus: Trigger election at startup if single node

2016-06-16 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3344 to look at the new patch set (#5). Change subject: RaftConsensus: Trigger election at startup if single node ..

[kudu-CR] RaftConsensus: Trigger election at startup if single node

2016-06-16 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: RaftConsensus: Trigger election at startup if single node .. Patch Set 4: (2 comments)

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

2016-06-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: docs: informal design for handling permanent master failures .. Patch Set 1: Rendered content available here:

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

2016-06-16 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Mike Percy, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3393 to review the following change. Change subject: docs: informal design for handling permanent master failures

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-16 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java-client] use tablet cache for locateTablet calls .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3386/2//COMMIT_MSG Commit Message: PS2,

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-16 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java-client] use tablet cache for locateTablet calls .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1837/ -- To view, visit

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-16 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3386 to look at the new patch set (#3). Change subject: [java-client] use tablet cache for locateTablet calls

[kudu-CR] RaftConsensus: Trigger election at startup if single node

2016-06-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: RaftConsensus: Trigger election at startup if single node .. Patch Set 4: (2 comments)

[kudu-CR] Add required Debian version to installation page

2016-06-16 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add required Debian version to installation page .. Patch Set 1: Ah, so we should probably hold of on this till we can verify that it works. Or we should remove

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java-client] use tablet cache for locateTablet calls .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-1309: [java client] support tables with non-covering partition-key ranges

2016-06-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1309: [java client] support tables with non-covering partition-key ranges .. Patch Set 2: (3 comments)

[kudu-CR] [java-client] use tablet cache for locateTablet calls

2016-06-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] use tablet cache for locateTablet calls .. Patch Set 2: (2 comments)