Fang-Yu Rao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20145 )

Change subject: IMPALA-12256: Fix DDLs losing create event ids in reloading 
partitions
......................................................................


Patch Set 6:

(1 comment)

Sorry I have another question that I forgot to ask in my previous review. I 
asked this question mostly for my own understanding. Thanks!

http://gerrit.cloudera.org:8080/#/c/20145/6/tests/custom_cluster/test_events_custom_configs.py
File tests/custom_cluster/test_events_custom_configs.py:

http://gerrit.cloudera.org:8080/#/c/20145/6/tests/custom_cluster/test_events_custom_configs.py@858
PS6, Line 858:       self.client.execute(
             :           "alter table %s.part drop partition (p=0)" % 
unique_database)
             :       self.client.execute(
             :           "insert into %s.part partition(p=0) values (1),(2)" % 
unique_database)
             :       self.client.execute(ddl)
I have another question regarding the events corresponding to the 3 queries 
above.

When we submit the 3 queries above to Impala, is it true that the order of the 
respective events received from HMS is non-deterministic?

What if we receive the DROP_PARTITION event prior to the events corresponding 
to the other 2 queries? Will this DROP_PARTITION event be skipped in this case? 
Or some logic ensures even in this case (the DROP_PARTITION event arriving 
first) this DROP_PARTITION will still be skipped?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I052faa093bda69fb16db0d424c1478bba103dad9
Gerrit-Change-Number: 20145
Gerrit-PatchSet: 6
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]>
Gerrit-Comment-Date: Sat, 08 Jul 2023 01:29:51 +0000
Gerrit-HasComments: Yes

Reply via email to