Remove validation for ShardID being 0 as it is valid
@@ -1396,9 +1396,11 @@ func (h *Handler) SyncShardStatus(ctx context.Context, request *historyservice.S
return nil, h.error(errSourceClusterNotSet, scope, "", "")
}
- if request.GetShardId() == 0 {
- return nil, h.error(errShardIDNotSet, scope, "", "")
- }
+ // TODO: Disabling this check as 0 is a valid ShardID. Correct long term fix is to have ShardID start from 1
+ // so we can enable this check to validate ShardID is not set.
+ // if request.GetShardId() == 0 {
+ // return nil, h.error(errShardIDNotSet, scope, "", "")
+ // }
if request.GetTimestamp() == 0 {
return nil, h.error(errTimestampNotSet, scope, "", "")
