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

re_datastore: all around performance pass tracking issue #443

Closed
11 tasks done
teh-cmc opened this issue Dec 2, 2022 · 1 comment · Fixed by #942
Closed
11 tasks done

re_datastore: all around performance pass tracking issue #443

teh-cmc opened this issue Dec 2, 2022 · 1 comment · Fixed by #942
Labels
⛃ re_datastore affects the datastore itself 🎄 tracking issue issue that tracks a bunch of subissues

Comments

@teh-cmc
Copy link
Member

teh-cmc commented Dec 2, 2022

First pass of performance work

  • put performance probes everywhere
  • provide helpers to gather detailed metrics: global, per-table, per-table-per-bucket
  • a lot superfluous clones & copies that need to go away asap
  • need integration benchmark suites
    • write path, read path, write+read path

The end game, of course, is that query results from the datastore should always be references to the stored data.

And all of the following:

Optional:

@teh-cmc teh-cmc added 🏹 arrow concerning arrow ⛃ re_datastore affects the datastore itself labels Dec 2, 2022
@teh-cmc teh-cmc changed the title re_datastore: do a first all around performance pass re_datastore: all around performance pass tracking issue Dec 12, 2022
@teh-cmc teh-cmc added 🎄 tracking issue issue that tracks a bunch of subissues and removed 🏹 arrow concerning arrow labels Dec 12, 2022
@emilk
Copy link
Member

emilk commented Jan 23, 2023

Let's just add some probes this cycle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⛃ re_datastore affects the datastore itself 🎄 tracking issue issue that tracks a bunch of subissues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants