kanadi icon indicating copy to clipboard operation
kanadi copied to clipboard

Nakadi clients resilience to partial outage and partial success

Open adyach opened this issue 1 year ago • 5 comments

Nakadi publishing API accepts events in batches. It can fail to publish some events from the batch to underlying storage (Apache Kafka). In that case Nakadi publishing API will return error that batch was partially successful. It can create problems the following problems, depending on how the Nakadi client and the publishing application deals with this partial success response:

  • increase in traffic on Nakadi publishing API due to Nakadi clients retrying the whole batch over and over
  • the application retries identical batches which prevents application from progressing

The following should be done to decrease the possibility of mentioned problems:

  • Nakadi client should contain a note to developers that publishing can experience partial success. This should be in the client documentation and ideally also within the self contained code documentation, raising awareness for the users, e.g. via docstrings.

  • An optional retry method on batch level can be provided for the whole batch, but the default strategy must contain a backoff - solution in case of continued errors to publish to Nakadi.

  • An optional retry method can be provided that only re-publishes unsuccessful events to Nakadi. This retry must also support a backoff strategy by default.

  • Clients must expose the result of a publishing request in a way that developers can understand that there is the possibility of a partial success for batch publishing.

adyach avatar Sep 08 '23 12:09 adyach

Hi @adyach Thank you for reaching out,

At the moment there should be a backoff logic that retries the publishing. And the result of partial publishing is already exposed to the clients.

I'll update the readme to clarify the behavior.

gchudnov avatar Sep 12 '23 08:09 gchudnov

@gchudnov docs update is great. thank you!

adyach avatar Sep 12 '23 08:09 adyach

@gchudnov any update on this ?

adyach avatar Oct 09 '23 07:10 adyach

@adyach sorry, being quite busy :( will finish the doc by the end of tomorrow (Oct 10, 2023)

gchudnov avatar Oct 09 '23 10:10 gchudnov

@adyach just made a PR, please take a look: https://github.com/zalando-nakadi/kanadi/pull/219

gchudnov avatar Oct 10 '23 19:10 gchudnov

closed as migrated to the internal repo

gchudnov avatar Jul 30 '24 11:07 gchudnov