Thanks David, wasn't quite sure how that worked On Wed, Sep 14, 2016, 8:43 PM David Alves <[email protected]> wrote:
> 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 that > understand the component we're working on the best, or if we can't think of > anyone specific, then at random :) > For now you can add me as a reviewer to your python patches, I've likely > been the last one to work more extensively on it. > > Best > David > > On Wed, Sep 14, 2016 at 5:17 PM, Jordan Birdsell < > [email protected]> > wrote: > > > Hey folks, > > > > I've put together a couple more python patches: > > > > For KUDU-1612, KUDU-1615 and KUDU-1615: > > https://gerrit.cloudera.org/#/c/4417/ > > This addresses enabling getting/setting timestamps (unixtime_micros), > > scanner read at snapshot and a minor bug with write operations. I > combined > > them because of the overlap in functionality, however if this is bad > form, > > let me know. > > > > For KUDU-1611: > > https://gerrit.cloudera.org/#/c/4408/ > > This enables setting the selection mode for scanners/scan token builders. > > > > Someone mind taking a look? > > > > Thanks, > > Jordan > > >
