Sourabh Goyal has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17859 )

Change subject: IMPALA-10926: Improve catalogd consistency and self events 
detection
......................................................................


Patch Set 36:

@Vihang: Thanks for reviewing it thoroughly. I already have a jira for the 
followup https://issues.apache.org/jira/browse/IMPALA-10976.

I will resolve the conflicts after rebasing the patch.
> Patch Set 36: Code-Review+2
>
> The patch looks good to me. Thanks a lot for seeing it through. I know it 
> took a lot of iterations. I think as a followup, we would need to update the 
> lastSyncedEventId for DDLs from catalogOpExecutor too. For now, this patch 
> enables strong consistency guarantee for catalog's metastore service endpoint.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I36364e401911352c4666674eb98c8d61bbaae9b9
Gerrit-Change-Number: 17859
Gerrit-PatchSet: 36
Gerrit-Owner: Sourabh Goyal <soura...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward <kis...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Sourabh Goyal <soura...@cloudera.com>
Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com>
Gerrit-Reviewer: Yu-Wen Lai <yu-wen....@cloudera.com>
Gerrit-Comment-Date: Thu, 02 Dec 2021 12:42:12 +0000
Gerrit-HasComments: No

Reply via email to