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

replace out_of_order with entry_too_far_behind #4330

Closed
owen-d opened this issue Sep 14, 2021 · 4 comments
Closed

replace out_of_order with entry_too_far_behind #4330

owen-d opened this issue Sep 14, 2021 · 4 comments
Labels

Comments

@owen-d
Copy link
Member

owen-d commented Sep 14, 2021

Now that we have support for unordered_writes, we should replace out_of_order errors & metrics with entry_too_far_behind

cc #1544

@cstyan
Copy link
Contributor

cstyan commented Sep 15, 2021

Do we want to rename this even though ingestion of out of order data is opt in via ingester config?

@sherifkayad
Copy link

@owen-d Is there an ETA on the release with this awesome feature just yet?

@slim-bean
Copy link
Collaborator

Do we want to rename this even though ingestion of out of order data is opt in via ingester config?

two thoughts, we should make out of order the default and remove the ability to disable it, remove the config entirely.

We should consider using different limit reasons based on if the feature is enabled

@owen-d
Copy link
Member Author

owen-d commented Oct 29, 2021

Closed by #4592

@owen-d owen-d closed this as completed Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants