Re: [VOTE] Apache Kudu 1.6.0 RC1

2017-12-05 Thread David Alves
+1 built and ran debug/release tests on macso high sierra. -david On Tue, Dec 5, 2017 at 7:16 PM, Dan Burkert wrote: > +1 built and ran tests on macos. > > - Dan > > On Tue, Dec 5, 2017 at 7:06 PM, Mike Percy wrote: > >> +1 >> >> I built 1.6.0-RC1 on

Re: Flaky tablet_history_gc-itest

2016-10-12 Thread David Alves
> > workload.set_num_replicas(1); > > workload.Setup(); // Convenient way to create a table. > > > > Did I miss something? I.e. should I toggle just another control knob > somewhere? > > > Thanks, > > Alexey > > On Wed, Oct 12, 2016 at 1:43 PM, David Alves <davi

Re: [VOTE] Apache Kudu 1.0.1 RC1

2016-10-10 Thread David Alves
+0 Compiled thirdparty successfully, but couldn't compile the sources in macOS Sierra/XCode 8 as this is missing https://gerrit.cloudera.org/#/c/4482/ and https://gerrit.cloudera.org/#/c/4495/ Since this is just a bug fix/doc update release and this problem only affects macOS (macOS is only

Re: Translating Kudu documentation from English into Brazilian Portuguese

2016-09-22 Thread David Alves
Hi Adalberto I share Todd's concern about maintenance, particularly if we're setting out to translate the whole set. As native portuguese speaker though, I'd love to see more portuguese writing on Kudu. I'd be happy to review blog posts or any other kind of written format contribution, just

RFC: removing raft_consensus-test (mock-based)

2016-09-20 Thread David Alves
+1, we've since broadened our coverage in other tests that those cases are covered elsewhere. Plus I agree that with the queue taking more responsibility mocking it becomes a hardship. -david > I'm doing a bit of Consensus refactoring, and finding that the > raft_consensus-test mock-based test

Re: A couple more python patches

2016-09-14 Thread David Alves
Hi Jordan Feel free to add reviewers to your patches (you can do so in gerrit's web ui, or at push time), that way folks get notified when you do a push (if they aren't subscribed to the whole set of notifications, for instance). Usually we name a couple of folks that we feel are the ones

Re: Potential Java client issue?

2016-09-11 Thread David Alves
Look at it for a bit and think I figured out the problem and fixed it (put on the ip2client map is synchronized with the put on client2tablets so it makes sense that the removing is too). The problem is that I'm having a hard time reproing the first failure. I'll ask JD's opinion tomorrow. -david

Re: Projection schema drops key

2016-09-09 Thread David Alves
Hi Jordan KuduScanner::GetProjectionSchema returns the schema of the projection that was previously set on the scanner. If you don't a projection it should indeed return all the columns. Are you sure you didn't set a projection (with SetProjectedColumnNames or SetProjectedColumnIndexes) that