-
Notifications
You must be signed in to change notification settings - Fork 60
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
Messages category correlation index is a separate index #45
base: master
Are you sure you want to change the base?
Messages category correlation index is a separate index #45
Conversation
aaronjensen
commented
Oct 30, 2022
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) |
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. |