temporal icon indicating copy to clipboard operation
temporal copied to clipboard
verified publisher icon temporalio

Assert shard ownership against source of truth when acquireShards is invoked

Open wxing1292 opened this issue 3 years ago • 1 comments

acquireShards function right now only assert shard (in mem) data structure is in valid state, additional assertion should also be performed against DB (source of truth), e.g. using AssertShardOwnership persistence API

https://github.com/temporalio/temporal/blob/v1.17.1/service/history/shard/controller_impl.go#L344

wxing1292 avatar Jul 22 '22 20:07 wxing1292

In order to AssertShardOwnership you need to have the current RangeID, right? So this is only for shards that are already loaded?

dnr avatar Jul 27 '22 01:07 dnr