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

CosmosStore: Add raw event counts to eqx stats #240

Open
bartelink opened this issue Sep 16, 2020 · 0 comments
Open

CosmosStore: Add raw event counts to eqx stats #240

bartelink opened this issue Sep 16, 2020 · 0 comments
Labels
enhancement up-for-grabs https://up-for-grabs.net

Comments

@bartelink
Copy link
Collaborator

eqx stats cosmos determines the event counts by assuming the n value of the Tip (-1) document matches then number of events stored in the other batch documents within any given stream.

This assumption is broken when pruning is in effect (see #226); the solution is to sum n-i for all documents.

It also makes it harder to demonstrate the degree to which pruning has had an effect in a given container.

Given this is a much more expensive query than the current one, we should consider:

  • it should be added as a separate sub-option of the stats command
  • it should probably not run in parallel with other queries ?
@bartelink bartelink added up-for-grabs https://up-for-grabs.net good-first-issue labels Sep 16, 2020
@bartelink bartelink changed the title Cosmos: Add raw event counts Cosmos: Add raw event counts to eqx stats Oct 22, 2020
@bartelink bartelink changed the title Cosmos: Add raw event counts to eqx stats CosmosStore: Add raw event counts to eqx stats Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement up-for-grabs https://up-for-grabs.net
Projects
None yet
Development

No branches or pull requests

1 participant