Skip to content

Bigtable: Deemphasize most of the settings and move them to the StubSettings - #4509

Merged
igorbernstein2 merged 5 commits into
googleapis:masterfrom
igorbernstein2:settings--5
Feb 20, 2019
Merged

Bigtable: Deemphasize most of the settings and move them to the StubSettings#4509
igorbernstein2 merged 5 commits into
googleapis:masterfrom
igorbernstein2:settings--5

Conversation

@igorbernstein2

Copy link
Copy Markdown
Contributor

This is a breaking change because it removes the ClientSettings base class. However to ease the transition, most methods have been copied over but marked deprecated to BigtableDataSettings

Extracted from #4498

@igorbernstein2igorbernstein2 added the api: bigtable Issues related to the Bigtable API. label Feb 15, 2019
@igorbernstein2
igorbernstein2 requested a review from a teamFebruary 15, 2019 19:55
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 15, 2019
@codecov

codecovBot commented Feb 15, 2019

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@e6cf8d3). Click here to learn what that means.
The diff coverage is 16.12%.

Impacted file tree graph

@@ Coverage Diff @@## master #4509 +/- ##
=========================================
Coverage ? 49.15% Complexity ? 21929 =========================================
Files ? 2077 Lines ? 207109 Branches ? 24097 =========================================
Hits ? 101804 Misses ? 97132 Partials ? 8173
Impacted FilesCoverage ΔComplexity Δ
...gle/cloud/bigtable/data/v2/BigtableDataClient.java89.58% <100%> (ø)32 <1> (?)
...e/cloud/bigtable/data/v2/BigtableDataSettings.java22.22% <14.75%> (ø)3 <2> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e6cf8d3...656a175. Read the comment docs.

…ettings
This is a breaking change because it removes the ClientSettings base class. However to ease the transition, most methods have been copied over but marked deprecated to BigtableDataSettings
# Conflicts:
#	google-cloud-clients/google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/BigtableDataSettings.java
@igorbernstein2igorbernstein2 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 19, 2019
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 19, 2019
@igorbernstein2
igorbernstein2 merged commit 3559aa7 into googleapis:masterFeb 20, 2019
@igorbernstein2
igorbernstein2 deleted the settings--5 branch February 20, 2019 17:25
benjaminp pushed a commit to benjaminp/google-cloud-java that referenced this pull request Jul 13, 2026
…ettings (googleapis#4509)
* Bigtable: Deemphasize most of the settings and move them to the StubSettings
This is a breaking change because it removes the ClientSettings base class. However to ease the transition, most methods have been copied over but marked deprecated to BigtableDataSettings
* format
* fix bad merge
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtableIssues related to the Bigtable API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@igorbernstein2@sduskis@googlebot@yoshi-kokoro