Skip to content

Bigtable: 15. Implement CheckAndMutateRow - #2980

Merged
garrettjonesgoogle merged 4 commits into
googleapis:masterfrom
igorbernstein2:15-impl-checkandmutaterow
Mar 2, 2018
Merged

Bigtable: 15. Implement CheckAndMutateRow#2980
garrettjonesgoogle merged 4 commits into
googleapis:masterfrom
igorbernstein2:15-impl-checkandmutaterow

Conversation

@igorbernstein2

@igorbernstein2igorbernstein2 commented Mar 1, 2018

Copy link
Copy Markdown
Contributor

No description provided.

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 1, 2018
@igorbernstein2
igorbernstein2force-pushed the 15-impl-checkandmutaterow branch from 2affa3c to 94f7190CompareMarch 2, 2018 03:33
@igorbernstein2
igorbernstein2force-pushed the 15-impl-checkandmutaterow branch from 8523ab7 to a8e5b01CompareMarch 2, 2018 13:46
kevinsi4508
kevinsi4508 approved these changes Mar 2, 2018
View reviewed changes

@kevinsi4508kevinsi4508 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

Comment thread...loud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/stub/EnhancedBigtableStub.java Outdated
return mutateRowsCallable;
}

/** Returns the callable chain created in {@link #createCheckAndMutateRowCallable()}. */

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

garrettjonesgoogle
garrettjonesgoogle reviewed Mar 2, 2018
View reviewed changes

@garrettjonesgooglegarrettjonesgoogle 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.

LGTM

@igorbernstein2

igorbernstein2 commented Mar 2, 2018

Copy link
Copy Markdown
ContributorAuthor

Thanks for reviewing! Please merge when ready

@garrettjonesgoogle
garrettjonesgoogle merged commit 227b1fb into googleapis:masterMar 2, 2018
@igorbernstein2
igorbernstein2 deleted the 15-impl-checkandmutaterow branch March 6, 2018 17:45
chingor13 pushed a commit that referenced this pull request Feb 20, 2026
@renovate-bot
deps: update dependency org.json:json to v20250517 (#2980)
57b944e
lqiu96 pushed a commit that referenced this pull request Mar 20, 2026
@renovate-bot
deps: update dependency org.json:json to v20250517 (#2980)
8e12fb0
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
@gcf-owl-bot@JoeWang1127
chore: update ignore paths in renovate config (#1945) (#2980)
478a1d6
* chore: update ignore paths in renovate config (#1945)
Source-Link: googleapis/synthtool@571a091
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:81c3ec554428c8ff6c92f0d58668b7ef52265d053a82284c97a326745e786949
* update version
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Joe Wang <joewa@google.com>
chingor13 pushed a commit that referenced this pull request Mar 30, 2026
@BenWhitehead
fix: ensure object generation is sent for Storage#update(BlobInfo) us…
805745d
…ing HTTP Transport (#3006)
If a generation is present in the provided BlobInfo ensure it is sent to GCS. This is
a hard requirement when using versioned buckets and trying to update a non-current
object version.
Effects all version 2.42.0 through 2.50.0
Regression introduced in #2664Fixes#2980
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pongadpongadAwaiting requested review from pongad
2 more reviewers
@garrettjonesgooglegarrettjonesgooglegarrettjonesgoogle left review comments
@kevinsi4508kevinsi4508kevinsi4508 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@igorbernstein2@kevinsi4508@garrettjonesgoogle@googlebot