[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/3413 Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. IMPALA-3729: batch_size=1 coverage for avro scanner Also fix a st

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/3413/1//COMMIT_MSG Commit Message: PS1

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. IMPALA-3729: batch_size=1 coverage for avro scanner Also fix a stale comment in the avro scanner header. Th

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/3413/1//COMMIT_MSG Commit Message: PS1, Li

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3413/2/tests/common/test_result_verifie

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/3413/2/tests/common/test_result_verifier.py

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#4). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. IMPALA-3729: batch_size=1 coverage for avro scanner Also fix a stale comment in the avro scanner header. Th

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-22 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3413/2/tests/common/test_result_verifier

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-22 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 4: (3 comments) In this patch, we are not differentiating between the result when batchsize

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/3413/4//COMMIT_MSG Commit Message: PS4, Li

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. IMPALA-3729: batch_size=1 coverage for avro scanner Also fix a stale comment in the avro scanner header. Th

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-24 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/3413/5/testdata/workloads/functional-que

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-24 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 5: Ah sorry, missed that previous comment. My feeling is that, given data corruption, we don't

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-29 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 5: Code-Review+1 +2 backend change. Please get the review for test/scripts from Taras. -- To view,

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 5: Did you have any more comments Taras? -- To view, visit http://gerrit.cloudera.org:8080/341

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 5: Ping -- To view, visit http://gerrit.cloudera.org:8080/3413 To unsubscribe, visit http://ge

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-18 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 5: Code-Review+1 (1 comment) The Python changes look good to me. http://gerrit.cloudera.or

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Hello Taras Bobrovytsky, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3413 to look at the new patch set (#6). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/3413/5/bin/load-data.py File bin/load-data.p

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Hello Taras Bobrovytsky, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3413 to look at the new patch set (#7). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 7: Addressed comment, rebased. Carry +2 for be and test changes -- To view, visit http://gerri

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3413 To unsubscribe, visit h

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 7: Verified-1 Build failed: http://sandbox.jenkins.cloudera.com/job/impala-external-gerrit-v

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 7: Build failed: http://sandbox.jenkins.cloudera.com/job/impala-external-gerrit-verify-merge

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 7: Build failed: http://sandbox.jenkins.cloudera.com/job/impala-external-gerrit-verify-merge

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Hello Taras Bobrovytsky, Internal Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3413 to look at the new patch set (#8). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner ...

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 8: Code-Review+2 Hit some flaky infra issues and a deleted import during rebase. -- To view, v

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 8: Verified-1 Build failed: http://sandbox.jenkins.cloudera.com/job/impala-external-gerrit-v

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/3413/8/tests/common/test_result_verifier.py

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Hello Taras Bobrovytsky, Internal Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3413 to look at the new patch set (#9). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner ...

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 9: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3413 To unsubscribe, visit h

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3413 To unsubscribe, visit h

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-07-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. IMPALA-3729: batch_size=1 coverage for avro scanner Also fix a stale comment in the avro scanner