Re: 1.0.0 -> 0.10.0?

2016-07-14 Thread Jean-Daniel Cryans
dea of having a bit more time to shake > any > >> last bugs out before the big release. > >> > >> - Dan > >> > >> On Fri, Jul 8, 2016 at 8:05 AM, Jean-Daniel Cryans > > >> wrote: > >> > >> > Hey devs, > >> > &

1.0.0 -> 0.10.0?

2016-07-08 Thread Jean-Daniel Cryans
Hey devs, 5 months ago I wrote up this plan[1] about release cadence and 1.0, and we've been sticking pretty close to it. After 0.9.0 I bumped up the version to 1.0.0 assuming it'd be the next release, but on second thought this might have been a mistake since it doesn't allow for any other interm

Re: [VOTE] Graduate Apache Kudu as a TLP (v2)

2016-07-01 Thread Jean-Daniel Cryans
u Project; and be it further > > RESOLVED, that the persons listed immediately below be and > hereby are appointed to serve as the initial members of the > Apache Kudu Project: > > * Adar Dembo > * Binglin Chang > * Chris Mattman > * Dan Burkert > * David Alve

Re: [VOTE] Graduate Apache Kudu as a TLP

2016-07-01 Thread Jean-Daniel Cryans
of the projects within the scope of > responsibility of the Apache Kudu Project; and be it further > > RESOLVED, that the persons listed immediately below be and > hereby are appointed to serve as the initial members of the > Apache Kudu Project: > > * Adar Dembo > * A

Re: [VOTE] Apache Kudu (incubating) 0.9.1 RC1

2016-06-23 Thread Jean-Daniel Cryans
+1 I downloaded the tarball, checked the signature, checked the hashes, built Kudu on centos 6.6, ran all the unit tests. J-D On Thu, Jun 23, 2016 at 2:15 PM, Todd Lipcon wrote: > Hi, > > We're happy to announce the first release candidate for Apache Kudu > (incubating) 0.9.1. > > This is a bu

Re: Making an 0.9.1 release?

2016-06-22 Thread Jean-Daniel Cryans
+1 I think we could grab https://github.com/cloudera/kudu/commit/5f7d79910d8da492195b5c22a6d9a7911f950107 and https://github.com/cloudera/kudu/commit/679c6b2c93313bd4371916678b4ea8f52b621a50 that fix issues committed in the master for 0.9.0. Adar can confirm if it's desirable. This Java client fi

[ANNOUNCE] Apache Kudu (incubating) 0.9.0 released

2016-06-10 Thread Jean-Daniel Cryans
The Apache Kudu (incubating) team is happy to announce the release of Kudu 0.9.0! Kudu is an open source storage engine for structured data which supports low-latency random access together with efficient analytical access patterns. It is designed within the context of the Apache Hadoop ecosystem

[RESULT] [VOTE] Apache Kudu (incubating) 0.9.0 RC1

2016-06-06 Thread Jean-Daniel Cryans
The vote is now closed. It passes with: 4 binding +1s: Adar D, Misty SJ, J-D C, Chris M 0 -1s of any nature. I will now be taking this up to the IPMC on the general@ list. Thanks! J-D On Wed, Jun 1, 2016 at 7:09 PM, Jean-Daniel Cryans wrote: > Hi, > > We're happy to anno

Re: [VOTE] Apache Kudu (incubating) 0.9.0 RC1

2016-06-06 Thread Jean-Daniel Cryans
+1, I downloaded the tarball on a CentOS 6.4 machine, made sure the files looked ok, compiled the Java client, made sure the md5 matched. J-D On Wed, Jun 1, 2016 at 7:09 PM, Jean-Daniel Cryans wrote: > Hi, > > We're happy to announce the first release candidate for Apache Kudu

Re: [VOTE] Apache Kudu (incubating) 0.9.0 RC1

2016-06-03 Thread Jean-Daniel Cryans
t something wrong > with Ubuntu 16.04. > > On Fri, Jun 3, 2016 at 2:40 AM, Todd Lipcon wrote: > > I can take a look at the logs later today London-time. Adar, any chance > you > > can determine if the failures are a regression since 0.8 on your machine? > > > &g

Re: [VOTE] Apache Kudu (incubating) 0.9.0 RC1

2016-06-02 Thread Jean-Daniel Cryans
; > Dload Upload Total SpentLeft > Speed > > 10069 100690 0267 0 --:--:-- --:--:-- > --:--:-- 268 > > LMC-053601:apache-kudu-0.9.0-RC1 mattmann$ $HOME/bin/verify_gpg_sigs > > Verifying Signature for file ap

[VOTE] Apache Kudu (incubating) 0.9.0 RC1

2016-06-01 Thread Jean-Daniel Cryans
Hi, We're happy to announce the first release candidate for Apache Kudu (incubating) 0.9.0. We are doing time-based releases leading up to 1.0.0 towards the end of the summer, with one 0.x.0 release every two months. The is a source-only release. The artifacts were staged here: https://dist.apac

Re: Change packaging from org.kududb to org.apache.kudu?

2016-06-01 Thread Jean-Daniel Cryans
> package name changes were, by definition, breaking changes. > > > > > > > > > > > > On Tue, May 31, 2016 at 10:48 AM, Dan Burkert > wrote: > > > > > > > I'm +1 to changing the package prior to 1.0. Sooner the

Last call for 0.9.0

2016-05-31 Thread Jean-Daniel Cryans
Hey devs, I'm wrapping up 0.9.0 today in order to put up RC1 for vote tomorrow or Thursday. Here's a list of patches that I want to get in: - Release notes (everyone should review them!): http://gerrit.cloudera.org:8080/#/c/3176/ - NPE fix: http://gerrit.cloudera.org:8080/#/c/3243/ Then there ar

Re: Change packaging from org.kududb to org.apache.kudu?

2016-05-31 Thread Jean-Daniel Cryans
ones. Nasty. > > > > On Tue, May 31, 2016 at 10:48 AM, Dan Burkert wrote: > > > I'm +1 to changing the package prior to 1.0. Sooner the better, in my > > opinion. I'm -0 to trying to do it in a backwards compatible way, I > don't > > think t

Change packaging from org.kududb to org.apache.kudu?

2016-05-31 Thread Jean-Daniel Cryans
Hey devs, I'd like to start a discussion around package names. Right now our packages are prefixed with "org.kududb", a domain that AFAIK is owned by Cloudera with a plan to transfer it to the ASF. In a perfect world they should be named "org.apache.kudu" but it's not a requirement. Many projects

Re: 0.9.0 branched

2016-05-25 Thread Jean-Daniel Cryans
Yeah that's good you caught it! +1 for the cherry-pick. On Wed, May 25, 2016 at 12:18 AM, Todd Lipcon wrote: > Another blocker that needs to be incorporated: > http://gerrit.cloudera.org:8080/#/c/3208/ > > On Tue, May 24, 2016 at 10:14 AM, Jean-Daniel Cryans > wrote: > &

Re: 0.9.0 branched

2016-05-24 Thread Jean-Daniel Cryans
gt; flakiness on client-test: https://issues.apache.org/jira/browse/KUDU-1466 > . > I'm hoping this will be a pretty short fix of just a few lines of code, in > which case I think it's a good candidate for 0.9.0. > > -Todd > > > > On Mon, May 23, 2016 at 6:48

[kudu-CR] catalog_manager: move VLOG(1) messages above Async RPC calls to avoid crash

2016-05-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: catalog_manager: move VLOG(1) messages above Async RPC calls to avoid crash .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org

[kudu-CR] java: drop verbose level to --v=1, skip fsync for Java tests

2016-05-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: drop verbose level to --v=1, skip fsync for Java tests .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3172 To

[kudu-CR] java: drop verbose level to --v=1, skip fsync for Java tests

2016-05-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: drop verbose level to --v=1, skip fsync for Java tests .. Patch Set 2: > Seems like these tests are more flaky lately. Any idea what's goi

Re: 0.9.0 branched

2016-05-23 Thread Jean-Daniel Cryans
th need more work but I think in both cases it's doable this week. > > Mike > > On Mon, May 23, 2016 at 2:59 PM, Jean-Daniel Cryans > wrote: > > > I'll send the list of pre-authorized patches/jiras that can go in that > > branch later today, else please ping me on this thread or add me on the > > review. > > > > J-D > > >

0.9.0 branched

2016-05-23 Thread Jean-Daniel Cryans
I'll send the list of pre-authorized patches/jiras that can go in that branch later today, else please ping me on this thread or add me on the review. J-D

[kudu-CR] Bump the master branch's version to 1.0.0-SNAPSHOT

2016-05-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3178 Change subject: Bump the master branch's version to 1.0.0-SNAPSHOT .. Bump the master branch's version to 1.0.0-SNAPSHOT

[kudu-CR](gh-pages) Add weekly update for 5/23

2016-05-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 5/23 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3175 To unsubscribe, visit http

[kudu-CR] java: drop verbose level to --v=1, skip fsync for Java tests

2016-05-23 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: java: drop verbose level to --v=1, skip fsync for Java tests .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3172 To

0.9.0 branching today at 2:30 PM PDT

2016-05-23 Thread Jean-Daniel Cryans
I'll send another email when it's done.

Pre-boarding announcement for Kudu 0.9.0

2016-05-19 Thread Jean-Daniel Cryans
Hello Kudu devs, June 1st is approaching fast, so it's time to branch for 0.9.0! I know there's a lot of things in flight right now, so I'm giving you until May 22nd before branching and imposing restriction. I will send my next email before branching. Thanks, J-D

[kudu-CR] Remove default table partitioning

2016-05-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Remove default table partitioning .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3131/2/docs/release_notes.adoc File docs

[kudu-CR] Demote rpc sampling log message to VLOG(1)

2016-05-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Demote rpc sampling log message to VLOG(1) .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3144 To unsubscribe, visit http

[kudu-CR] python: add support for Upsert

2016-05-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: python: add support for Upsert .. python: add support for Upsert This also adds a pytest.ini file which explicitly sets the root path for tests. I

[kudu-CR] python: add support for Upsert

2016-05-19 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: python: add support for Upsert .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3128 To unsubscribe, visit http

[kudu-CR] [java client] RowResult.rowToString doesn't handle BOOL

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] RowResult.rowToString doesn't handle BOOL .. [java client] RowResult.rowToString doesn't handle BOOL

[kudu-CR] [java client] Add Upsert

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Add Upsert .. Patch Set 1: Forgot to add Upsert... -- To view, visit http://gerrit.cloudera.org:8080/3123 To unsubscribe, visit http

[kudu-CR] [java client] Add Upsert

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
eType: newpatchset Gerrit-Change-Id: I51112981cc82146802e8484000a6b74dd724116e Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon

[kudu-CR] KUDU-1002. Add support for UPSERT

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
beiro Alves Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy

[kudu-CR] [java client] Add Upsert

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3123 Change subject: [java client] Add Upsert .. [java client] Add Upsert Change-Id: I51112981cc82146802e8484000a6b74dd724116e --- M

[kudu-CR] [java client] RowResult.rowToString doesn't handle BOOL

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3122 Change subject: [java client] RowResult.rowToString doesn't handle BOOL .. [java client] RowResult.rowToString doesn't h

[kudu-CR] [java client] Get the server logs in test logs

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Get the server logs in test logs .. [java client] Get the server logs in test logs Right now, MiniKuduCluster sends the server logs

[kudu-CR] [java client] Get the server logs in test logs

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
13 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Icc18fbd700871dc21ef6b915c2001302067b364d Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer:

[kudu-CR] [java client] Get the server logs in test logs

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Get the server logs in test logs .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/3113/1/java/kudu-client/src/test/java/org

[kudu-CR] [java client] Get the server logs in test logs

2016-05-18 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3113 Change subject: [java client] Get the server logs in test logs .. [java client] Get the server logs in test logs Right now

[kudu-CR](gh-pages) Add 5/16 weekly update

2016-05-16 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add 5/16 weekly update .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3089 To unsubscribe, visit http

[kudu-CR] RFC [java client] Redo how we manage exceptions

2016-05-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: RFC [java client] Redo how we manage exceptions .. Patch Set 1: (3 comments) > (3 comments) > > I'm generally a fan of inserting S

[kudu-CR] RFC [java client] Redo how we manage exceptions

2016-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3055 Change subject: RFC [java client] Redo how we manage exceptions .. RFC [java client] Redo how we manage exceptions Right now the

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Allow scanners to have projections with out-of-order keys .. [java client] Allow scanners to have projections with out-of-order keys

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-12 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Allow scanners to have projections with out-of-order keys .. Patch Set 4: Verified+1 Not sure what's going on with TSAN but I don&#x

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Allow scanners to have projections with out-of-order keys .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3015/3/java/kudu

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-11 Thread Jean-Daniel Cryans (Code Review)
errit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-11 Thread Jean-Daniel Cryans (Code Review)
errit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon

[kudu-CR] [java client] fix ClassCastException: DeferredGroupException cannot be cast to ArrayList

2016-05-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] fix ClassCastException: DeferredGroupException cannot be cast to ArrayList .. [java client] fix ClassCastException

[kudu-CR] [java client] fix ClassCastException: DeferredGroupException cannot be cast to ArrayList

2016-05-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] fix ClassCastException: DeferredGroupException cannot be cast to ArrayList .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c

[kudu-CR] [java client] fix ClassCastException: DeferredGroupException cannot be cast to ArrayList

2016-05-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] fix ClassCastException: DeferredGroupException cannot be cast to ArrayList .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-11 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Allow scanners to have projections with out-of-order keys .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/3015/2/java/kudu

[kudu-CR] [java client] fix ClassCastException: DeferredGroupException cannot be cast to ArrayList

2016-05-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3020 Change subject: [java client] fix ClassCastException: DeferredGroupException cannot be cast to ArrayList .. [java client] fix

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-10 Thread Jean-Daniel Cryans (Code Review)
ge-Id: I550cb255460bb54815d3a30eb973f8c143573e62 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Dan Burkert Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Allow scanners to have projections with out-of-order keys .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3015/1/java/kudu

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Allow scanners to have projections with out-of-order keys .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3015/1/java/kudu

[kudu-CR] [java client] Allow scanners to have projections with out-of-order keys

2016-05-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3015 Change subject: [java client] Allow scanners to have projections with out-of-order keys .. [java client] Allow scanners to have

[kudu-CR](gh-pages) Add weekly update for 5/9

2016-05-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: Add weekly update for 5/9 .. Add weekly update for 5/9 Change-Id: I387c45024fe0308cc8d4c19a56b964d4da4e8f6d Reviewed-on: http

[kudu-CR](gh-pages) Add weekly update for 5/9

2016-05-10 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 5/9 .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3010 To unsubscribe, visit http

[kudu-CR](gh-pages) Add weekly update for 5/9

2016-05-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 5/9 .. Patch Set 3: I missed one important contrib. -- To view, visit http://gerrit.cloudera.org:8080/3010 To unsubscribe, visit

[kudu-CR](gh-pages) Add weekly update for 5/9

2016-05-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#3). Change subject: Add weekly update for 5/9 .. Add weekly update for 5/9 Change-Id: I387c45024fe0308cc8d4c19a56b964d4da4e8f6d --- A _posts/2016-05-09-weekly-update.md 1 file

[kudu-CR](gh-pages) Add weekly update for 5/9

2016-05-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#2). Change subject: Add weekly update for 5/9 .. Add weekly update for 5/9 Change-Id: I387c45024fe0308cc8d4c19a56b964d4da4e8f6d --- A _posts/2016-05-09-weekly-update.md 1 file

[kudu-CR](gh-pages) Add weekly update for 5/9

2016-05-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Add weekly update for 5/9 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3010/1/_posts/2016-05-09-weekly-update.md File _posts/2016-05-09

[kudu-CR](gh-pages) Add weekly update for 5/9

2016-05-09 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/3010 Change subject: Add weekly update for 5/9 .. Add weekly update for 5/9 Change-Id: I387c45024fe0308cc8d4c19a56b964d4da4e8f6d --- A

[kudu-CR] change scope of jdiff to provided

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: change scope of jdiff to provided .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2874 To unsubscribe, visit http

[kudu-CR] change scope of jdiff to provided

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: change scope of jdiff to provided .. change scope of jdiff to provided Previously jdiff was in a compile scope, which was leaking into transitive

[kudu-CR] [java client] Fix: ignoring option MaxPermSize=100m

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Fix: ignoring option MaxPermSize=100m .. [java client] Fix: ignoring option MaxPermSize=100m The full line: Server VM warning

[kudu-CR] [java client] Disconnected TabletClient can double retry RPCs

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Disconnected TabletClient can double retry RPCs .. [java client] Disconnected TabletClient can double retry RPCs Noticed on Jenkins

[kudu-CR] [java client] Add a method to return rows in order

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Add a method to return rows in order .. [java client] Add a method to return rows in order This patch adds an unstable package

[kudu-CR] KUDU-815. Improve performance of first scan following restart

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-815. Improve performance of first scan following restart .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2974 To

[kudu-CR] Short-circuit CheckRowDeleted if there are no deletes in DeltaFile

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Short-circuit CheckRowDeleted if there are no deletes in DeltaFile .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2975

[kudu-CR] [java client] Add a method to return rows in order

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Add a method to return rows in order .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2951 To unsubscribe

[kudu-CR] [java client] Add a method to return rows in order

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Add a method to return rows in order .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/2951/5/java/kudu-client/src/main/java

[kudu-CR] [java client] Fix: ignoring option MaxPermSize=100m

2016-05-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Fix: ignoring option MaxPermSize=100m .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2977/1/java/pom.xml File java/pom.xml

[kudu-CR] [java client] Fix: ignoring option MaxPermSize=100m

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/2977 Change subject: [java client] Fix: ignoring option MaxPermSize=100m .. [java client] Fix: ignoring option MaxPermSize=100m The

[kudu-CR] [java client] Disconnected TabletClient can double retry RPCs

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/2978 Change subject: [java client] Disconnected TabletClient can double retry RPCs .. [java client] Disconnected TabletClient can double

[kudu-CR] [java client] scanNextRows() needs to always increment attempts

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] scanNextRows() needs to always increment attempts .. Patch Set 2: Verified+1 Forcing +1 due to leakage between reviews somehow failing the

[kudu-CR] [java client] scanNextRows() needs to always increment attempts

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] scanNextRows() needs to always increment attempts .. [java client] scanNextRows() needs to always increment attempts This bug was

[kudu-CR] Bump guava version to 18.0

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump guava version to 18.0 .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/2972/3//COMMIT_MSG Commit Message: Line 14: The

[kudu-CR] [java-client] make AsyncKuduSession#close idempotent

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java-client] make AsyncKuduSession#close idempotent .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2976 To unsubscribe

[kudu-CR] Short-circuit CheckRowDeleted if there are no deletes in DeltaFile

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Short-circuit CheckRowDeleted if there are no deletes in DeltaFile .. Patch Set 1: Code-Review+2 Another nice find. -- To view, visit http

[kudu-CR] KUDU-815. Improve performance of first scan following restart

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-815. Improve performance of first scan following restart .. Patch Set 1: Code-Review+2 Nice find. -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] Bump guava version to 18.0

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump guava version to 18.0 .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2972 To unsubscribe, visit http

[kudu-CR] [java client] scanNextRows() needs to always increment attempts

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#2). Change subject: [java client] scanNextRows() needs to always increment attempts .. [java client] scanNextRows() needs to always increment attempts This bug was introduced in

[kudu-CR] [java client] scanNextRows() needs to always increment attempts

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] scanNextRows() needs to always increment attempts .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2973/1/java/kudu-client

[kudu-CR] Bump guava version to 18.0

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Bump guava version to 18.0 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2972/1/java/kudu-client/pom.xml File java/kudu-client/pom.xml

[kudu-CR] [java client] scanNextRows() needs to always increment attempts

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/2973 Change subject: [java client] scanNextRows() needs to always increment attempts .. [java client] scanNextRows() needs to always

[kudu-CR] [java client] Add a method to return rows in order

2016-05-05 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Add a method to return rows in order .. Patch Set 3: > I recall Adar had some concerns on this, may want to check with him >

[kudu-CR] [java client] Upgrade to Netty 3.10.5

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Upgrade to Netty 3.10.5 .. Patch Set 1: > OK, a project for another time then. > > If there are non-trivial gains to be had by

[kudu-CR] [java client] Upgrade to Netty 3.10.5

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Upgrade to Netty 3.10.5 .. Patch Set 1: > I don't know. I got as far as netty.io, clicked on downloads, and > saw a 4.x sta

[kudu-CR] [java client] Upgrade to Netty 3.10.5

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Upgrade to Netty 3.10.5 .. Patch Set 1: > That was easy. Why not upgrade to the 4.x series though? Isn't 4.x a massive rework? --

[kudu-CR] [java client] Add a method to return rows in order

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
To view, visit http://gerrit.cloudera.org:8080/2951 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I865e28bddd945111ee159b6f2715a8629b75743b Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer:

[kudu-CR] [java client] Upgrade to Netty 3.10.5

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/2955 Change subject: [java client] Upgrade to Netty 3.10.5 .. [java client] Upgrade to Netty 3.10.5 We're currently on 3.8.0 whic

[kudu-CR] KUDU-1415. Add statistics in java client.

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1415. Add statistics in java client. .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2858 To unsubscribe, visit http

[kudu-CR] KUDU-1415. Add statistics in java client.

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: KUDU-1415. Add statistics in java client. .. KUDU-1415. Add statistics in java client. The Statistics object will store information such as rpc

[kudu-CR] [java client] Add a method to return rows in order

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Add a method to return rows in order .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/2951/2/java/kudu-client/src/main/java

[kudu-CR] [java client] Add a method to return rows in order

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Add a method to return rows in order .. Patch Set 2: I cleaned up a little too hard for patch 1 and removed something that was used, oops

[kudu-CR] [java client] Add a method to return rows in order

2016-05-04 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#2). Change subject: [java client] Add a method to return rows in order .. [java client] Add a method to return rows in order This patch adds an "unstable" and "

  1   2   3   4   5   6   7   >