Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase default projection retries #381

Open
zenyui opened this issue Jun 15, 2021 · 1 comment
Open

Increase default projection retries #381

zenyui opened this issue Jun 15, 2021 · 1 comment

Comments

@zenyui
Copy link
Contributor

zenyui commented Jun 15, 2021

Is your feature request related to a problem? Please describe.
Projection retries are too low, so it restarts the projection actor too frequently

Describe the solution you'd like
increase the default projection retries to much higher, and make it configurable via an env var, like COS_READ_SIDE_PROJECTION_RETRIES

For now, we can add this setting but leave it out of the docs so we can play with it.

See the application.conf file for other examples of configurable env var settings with defaults

Describe alternatives you've considered
n/a

Additional context
COS implements the read handler with Akka projections, and when a read side fails, Akka will retry the projection up to the configured times, then restart the actor. Long term, we would like a more graceful akka projection implementation, but for now, it would be nice to increase this default retry and make it configurable.

@Tochemey
Copy link
Contributor

Tochemey commented Aug 7, 2021

@zenyui I think you mean the interval between projection retries which is 1s set here. I will suggest we call that env var COS_READ_SIDE_PROJECTION_RETRIES_INTERVAL for the meantime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants