Xianqing He has posted comments on this change. ( http://gerrit.cloudera.org:8080/17729 )
Change subject: IMPALA-10825: fix impalad crashes when closing the retrying query ...................................................................... Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/17729/2/be/src/runtime/query-driver.cc File be/src/runtime/query-driver.cc: http://gerrit.cloudera.org:8080/#/c/17729/2/be/src/runtime/query-driver.cc@190 PS2, Line 190: // Update the state and then schedule the retry asynchronously. > Should the MarkAsRetrying() call above be skipped or reset in this case? Yeah, I agree. Done. http://gerrit.cloudera.org:8080/#/c/17729/2/tests/custom_cluster/test_query_retries.py File tests/custom_cluster/test_query_retries.py: http://gerrit.cloudera.org:8080/#/c/17729/2/tests/custom_cluster/test_query_retries.py@760 PS2, Line 760: select count(*) from tpch_parquet.lineitem > nit. Test robustness. Wonder if this query should be relatively complex (su Not need. We use the debug action to delay and have enough time to do close_query() http://gerrit.cloudera.org:8080/#/c/17729/2/tests/custom_cluster/test_query_retries.py@764 PS2, Line 764: time.sleep(1) : # close the query > nit. May add a comment "close the query" here. Done http://gerrit.cloudera.org:8080/#/c/17729/2/tests/custom_cluster/test_query_retries.py@773 PS2, Line 773: > nit. should we check all remaining impalads are running instead of the 1st This bug only crashes the coordinator, so check the first one is enough -- To view, visit http://gerrit.cloudera.org:8080/17729 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3b9a2cccbfbdca00b099e0f8d5f2d4bcb4d0a8c3 Gerrit-Change-Number: 17729 Gerrit-PatchSet: 3 Gerrit-Owner: Xianqing He <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Kurt Deschler <[email protected]> Gerrit-Reviewer: Qifan Chen <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Xianqing He <[email protected]> Gerrit-Comment-Date: Wed, 28 Jul 2021 06:15:13 +0000 Gerrit-HasComments: Yes
