quickwit-oss / quickwit-oss/quickwit

Remove call to `last_delete_opstamp` at each indexing workbench creation.

Open
#1,938 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Rust
Stars
11.7k
Forks
597
Avg merge
2d 22h
Merged PRs (30d)
37

Description

In PR #1916, we introduced a call to the metastore each time we create a workbench.

That's one extra call every 30s per index.

Discussed here: https://github.com/quickwit-oss/quickwit/pull/1916#discussion_r966762160

We can avoid that by setting the last opstamp directly in the metastore.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Review PR #1916 and its linked discussion first, then trace indexing workbench creation and the metastore call to last_delete_opstamp. Done means workbench creation no longer makes that extra metastore call and the last opstamp is set directly in the metastore; the payload names no tests to run.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend, databases
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.