batchingexperimental: report size of batch by pongad · Pull Request #3029 · googleapis/google-cloud-java · GitHub

and make BatchingWriter use it.

LGTM. Feel free to ignore my comment about the flush bug if you plan on dealing with it later

* chore(main): release 3.16.0 * chore: generate libraries at Fri Jul 11 20:31:51 UTC 2025 --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>

🤖 I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).

* chore(main): release 3.16.0 * chore: generate libraries at Fri Jul 11 20:31:51 UTC 2025 --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [net.bytebuddy:byte-buddy](https://bytebuddy.net) | `1.14.17` -> `1.14.18` | [![age](https://developer.mend.io/api/mc/badges/age/maven/net.bytebuddy:byte-buddy/1.14.18?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/maven/net.bytebuddy:byte-buddy/1.14.18?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/maven/net.bytebuddy:byte-buddy/1.14.17/1.14.18?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/net.bytebuddy:byte-buddy/1.14.17/1.14.18?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/googleapis/sdk-platform-java).

…lexed sessions. (#3029)

batchingexperimental: report size of batch

37dddcd

and make BatchingWriter use it.

pongad requested review from garrettjonesgoogle and igorbernstein2 March 13, 2018 05:37

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 13, 2018

igorbernstein2 approved these changes Mar 14, 2018

View reviewed changes

Comment thread google-cloud-logging/src/main/java/com/google/cloud/logging/BatchingWriter.java Outdated

@Override

public void run() {

send(request);

send(request, size);

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

pr comment

527898f

pongad merged commit 7b79b97 into googleapis:batching-expr Mar 14, 2018

pongad deleted the batching-size branch March 14, 2018 23:19

chingor13 pushed a commit that referenced this pull request Mar 24, 2026

chore: add a CI target to run emulator integration tests using multip…

ecffa4c

…lexed sessions. (#3029)

batchingexperimental: report size of batch by pongad · Pull Request #3029 · googleapis/google-cloud-java · GitHub

Uh oh!

batchingexperimental: report size of batch - #3029

batchingexperimental: report size of batch#3029
pongad merged 2 commits into
googleapis:batching-exprfrom
pongad:batching-size

pongad commented Mar 13, 2018

Uh oh!

igorbernstein2 left a comment

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

                             @Override
                             public void run() {
-                              send(request);
+                              send(request, size);

batchingexperimental: report size of batch by pongad · Pull Request #3029 · googleapis/google-cloud-java · GitHub

Uh oh!

batchingexperimental: report size of batch - #3029

batchingexperimental: report size of batch#3029pongad merged 2 commits intogoogleapis:batching-exprgoogleapis/google-cloud-java:batching-exprfrom pongad:batching-sizeCopy head branch name to clipboard

Conversation

pongad commented Mar 13, 2018

Uh oh!

igorbernstein2 left a comment

Choose a reason for hiding this comment

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

batchingexperimental: report size of batch#3029
pongad merged 2 commits into
googleapis:batching-exprfrom
pongad:batching-size