[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to pad UNIXTIME_MICROS slots in scan results This changes the wire protocol to, upon

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6623 To unsubscribe, visit

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-20 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#10). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 9: (2 comments)

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 9: (2 comments) lgtm aside from a couple nits

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/6623/8/src/kudu/common/wire_protocol.h

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#9). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/6623/7/src/kudu/common/wire_protocol.h

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#8). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/6623/7/src/kudu/common/wire_protocol.h File

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-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/6623 to look at the new patch set (#7). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 4: (6 comments)

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 4: (3 comments)

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-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/6623 to look at the new patch set (#6). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6623/4/src/kudu/common/wire_protocol.cc File

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 5: (2 comments) MJ: Still need to address some comments, but included the client-side fix

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-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/6623 to look at the new patch set (#5). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-14 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6623/4/src/kudu/common/wire_protocol.cc

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-14 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6623/4/src/kudu/common/wire_protocol.cc

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-14 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/6623/4/src/kudu/common/wire_protocol-test.cc

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-14 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/6623/4/src/kudu/common/wire_protocol-test.cc

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#4). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#3). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to pad UNIXTIME_MICROS slots in scan results This changes the wire protocol to, upon request,