Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 10: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7073
To unsubscribe, visit
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 10:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/796/
--
To view,
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 10: Code-Review+2
Rebased, carrying Alex's +2.
--
To view, visit http://gerrit.cloudera.org:8080/707
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 9:
Thank you for the review. I will rebase the change and make sure one more time
that tests pass on
Alex Behm has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 9: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7073
To unsubscribe, visit http://ge
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 7:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analysis
Lars Volker has uploaded a new patch set (#9).
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make them run on Java 8, b
Alex Behm has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 7:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analysis/C
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 7:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analysis
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 7:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analys
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 8:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analys
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 7:
(3 comments)
Thank you for the review. Please see my comments and PS8.
http://gerrit.cloudera.org
Lars Volker has uploaded a new patch set (#8).
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make them run on Java 8, b
Alex Behm has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 7:
(3 comments)
Let's move forward with this change.
http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src
Lars Volker has uploaded a new patch set (#7).
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make them run on Java 8, b
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 6:
> Lars can obviously chime in but I'll give my rationale for the +1.
>
> My understanding is we
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 6:
(11 comments)
http://gerrit.cloudera.org:8080/#/c/7073/6/fe/src/main/cup/sql-parser.cup
File fe/sr
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 6:
Lars can obviously chime in but I'll give my rationale for the +1.
My understanding is we will s
Jim Apple has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 6:
> Maybe the associated JIRA is a better place to have this
> discussion, but I am trying to understa
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 6:
Maybe the associated JIRA is a better place to have this discussion, but I am
trying to
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 6:
(12 comments)
Looks good. Just have minor comments then I'm ready to +1. I think Alex or
Dimitr
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 6:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/7073/5/fe/src/main/java/org/apache/impala/analysi
Lars Volker has uploaded a new patch set (#6).
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make them run on Java 8, b
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 5:
I'm reluctant to hold up the fix because it's important, but I think if this
goes in as-is it co
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 5:
(6 comments)
I had a few high-level comments:
* A lot of this depends on order being preserved a
Taras Bobrovytsky has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 2:
Dimitris, also there are changes the behavior of the union node. Pass through
is enabled mor
Jim Apple has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 5:
> Many of these changes are essentially needed just for generating
> consistent tests results when w
Dimitris Tsirogiannis has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 5:
Many of these changes are essentially needed just for generating consistent
tests result
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7073/2/fe/src/main/java/org/apache/impala/analysi
Lars Volker has uploaded a new patch set (#5).
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make them run on Java 8, b
Taras Bobrovytsky has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 4:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/7073/4/testdata/workloads/functional-planne
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7073/2/fe/src/main/java/org/apache/impala/analy
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7073/2/testdata/workloads/functional-planner/quer
Jim Apple has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 4:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7073/2//COMMIT_MSG
Commit Message:
PS2, Line 10: L
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7073/2/testdata/workloads/functional-planner/que
Lars Volker has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 2:
(6 comments)
Thank you for the review. Please see my inline comments and PS4.
http://gerrit.cloud
Lars Volker has uploaded a new patch set (#4).
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make them run on Java 8, b
Jim Apple has posted comments on this change.
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
Patch Set 2:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/7073/2//COMMIT_MSG
Commit Message:
PS2, Line 10: r
Lars Volker has uploaded a new patch set (#3).
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make them run on Java 8, b
Lars Volker has uploaded a new patch set (#2).
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the frontend tests to make them run on Java 8, b
Lars Volker has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7073
Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
..
IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8
This change fixes the f
42 matches
Mail list logo