[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4417 To unsubscribe, visit

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. KUDU-1614 - [python] Enable Set/Get of unixtime_micros Currently, the python client in Kudu does

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/4417/8/python/kudu/client.pyx File python/k

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-19 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 9: (4 comments) http://gerrit.cloudera.org:8080/#/c/4417/8/python/kudu/client.pyx File pyt

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-19 Thread Jordan Birdsell (Code Review)
Hello David Ribeiro Alves, Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4417 to look at the new patch set (#9). Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros ..

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 8: (4 comments) http://gerrit.cloudera.org:8080/#/c/4417/8/python/kudu/client.pyx File python/

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 8: Code-Review+1 Leaving a +1 so that Todd can take one last look -- To view, visit h

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-16 Thread Jordan Birdsell (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4417 to look at the new patch set (#8). Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros ...

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-16 Thread Jordan Birdsell (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4417 to look at the new patch set (#7). Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros ...

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-15 Thread Jordan Birdsell (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4417 to look at the new patch set (#6). Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros ...

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-15 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/4417/4/python/kudu/util.py File python

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-15 Thread Jordan Birdsell (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4417 to look at the new patch set (#5). Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros ...

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/4417/4/python/kudu/util.py File python/kud

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-15 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 4: Code-Review+1 LGTM but this is one of my first times looking at the Python code so I will

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-15 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c/4417/3/python/kudu/client.pyx File pyt

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-15 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has uploaded a new patch set (#4). Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. KUDU-1614 - [python] Enable Set/Get of unixtime_micros Currently, the python client in Kudu does not su

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-14 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 3: (5 comments) Looks nice; mostly nits. It's good to be able to use a datetime to set a ti

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-14 Thread Jordan Birdsell (Code Review)
Jordan Birdsell has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4417/1//COMMIT_MSG Commit Message: Lin

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/3434/ (1/2) -- To view, visit http://

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-tidy-bot/19/ (2/2) -- To view, visit http://

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-14 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4417 to look at the new patch set (#3). Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. KUDU-1

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/3433/ (1/2) -- To view, visit http://

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-tidy-bot/17/ (2/2) -- To view, visit http://

[kudu-CR] KUDU-1614 - [python] Enable Set/Get of unixtime micros

2016-09-14 Thread Jordan Birdsell (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4417 to look at the new patch set (#2). Change subject: KUDU-1614 - [python] Enable Set/Get of unixtime_micros .. KUDU-1