Merge pull request #35 from vmware/spentakota_retLeaseErr

fix: return err log in case of ErrLeaseNotAcquired
This commit is contained in:
spentakota 2023-04-04 11:21:14 -07:00 committed by GitHub
commit 8ecb5b40a2
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -306,10 +306,6 @@ func (sc *PollingShardConsumer) renewLease() error {
log.Debugf("Refreshing lease on shard: %s for worker: %s", sc.shard.ID, sc.consumerID) log.Debugf("Refreshing lease on shard: %s for worker: %s", sc.shard.ID, sc.consumerID)
err := sc.checkpointer.GetLease(sc.shard, sc.consumerID) err := sc.checkpointer.GetLease(sc.shard, sc.consumerID)
if err != nil { if err != nil {
if errors.As(err, &chk.ErrLeaseNotAcquired{}) {
log.Warnf("Failed in acquiring lease on shard: %s for worker: %s", sc.shard.ID, sc.consumerID)
return nil
}
// log and return error // log and return error
log.Errorf("Error in refreshing lease on shard: %s for worker: %s. Error: %+v", log.Errorf("Error in refreshing lease on shard: %s for worker: %s. Error: %+v",
sc.shard.ID, sc.consumerID, err) sc.shard.ID, sc.consumerID, err)