[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, vis

[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 http://gerrit.cloudera.org:8080/339

[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 3.5

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

2016-06-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: build: squelch symbol visibility warnings seen with cmake 3.5 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3405 To unsubscribe,

[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 .. build

[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 http:/

[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 sema

[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 http://gerrit.cloudera.org:80

[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 http://gerrit.cloudera.org:80

[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 sema

[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-2 -- To view, visit http://gerrit.cloudera.org:8080/3389 To unsubscribe, visit http:/

[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. http://gerrit.cloudera.org:8080/#/c/2593/15/src/kudu/cf

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

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

[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 per

[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. http://

[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 http://gerrit.cloudera.org:80

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

2016-06-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/3403 Change subject: WIP: Exactly once semantics for writes .. WIP: Exactly once semantics for writes Not for review, just getting jenk

[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 http

[kudu-CR] docs: 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: design for handling permanent master failures .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1861/ -- To view, visit http://gerrit.cl

[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: des

[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) http://gerrit.cloudera.org:8080/#/c/3393/1/docs/design-docs/master-perm-

[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 do

[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, s/quorum/configurat

[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 ht

[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 do

[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 http://gerrit.

[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 do

[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) http://gerrit.cloudera.org:8080/#/c/3399/3/docs/design-docs/raft-config-cha

[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: I3551e8fcd19628cfbeb25f822a

[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 http://gerrit.cloud

[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 http:/

[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 Rev

[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 http

[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 http://gerri

[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 http://104.196.14.100/job/kudu-gerrit/1856

[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 http://ge

[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 http://gerrit.

[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, vi

[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 http://gerrit.cloudera.org:8080

[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, vis

[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 http://