[kudu-CR] KUDU-1363: Add in-list predicates for extracting a set of equalities.

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities. .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/1867/ -- To view,

[kudu-CR] KUDU-1363: Add in-list predicates for extracting a set of equalities.

2016-06-17 Thread Sameer Abhyankar (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2986 to look at the new patch set (#8). Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities.

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

2016-06-17 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: docs: design for handling permanent master failures .. Patch Set 2: Mind updating the rendered HTML version? -- To view, visit

[kudu-CR] build: squelch symbol visibility warnings seen with cmake 3.5

2016-06-17 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: build: squelch symbol visibility warnings seen with cmake 3.5 .. build: squelch symbol visibility warnings seen with cmake 3.5 When building with cmake

[kudu-CR] build: squelch symbol visibility warnings seen with cmake 3.5

2016-06-17 Thread Adar Dembo (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3405 to review the following change. Change subject: build: squelch symbol visibility warnings seen with cmake 3.5 ..

[kudu-CR] build: squelch symbol visibility warnings seen with cmake 3.5

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: build: squelch symbol visibility warnings seen with cmake 3.5 .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1866/ -- To view, visit

[kudu-CR] WIP: Exactly once semantics for writes

2016-06-17 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3403 to look at the new patch set (#3). Change subject: WIP: Exactly once semantics for writes .. WIP: Exactly once

[kudu-CR] WIP: Exactly once semantics for writes

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Exactly once semantics for writes .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1865/ -- To view, visit

[kudu-CR] WIP: Exactly once semantics for writes

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Exactly once semantics for writes .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1864/ -- To view, visit

[kudu-CR] WIP: Exactly once semantics for writes

2016-06-17 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3403 to look at the new patch set (#2). Change subject: WIP: Exactly once semantics for writes .. WIP: Exactly once

[kudu-CR] Persistent cache support for NVM

2016-06-17 Thread Sarah Jelinek (Code Review)
Sarah Jelinek has posted comments on this change. Change subject: Persistent cache support for NVM .. Patch Set 15: (28 comments) Fixed all comments. Questions answered.

[kudu-CR] Update Java client for new master GetTableLocations semantics

2016-06-17 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Update Java client for new master GetTableLocations semantics .. Update Java client for new master GetTableLocations semantics The Java client was

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

2016-06-17 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java-client] use tablet cache for locateTablet calls .. [java-client] use tablet cache for locateTablet calls This simplifies and improves the

[kudu-CR] Add implementation of the block cache on NVM running in persistent mode.

2016-06-17 Thread Sarah Jelinek (Code Review)
Sarah Jelinek has posted comments on this change. Change subject: Add implementation of the block cache on NVM running in persistent mode. .. Patch Set 8: (6 comments) Addressed all comments. Will push changes today.

[kudu-CR] WIP: Exactly once semantics for writes

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: Exactly once semantics for writes .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1863/ -- To view, visit

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

2016-06-17 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 5: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1862/ -- To view, visit

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

2016-06-17 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 (#2). Change subject: docs: design for handling permanent master failures .. docs:

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

2016-06-17 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: (15 comments)

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

2016-06-17 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 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3399/3/docs/design-docs/raft-config-change.md File

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

2016-06-17 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Reformat raft-config-change.md to clean it up .. Reformat raft-config-change.md to clean it up It needed a bit of markdown love. Also,

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

2016-06-17 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 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3386 To unsubscribe, visit

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

2016-06-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Reformat raft-config-change.md to clean it up .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3399/3/docs/design-docs/raft-config-change.md File

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

2016-06-17 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 5: Build Started http://104.196.14.100/job/kudu-gerrit/1860/ -- To view, visit

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

2016-06-17 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Adar Dembo, 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 (#5). Change subject: [java-client] use tablet cache for locateTablet calls

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

2016-06-17 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 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3399/3/docs/design-docs/raft-config-change.md File

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

2016-06-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Reformat raft-config-change.md to clean it up .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-17 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. 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-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add Raft remote bootstrap design doc .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3395 To unsubscribe, visit

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

2016-06-17 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: Code-Review-1 -- To view, visit http://gerrit.cloudera.org:8080/3389 To unsubscribe, visit

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

2016-06-17 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add blog post about removing LocalConsensus .. Add blog post about removing LocalConsensus Change-Id: I571de0cbf4d73511f17bd57d374fed9e8ca302e4

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

2016-06-17 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 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3398 To unsubscribe, visit

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

2016-06-17 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: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1854/ -- To view, visit

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

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1309: [java client] support tables with non-covering partition-key ranges .. Patch Set 2: -Verified Build Started

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

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add required Debian version to installation page .. Patch Set 1: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1858/ -- To view, visit

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

2016-06-17 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 4: Build Started http://104.196.14.100/job/kudu-gerrit/1855/ -- To view, visit

[kudu-CR] Update Java client for new master GetTableLocations semantics

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Update Java client for new master GetTableLocations semantics .. Patch Set 4: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1859/ -- To view,

[kudu-CR] Add Raft remote bootstrap design doc

2016-06-17 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/1857/ -- To view, visit

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

2016-06-17 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: docs: informal design for handling permanent master failures .. Patch Set 1: -Verified Build Started http://104.196.14.100/job/kudu-gerrit/1853/ -- To view,

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

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

[kudu-CR] Remove the LocalConsensus implementation

2016-06-17 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Remove the LocalConsensus implementation .. Remove the LocalConsensus implementation This implementation was intended as scaffolding while we built a true

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

2016-06-17 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: Use RaftConsensus instead of LocalConsensus in tests .. Use RaftConsensus instead of LocalConsensus in tests This paves the way to remove the

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

2016-06-17 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add blog post about removing LocalConsensus .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3398 To unsubscribe, visit