Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19445 )

Change subject: KUDU-1945 Update auto incrementing counter during bootstrap
......................................................................


Patch Set 6:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/19445/6//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/19445/6//COMMIT_MSG@38
PS6, Line 38: Columns - A Non Unique Primary Key column of type INT32
> No nothing changes as we update auto incrementing counter only during boots
Thank you for the clarification.

Just wanted to make sure my understanding is correct.  So, a few more 
questions, please :)

What's going to be the value for the auto-incrementing column once TS2 
bootstraps with all INSERT operations GCed, so only the segment with DELETE 
operations has left?  I guess it is going to be 0.

What's the value for the auto-incrementing column for the rest of tablet 
servers that haven't been restarted?  I guess the other two are going to have 
something non-zero, and that's what left from the very last INSERT.

With a re-election, when the tablet replica at TS2 becomes a leader, INSERTs 
funneled through that replica at TS2 are going to start with 0 for the 
auto-incrementing column.  Those values will be pushed to other two follower 
replicas and will override the non-zero value that has been kept there from the 
prior bunch of INSERT operations.

With that, all the replicas are going to have the same values for the 
auto-incrementing column for all newly inserted rows, so all is well.

Is my understanding correct?

Thanks!



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I61b305efd7d5a065a2976327567163956c0c2184
Gerrit-Change-Number: 19445
Gerrit-PatchSet: 6
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Comment-Date: Wed, 29 Mar 2023 05:38:52 +0000
Gerrit-HasComments: Yes

Reply via email to