Skip to content

scrape: switch scrape manager to AppenderV2#17959

Closed
bwplotka wants to merge 1 commit intomainfrom
bwplotka/switch-scrape1
Closed

scrape: switch scrape manager to AppenderV2#17959
bwplotka wants to merge 1 commit intomainfrom
bwplotka/switch-scrape1

Conversation

@bwplotka
Copy link
Copy Markdown
Member

@bwplotka bwplotka commented Jan 29, 2026

Related to #17632

See also #17960

Does this PR introduce a user-facing change?

NONE

@bwplotka
Copy link
Copy Markdown
Member Author

/prombench main

@prombot
Copy link
Copy Markdown
Contributor

prombot commented Jan 29, 2026

⏱️ Welcome to Prometheus Benchmarking Tool. ⏱️

Compared versions: PR-17959 and main

After the successful deployment (check status here), the benchmarking results can be viewed at:

Available Commands:

  • To restart benchmark: /prombench restart main
  • To stop benchmark: /prombench cancel
  • To print help: /prombench help

@bwplotka
Copy link
Copy Markdown
Member Author

bwplotka commented Jan 30, 2026

Summary of the dual prombench run on the current state of things.

Health

Same head chunks, series, similar head appenders grow, etc

Similar traffic (although load on PR S is kind of smaller for /query)

image

I see weird lags on query engine for (S):

image

Same for (O):

image image

Latency is clearly impacted for (S)

image

On (O) not so much, but something on query_range

image

Resources

Generally this V2 still allocates and uses more CPU, especially weirdly heap over time.

Last 24h:

S) +5-6% alloc/s more, +5-10% of total memory

image

O) +1-3% alloc/s more, +6-10% of total memory

image image

CPU also higher:

S) +5%

image

O) +7%
image

@bwplotka
Copy link
Copy Markdown
Member Author

Closed (O) prombench and PR (#17960). Leaving this prombench for a day more just in case anyone wants to investigate.

@bwplotka
Copy link
Copy Markdown
Member Author

Cumulative diff shows appending float clearly

image

Are we not cleaning sync.Pool buffers somewhow for V2?

image

@bwplotka
Copy link
Copy Markdown
Member Author

Cumulative, those two are fun:

Is this... fanout?

image

vs ref

image

@bwplotka
Copy link
Copy Markdown
Member Author

Yup that's it, got it:

image

var partialErr AppendPartialError

@bwplotka bwplotka closed this Jan 30, 2026
@bwplotka
Copy link
Copy Markdown
Member Author

/prombench cancel

@prombot
Copy link
Copy Markdown
Contributor

prombot commented Jan 30, 2026

Benchmark cancel is in progress.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants