[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: KUDU-1679 Propagate timestamps for scans .. KUDU-1679 Propagate timestamps for scans Added the 'propagated_timestamp' field into the ScanResponseP

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5099 To unsubscribe, visit http:/

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-18 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5099 to look at the new patch set (#7). Change subject: KUDU-1679 Propagate timestamps for scans .. KUDU-1679 Propagate

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-17 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5099 to look at the new patch set (#6). Change subject: KUDU-1679 Propagate timestamps for scans .. KUDU-1679 Propagate

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/5099/5/src/kudu/client/client-test.cc File src/kudu/cl

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 5: > Could you put this patch after the consistency itest patch in the > git history and enable that test

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 5: Could you put this patch after the consistency itest patch in the git history and enable that tes

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8080/#/c/5099/5//COMMIT_MSG Commit Message: PS5, Line 10

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5099/4/src/kudu/client/scan_configuration.h File

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-16 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5099 to look at the new patch set (#5). Change subject: KUDU-1679 Propagate timestamps for scans .. KUDU-1679 Propagate

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-16 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 4: (7 comments) Thank you for the review! I'm sending in a new version in a moment. http://gerrit.cloude

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-16 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 4: (7 comments) http://gerrit.cloudera.org:8080/#/c/5099/4/python/kudu/tests/test_scanner.py File py

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-16 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5099 to look at the new patch set (#4). Change subject: KUDU-1679 Propagate timestamps for scans .. KUDU-1679 Propagate

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1679 Propagate timestamps for scans .. Patch Set 2: (1 comment) > I mention this yesterday, but it was pretty late and maybe you > didn't get a chance to

[kudu-CR] KUDU-1679 Propagate timestamps for scans

2016-11-15 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5099 to look at the new patch set (#3). Change subject: KUDU-1679 Propagate timestamps for scans .. KUDU-1679 Propagate