- Notifications
You must be signed in to change notification settings - Fork62
Messages category correlation index is a separate index#45
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Messages category correlation index is a separate index#45
Uh oh!
There was an error while loading.Please reload this page.
Conversation
aaronjensen commentedOct 30, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
100,000 Writes BenchmarkAll units in ms master2584 Avg: 2649 +/- 99 This branch2775 Avg: 2885 +/- 108 Result1-17.3% (8.9% avg) slower (stats here is probably bad, I just compared the edges of avg +/- std dev) |
aaronjensen commentedOct 30, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Timing when invoking This means that active category/correlation combos likely won't see much benefit from this index addition. The only thing that would is category/correlation combos that are infrequent but have a very active category w/ other (or no) correlations. With New Index
Without New Index
|
Side note on correlation data: it remains an open issue as to whether the correlationStreamName should be its own property on the messages table, and not an Eventide-specific piece of metadata. The jury is still out on that issue. The current implementation is a useful one, but should be considered a placeholder - though one that might be permanent. We'll know by the time the next product generation of Message DB goes into development next year. |