[kudu-CR](gh-pages) Add 0.9.0 release and refresh docs

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add 0.9.0 release and refresh docs .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3355 To unsubscribe, visit http://gerrit.clouder

[kudu-CR](gh-pages) Blog post for 0.9.0 release

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Blog post for 0.9.0 release .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3357/2/_posts/2016-06-10-apache-kudu-0-9-0-released.md File _posts/2016

[kudu-CR] Add WaitUntilLeader() to Consensus interface

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Add WaitUntilLeader() to Consensus interface .. Patch Set 2: Verified+1 Overriding Jenkins due to super flaky Java test org.kududb.client.TestAsyncKuduClient.testD

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

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Use RaftConsensus instead of LocalConsensus in tests .. Patch Set 2: Verified+1 Overriding Jenkins due to super flaky Java test org.kududb.client.TestAsyncKuduClie

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

2016-06-09 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 2: Verified+1 Overriding Jenkins due to super flaky Java test org.kududb.client.TestAsyncKud

[kudu-CR] Prepare some tests for RaftConsensus

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: Prepare some tests for RaftConsensus .. Abandoned squashing this one. hard to do it separate -- To view, visit http://gerrit.cloudera.org:8080/3348 To unsubscribe, visit h

[kudu-CR] Convert alter table-test to use RaftConsensus

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: Convert alter_table-test to use RaftConsensus .. Abandoned squashing this. hard to do it separate -- To view, visit http://gerrit.cloudera.org:8080/3347 To unsubscribe, vi

[kudu-CR] Remove the LocalConsensus implementation

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

[kudu-CR] Remove the LocalConsensus implementation

2016-06-09 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 (#3). Change subject: Remove the LocalConsensus implementation .. Remove the LocalCons

[kudu-CR] Always write last known addr into ConsensusMetadata

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: Always write last_known_addr into ConsensusMetadata .. Abandoned abandoning this one -- To view, visit http://gerrit.cloudera.org:8080/3349 To unsubscribe, visit http://ge

[kudu-CR] KUDU-1469. Fix handling of fully-deduped requests after a leader change

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1469. Fix handling of fully-deduped requests after a leader change .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3228/4/src/kudu/consensus/

[kudu-CR] [java client] Fix a race in TabletClient cleanup

2016-06-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] Fix a race in TabletClient cleanup .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/1807/ -- To view, visit http://gerrit.cloud

[kudu-CR] [java client] Fix a race in TabletClient cleanup

2016-06-09 Thread Jean-Daniel Cryans (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3340 to look at the new patch set (#3). Change subject: [java client] Fix a race in TabletClient cleanup ..

[kudu-CR] [java client] Fix a race in TabletClient cleanup

2016-06-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Fix a race in TabletClient cleanup .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3340/2/java/kudu-client/src/main/java/org

[kudu-CR] Do not run (g)addr2line translator on MacOS X

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Do not run (g)addr2line translator on MacOS X .. Patch Set 1: (2 comments) My understanding is that sanitizer-enabled Kudu builds on Mac OS will report correct li

[kudu-CR] Do not run (g)addr2line translator on MacOS X

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Do not run (g)addr2line translator on MacOS X .. Patch Set 1: Code-Review+1 lgtm but Dan should take a look too, since he develops on a Mac -- To view, visit http

[kudu-CR] Do not run (g)addr2line translator on MacOS X

2016-06-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Do not run (g)addr2line translator on MacOS X .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1806/ -- To view, visit http://gerrit.cloudera

[kudu-CR] Do not run (g)addr2line translator on MacOS X

2016-06-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/3360 Change subject: Do not run (g)addr2line translator on MacOS X .. Do not run (g)addr2line translator on MacOS X When running tests on Mac

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. KUDU-1473: fix some tablet lock usage in CatalogManager This was probably due to the refactoring

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 7: Verified+1 Overriding Jenkins, another Java testDisconnect failure. -- To view, visit http

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3309 To unsubscribe, visit

[kudu-CR] Prepare some tests for RaftConsensus

2016-06-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Prepare some tests for RaftConsensus .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1805/ -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] Remove the LocalConsensus implementation

2016-06-09 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 (#2). Change subject: Remove the LocalConsensus implementation .. Remove the LocalCons

[kudu-CR] Remove the LocalConsensus implementation

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

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

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

[kudu-CR] Add WaitUntilLeader() to Consensus interface

2016-06-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add WaitUntilLeader() to Consensus interface .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1802/ -- To view, visit http://gerrit.cloudera.

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

2016-06-09 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 2: Build Started http://104.196.14.100/job/kudu-gerrit/1803/ -- To view, visit http://ger

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

2016-06-09 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 (#2). Change subject: RaftConsensus: Trigger election at startup if single node .. Raf

[kudu-CR] Convert alter table-test to use RaftConsensus

2016-06-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Convert alter_table-test to use RaftConsensus .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1800/ -- To view, visit http://gerrit.cloudera

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-09 Thread Adar Dembo (Code Review)
Hello Jean-Daniel Cryans, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3309 to look at the new patch set (#7). Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager ..

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/1799/ -- To view, visit http://gerri

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 6: (1 comment) > (1 comment) > > I wish there was an easier way to hide the data object so

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 6: (1 comment) I wish there was an easier way to hide the data object so that we can only acc

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: KUDU-1353: remove per-tablet replica locations cache .. KUDU-1353: remove per-tablet replica locations cache This is the simpler alternative to rebuilding

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2887 To unsubscribe, visit ht

[kudu-CR](gh-pages) Add 0.9.0 release and refresh docs

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add 0.9.0 release and refresh docs .. Patch Set 1: Code-Review+2 (shipit) -- To view, visit http://gerrit.cloudera.org:8080/3355 To unsubscribe, visit http://gerr

[kudu-CR](gh-pages) Blog post for 0.9.0 release

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Blog post for 0.9.0 release .. Patch Set 2: Code-Review+2 (shipit) -- To view, visit http://gerrit.cloudera.org:8080/3357 To unsubscribe, visit http://gerrit.clou

[kudu-CR](gh-pages) Blog post for 0.9.0 release

2016-06-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#2). Change subject: Blog post for 0.9.0 release .. Blog post for 0.9.0 release Change-Id: I70b4be74e350232110e580844d4f71e012fa813f --- A _posts/2016-06-10-apache-kudu-0-9-0-rele

[kudu-CR](gh-pages) Add 0.9.0 release and refresh docs

2016-06-09 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Add 0.9.0 release and refresh docs .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3355 To unsubscribe, visit http://gerri

[kudu-CR](gh-pages) Blog post for 0.9.0 release

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Blog post for 0.9.0 release .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3357/1/_posts/2016-06-10-apache-kudu-0-9-0-released.md File _posts/2016

[kudu-CR](gh-pages) Blog post for 0.9.0 release

2016-06-09 Thread Misty Stanley-Jones (Code Review)
Misty Stanley-Jones has posted comments on this change. Change subject: Blog post for 0.9.0 release .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3357/1/_posts/2016-06-10-apache-kudu-0-9-0-released.md File _

[kudu-CR](gh-pages) Blog post for 0.9.0 release

2016-06-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Blog post for 0.9.0 release .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3357/1/_posts/2016-06-10-apache-kudu-0-9-0-released.md File _po

[kudu-CR](gh-pages) Blog post for 0.9.0 release

2016-06-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3357 Change subject: Blog post for 0.9.0 release .. Blog post for 0.9.0 release Change-Id: I70b4be74e350232110e580844d4f71e012fa813f ---

[kudu-CR](gh-pages) Add 0.9.0 release and refresh docs

2016-06-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3355 Change subject: Add 0.9.0 release and refresh docs .. Add 0.9.0 release and refresh docs Change-Id: Ib04c111bd40181f6cbe3978dc87d65

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

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities. .. Patch Set 7: > The main reason for not using unordered_set was because the > collection w

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

2016-06-09 Thread Sameer Abhyankar (Code Review)
Sameer Abhyankar has posted comments on this change. Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities. .. Patch Set 7: (4 comments) Hi Adar - The latest PS should have the additional change

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

2016-06-09 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 (#7). Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities. ..

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

2016-06-09 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 7: Build Started http://104.196.14.100/job/kudu-gerrit/1798/ -- To view, vis

[kudu-CR] [java client] Fix a race in TabletClient cleanup

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Fix a race in TabletClient cleanup .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3340/2/java/kudu-client/src/main/java/org/kududb/

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

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities. .. Patch Set 6: (5 comments) Did you see my high-level comment about std::unordered_set vs.

[kudu-CR] Always write last known addr into ConsensusMetadata

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Always write last_known_addr into ConsensusMetadata .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3349/1/src/kudu/master/sys_catalog.cc File src

[kudu-CR] Update documentation

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: Update documentation .. Update documentation Added 'unzip' package and the providers of `lsb_release` command required by enable_devtoolset.sh, for all 3 d

[kudu-CR] Update documentation

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Update documentation .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3351 To unsubscribe, visit http://gerrit.cloudera.org:8080/set

[kudu-CR] KUDU-1473: fix some tablet lock usage in CatalogManager

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1473: fix some tablet lock usage in CatalogManager .. Patch Set 6: Verified+1 Another Java testDisconnect failure. -- To view, visit http://gerrit.cloudera.o

[kudu-CR] KUDU-1353: remove per-tablet replica locations cache

2016-06-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1353: remove per-tablet replica locations cache .. Patch Set 11: Another testDisconnect Java failure. -- To view, visit http://gerrit.cloudera.org:8080/2887

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

2016-06-09 Thread Sameer Abhyankar (Code Review)
Sameer Abhyankar has posted comments on this change. Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities. .. Patch Set 6: (20 comments) Thanks for the review Adar. All the additional changes y

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

2016-06-09 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 (#6). Change subject: KUDU-1363: Add in-list predicates for extracting a set of equalities. ..

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

2016-06-09 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 6: Build Started http://104.196.14.100/job/kudu-gerrit/1797/ -- To view, vis

[kudu-CR] Update documentation

2016-06-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Update documentation .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/1796/ -- To view, visit http://gerrit.cloudera.org:8080/3351 To unsubsc

[kudu-CR] Update documentation

2016-06-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Update documentation .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/1795/ -- To view, visit http://gerrit.cloudera.org:8080/3351 To unsubsc

[kudu-CR] Update documentation

2016-06-09 Thread Cenk Uyan (Code Review)
Cenk Uyan has uploaded a new change for review. http://gerrit.cloudera.org:8080/3351 Change subject: Update documentation .. Update documentation Added 'unzip' package and the providers of `lsb_release` command required by en

[kudu-CR] Update documentation and build helper script

2016-06-09 Thread Cenk Uyan (Code Review)
Cenk Uyan has abandoned this change. Change subject: Update documentation and build helper script .. Abandoned will resubmit -- To view, visit http://gerrit.cloudera.org:8080/3336 To unsubscribe, visit http://gerrit.cloudera.