Kurt Deschler has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15473 )

Change subject: IMPALA-8533: Impala daemon crash on sort
......................................................................


Patch Set 4:

(3 comments)

Thanks for the feedback. Let's try to wrap this up today as it is a 3.4 release 
blocker.

http://gerrit.cloudera.org:8080/#/c/15473/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/15473/2//COMMIT_MSG@7
PS2, Line 7: IMPALA-8533: Impala daemon crash on sort
> Suggest adding the word 'Fix' at the beginning.
Done


http://gerrit.cloudera.org:8080/#/c/15473/2/fe/src/test/java/org/apache/impala/planner/PlannerTest.java
File fe/src/test/java/org/apache/impala/planner/PlannerTest.java:

http://gerrit.cloudera.org:8080/#/c/15473/2/fe/src/test/java/org/apache/impala/planner/PlannerTest.java@977
PS2, Line 977:   @Test
> Was this commented out for debugging ?
Yes. This test causes problems on my workstation. I reverted this.


http://gerrit.cloudera.org:8080/#/c/15473/2/testdata/workloads/functional-planner/queries/PlannerTest/union.test
File testdata/workloads/functional-planner/queries/PlannerTest/union.test:

http://gerrit.cloudera.org:8080/#/c/15473/2/testdata/workloads/functional-planner/queries/PlannerTest/union.test@4577
PS2, Line 4577:
> It would be good to add a short comment about what each of these tests is t
Done



--
To view, visit http://gerrit.cloudera.org:8080/15473
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If19303fbf55927c1e1b76b9b22ab354322b21c54
Gerrit-Change-Number: 15473
Gerrit-PatchSet: 4
Gerrit-Owner: Kurt Deschler <kdesc...@cloudera.com>
Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com>
Gerrit-Reviewer: Anurag Mantripragada <anu...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Kurt Deschler <kdesc...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com>
Gerrit-Comment-Date: Fri, 20 Mar 2020 15:06:25 +0000
Gerrit-HasComments: Yes

Reply via email to