Skip to content

compress: count uncompressed input bytes#13287

Merged
moonchen merged 1 commit into
apache:masterfrom
moonchen:compress-bytes-in-metric
Jun 21, 2026
Merged

compress: count uncompressed input bytes#13287
moonchen merged 1 commit into
apache:masterfrom
moonchen:compress-bytes-in-metric

Conversation

@moonchen

Copy link
Copy Markdown
Contributor

Add proxy.process.plugin.compress.bytes_in to track the volume of data the
compress plugin processes.

Copilot AI review requested due to automatic review settings June 16, 2026 23:16
@moonchen moonchen self-assigned this Jun 16, 2026
@moonchen moonchen added this to the 11.0.0 milestone Jun 16, 2026
@moonchen moonchen added Plugins compress compress plugin labels Jun 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new runtime metric to the compress plugin to track how many input (pre-compression) bytes the plugin processes, enabling operational visibility into compression workload.

Changes:

  • Introduces a new stat proxy.process.plugin.compress.bytes_in.
  • Initializes the stat during both global plugin init (TSPluginInit) and remap plugin init (TSRemapInit).
  • Increments the stat in the transform path based on bytes consumed from the upstream reader.

Comment thread plugins/compress/compress.cc Outdated
@moonchen

Copy link
Copy Markdown
Contributor Author

[approve ci autest]

@moonchen moonchen force-pushed the compress-bytes-in-metric branch from d2dfccb to e236367 Compare June 18, 2026 20:48
Comment thread plugins/compress/compress.cc Outdated
Copilot AI review requested due to automatic review settings June 20, 2026 23:42
@moonchen moonchen force-pushed the compress-bytes-in-metric branch from e236367 to 38ee87c Compare June 20, 2026 23:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread plugins/compress/compress.cc Outdated
Add proxy.process.plugin.compress.bytes_in to track the volume of data the
compress plugin processes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@moonchen moonchen force-pushed the compress-bytes-in-metric branch from 38ee87c to 18cb53b Compare June 20, 2026 23:52
@moonchen

Copy link
Copy Markdown
Contributor Author

[approve ci autest]

@moonchen moonchen requested a review from JosiahWI June 21, 2026 01:20

@JosiahWI JosiahWI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@moonchen moonchen merged commit 1e109ce into apache:master Jun 21, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this to For v10.2.0 in ATS v10.2.x Jun 21, 2026
@cmcfarlen cmcfarlen moved this from For v10.2.0 to Picked v10.2.0 in ATS v10.2.x Jun 26, 2026
@cmcfarlen cmcfarlen modified the milestones: 11.0.0, 10.2.0 Jun 26, 2026
@cmcfarlen

Copy link
Copy Markdown
Contributor

Cherry-picked to 10.2.x

cmcfarlen pushed a commit that referenced this pull request Jun 26, 2026
Add proxy.process.plugin.compress.bytes_in to track the volume of data the
compress plugin processes.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
(cherry picked from commit 1e109ce)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Picked v10.2.0

Development

Successfully merging this pull request may close these issues.

4 participants