Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 5: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7556
To unsubscribe, visit
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
IMPALA-5572: Timestamp codegen for text scanner
Currently codegen is disabled when scanning
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 5:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1013/
--
To view,
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 5:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1007/
--
To view,
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 5:
It looks like you're hitting a known-flaky test:
Tianyi Wang has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 5:
I will test the rebased version again.
--
To view, visit http://gerrit.cloudera.org:8080/7556
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 5: Verified-1
Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1002/
--
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 5:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1002/
--
To view,
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 5: Code-Review+2
rebase.
--
To view, visit http://gerrit.cloudera.org:8080/7556
To unsubscribe,
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 4:
Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/997/
--
To view,
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 4:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/997/
--
To view,
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 4: Verified-1
Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/991/
--
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 4:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/991/
--
To view,
Tianyi Wang has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 4:
> (1 comment)
Done
--
To view, visit http://gerrit.cloudera.org:8080/7556
To unsubscribe, visit
Hello Impala Public Jenkins, Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7556
to look at the new patch set (#4).
Change subject: IMPALA-5572: Timestamp codegen for text scanner
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7556/3/be/src/exec/text-converter.cc
File
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 3: Verified-1
Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/986/
--
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 3:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/986/
--
To view,
Tianyi Wang has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 3:
Rebase done
--
To view, visit http://gerrit.cloudera.org:8080/7556
To unsubscribe, visit
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 2: Code-Review+2
Looks great. I can start the merge after you rebase onto the latest master.
--
Tianyi Wang has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 2:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/7556/1//COMMIT_MSG
Commit Message:
Line 14:
>
Tianyi Wang has uploaded a new patch set (#2).
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
IMPALA-5572: Timestamp codegen for text scanner
Currently codegen is disabled when scanning text tables with
Tianyi Wang has uploaded a new patch set (#2).
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
IMPALA-5572: Timestamp codegen for text scanner
Currently codegen is disabled when scanning text tables with
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7556/1/be/src/exec/hdfs-scanner-ir.cc
File
Tianyi Wang has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7556/1/be/src/exec/hdfs-scanner-ir.cc
File
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
Patch Set 1:
(6 comments)
Nice! No major concerns, mostly comments about comments.
Tianyi Wang has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7556
Change subject: IMPALA-5572: Timestamp codegen for text scanner
..
IMPALA-5572: Timestamp codegen for text scanner
Currently codegen is
27 matches
Mail list logo