Skip to content

Conversation

@ggevay
Copy link
Contributor

@ggevay ggevay commented Jun 17, 2025

This is on top of #32668

Motivation

This is for investigating the spikes in the processing time of simple pgwire messages, e.g., parse. Discussed with @aljoscha on zoom.

Tips for reviewer

Checklist

  • This PR has adequate test coverage / QA involvement has been duly considered. (trigger-ci for additional test/nightly runs)
  • This PR has an associated up-to-date design doc, is a design doc (template), or is sufficiently small to not require a design.
  • If this PR evolves an existing $T ⇔ Proto$T mapping (possibly in a backwards-incompatible way), then it is tagged with a T-proto label.
  • If this PR will require changes to cloud orchestration or tests, there is a companion cloud PR to account for those changes that is tagged with the release-blocker label (example).
  • If this PR includes major user-facing behavior changes, I have pinged the relevant PM to schedule a changelog post.

@ggevay ggevay force-pushed the ensure_transaction-metric branch from 6db465e to 12ab89d Compare June 17, 2025 15:01
@ggevay ggevay marked this pull request as ready for review June 17, 2025 15:03
@ggevay ggevay requested a review from a team as a code owner June 17, 2025 15:03
@ggevay ggevay requested a review from aljoscha June 17, 2025 15:03
@ggevay ggevay force-pushed the ensure_transaction-metric branch from 12ab89d to 8e2c7d6 Compare June 17, 2025 16:15
Copy link
Contributor

@aljoscha aljoscha left a comment

Choose a reason for hiding this comment

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

excellent!

@ggevay ggevay force-pushed the ensure_transaction-metric branch from 8e2c7d6 to cd506dd Compare June 18, 2025 16:15
@ggevay ggevay enabled auto-merge June 18, 2025 16:15
@ggevay ggevay merged commit f747093 into MaterializeInc:main Jun 18, 2025
86 checks passed
@ggevay ggevay added the A-ADAPTER Topics related to the ADAPTER layer label Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ADAPTER Topics related to the ADAPTER layer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants