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
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
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
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
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
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
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
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
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
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
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
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
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
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,
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
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
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
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
.
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
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
.
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
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
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
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
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
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
...
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
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
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
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
...
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
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
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
33 matches
Mail list logo