Skip to content

chore: introduce Throughput and ThroughputSink#2138

Merged
BenWhitehead merged 2 commits intomainfrom
write-acceleration/m1/2/throughput-sink
Jul 31, 2023
Merged

chore: introduce Throughput and ThroughputSink#2138
BenWhitehead merged 2 commits intomainfrom
write-acceleration/m1/2/throughput-sink

Conversation

@BenWhitehead
Copy link
Collaborator

Pre-work

In some upcoming work we want to be able to keep a running window of throughput performance in order to provide improved throughput. This PR introduces a new utility class to model and compute throughput, and the concept of a ThroughputSink which values can be appended to.

@BenWhitehead BenWhitehead added the owlbot:ignore instruct owl-bot to ignore a PR label Jul 24, 2023
@BenWhitehead BenWhitehead requested a review from a team as a code owner July 24, 2023 22:05
@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. api: storage Issues related to the googleapis/java-storage API. labels Jul 24, 2023
Base automatically changed from write-acceleration/m1/1/BlobWriteSession to main July 25, 2023 19:01
@BenWhitehead BenWhitehead requested a review from a team July 25, 2023 19:01
@BenWhitehead BenWhitehead force-pushed the write-acceleration/m1/2/throughput-sink branch from c0e9cd3 to d277957 Compare July 25, 2023 19:02
_Pre-work_

In some upcoming work we want to be able to keep a running window of throughput performance in order to provide improved throughput. This PR introduces a new utility class to model and compute throughput, and the concept of a ThroughputSink which values can be appended to.
@BenWhitehead BenWhitehead force-pushed the write-acceleration/m1/2/throughput-sink branch from d277957 to b0c9cc1 Compare July 31, 2023 16:15
Copy link
Contributor

@sydney-munro sydney-munro left a comment

Choose a reason for hiding this comment

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

LGTM

@BenWhitehead BenWhitehead merged commit a422a70 into main Jul 31, 2023
@BenWhitehead BenWhitehead deleted the write-acceleration/m1/2/throughput-sink branch July 31, 2023 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. owlbot:ignore instruct owl-bot to ignore a PR size: xl Pull request size is extra large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants