wangsheng has posted comments on this change. ( http://gerrit.cloudera.org:8080/14319 )
Change subject: IMPALA-8984: Fix race condition in creating Kudu table ...................................................................... Patch Set 18: > (1 comment) > > Thanks for keep updating the patch! The patch looks good to me > overall except a question in the test. Thanks for code review many times, Quanlong. I've already adjust code according to your suggestion, and removes unnecessary modify, except time.sleep. This is to increase the possibility of hit the original bug (cause the table being deleted in Kudu but reserving in HMS). If I remove all time.sleep, it rarely reproduce the bug after testing in my environment. -- To view, visit http://gerrit.cloudera.org:8080/14319 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1a4047bcdaa6b346765b96e8c36bb747a2b0091d Gerrit-Change-Number: 14319 Gerrit-PatchSet: 18 Gerrit-Owner: wangsheng <sky...@163.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com> Gerrit-Reviewer: wangsheng <sky...@163.com> Gerrit-Comment-Date: Sat, 28 Dec 2019 02:53:33 +0000 Gerrit-HasComments: No
