Skip to content

Commit

Permalink
Merge pull request #833 from Aiven-Open/matyaskuti/fix_rest_proxy_com…
Browse files Browse the repository at this point in the history
…mit_empty_payload

Fix no-payload commit in consumer
  • Loading branch information
eliax1996 authored Mar 5, 2024
2 parents 4ae29e6 + c8f3923 commit af21074
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion karapace/kafka/consumer.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,10 @@ def commit( # type: ignore[override]
if message is not None:
return super().commit(message=message, asynchronous=False)

return super().commit(offsets=offsets, asynchronous=False)
if offsets is not None:
return super().commit(offsets=offsets, asynchronous=False)

return super().commit(asynchronous=False)
except KafkaException as exc:
raise_from_kafkaexception(exc)

Expand Down

0 comments on commit af21074

Please sign in to comment.