You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@impala.apache.org by "Quanlong Huang (Code Review)" <ge...@cloudera.org> on 2023/10/10 10:09:41 UTC

[Impala-ASF-CR] IMPALA-12448: Avoid getting stuck when refreshing a non-existent partitions

Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/20490 )

Change subject: IMPALA-12448: Avoid getting stuck when refreshing a non-existent partitions
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/20490/4//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/20490/4//COMMIT_MSG@15
PS4, Line 15: 
Can we add an e2e test for this? If the test requires no other queries are running at the same time, we can add an annotation of @pytest.mark.execute_serially on it, e.g.
https://github.com/apache/impala/blob/68c831176b82612f0146887e571554f20ce29c1e/tests/query_test/test_queries.py#L304



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iace7cdadda300b03896f92415822266354421887
Gerrit-Change-Number: 20490
Gerrit-PatchSet: 4
Gerrit-Owner: ttttttz <24...@qq.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <hu...@gmail.com>
Gerrit-Reviewer: ttttttz <24...@qq.com>
Gerrit-Comment-Date: Tue, 10 Oct 2023 10:09:41 +0000
Gerrit-HasComments: Yes