[Impala-ASF-CR] IMPALA-5446: dropped Sorter::Reset() status
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5446: dropped Sorter::Reset() status .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7134 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7d4f9e93a44531901e663b3f1e18edc514363f74 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim ArmstrongGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: anujphadke Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5446: dropped Sorter::Reset() status
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5446: dropped Sorter::Reset() status .. IMPALA-5446: dropped Sorter::Reset() status This patch aligns the sorter's methods closer with the ExecNode methods and moves the possibly-failing parts of Reset() into Open(). Testing: Added WARN_UNUSED_RESULT to all the sorter methods that return Status to prevent similar issues in future. Add a test that sometimes goes down this code path. It was able to cause a crash at least once every 5 executions. Ran an exhaustive build to make sure there were no other regressions. Change-Id: I7d4f9e93a44531901e663b3f1e18edc514363f74 Reviewed-on: http://gerrit.cloudera.org:8080/7134 Reviewed-by: Tim ArmstrongTested-by: Impala Public Jenkins --- M be/src/exec/sort-node.cc M be/src/runtime/sorter.cc M be/src/runtime/sorter.h M testdata/workloads/functional-query/queries/QueryTest/nested-types-tpch.test 4 files changed, 43 insertions(+), 21 deletions(-) Approvals: Impala Public Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/7134 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d4f9e93a44531901e663b3f1e18edc514363f74 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim Armstrong Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: anujphadke
[Impala-ASF-CR] IMPALA-5446: dropped Sorter::Reset() status
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5446: dropped Sorter::Reset() status .. Patch Set 2: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/7134 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7d4f9e93a44531901e663b3f1e18edc514363f74 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim ArmstrongGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: anujphadke Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5446: dropped Sorter::Reset() status
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5446: dropped Sorter::Reset() status .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/724/ -- To view, visit http://gerrit.cloudera.org:8080/7134 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7d4f9e93a44531901e663b3f1e18edc514363f74 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim ArmstrongGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: anujphadke Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5446: dropped Sorter::Reset() status
Dan Hecht has posted comments on this change. Change subject: IMPALA-5446: dropped Sorter::Reset() status .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7134 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7d4f9e93a44531901e663b3f1e18edc514363f74 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim ArmstrongGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: anujphadke Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5446: dropped Sorter::Reset() status
Alex Behm has posted comments on this change. Change subject: IMPALA-5446: dropped Sorter::Reset() status .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7134 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7d4f9e93a44531901e663b3f1e18edc514363f74 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Tim ArmstrongGerrit-Reviewer: Alex Behm Gerrit-Reviewer: anujphadke Gerrit-HasComments: No