Skip to content

[Bigtable] minor Javadoc clean up for ModifyColumnFamiliesRequest - #5568

Merged
igorbernstein2 merged 3 commits into
googleapis:masterfrom
rahulKQL:test_bigtableCI
Jun 27, 2019
Merged

[Bigtable] minor Javadoc clean up for ModifyColumnFamiliesRequest#5568
igorbernstein2 merged 3 commits into
googleapis:masterfrom
rahulKQL:test_bigtableCI

Conversation

@rahulKQL

@rahulKQLrahulKQL commented Jun 26, 2019

Copy link
Copy Markdown
Contributor

Removed some undefined tags from ModifyColumnFamiliesRequest.java

My local master is failing with some wired issue:
```
Caused by: org.apache.maven.enforcer.rule.api.EnforcerRuleException: Failed while enforcing RequireUpperBoundDeps. The error(s) are [
Require upper bound dependencies error for com.google.guava:guava:27.1-android paths to dependency are:
```
@rahulKQL
rahulKQL requested a review from a teamJune 26, 2019 15:42
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 26, 2019
@ajaaymajaaym added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 26, 2019
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 26, 2019
@rahulKQLrahulKQL changed the title WIP:To Test Bigtable CI failure[Bigtable] minor Javadoc clean up for ModifyColumnFamiliesRequestJun 26, 2019
@sduskissduskis added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 26, 2019
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 26, 2019
* @param familyId
* @return
*/
/** Configures the name of the new ColumnFamily to be created */

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.

Would it make sense to add {@link ColumnFamily}? (and the same applies for all of the other comments)

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Yes, It makes sense to me as well. I have added {@link ...}.

@codecov

codecovBot commented Jun 26, 2019

Copy link
Copy Markdown

Codecov Report

Merging #5568 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #5568 +/- ##
=========================================
Coverage 46.72% 46.72% - Complexity 24630 24645 +15 
=========================================
Files 2351 2351 Lines 256175 256175 Branches 29324 29319 -5 =========================================
Hits 119686 119686 Misses 127569 127569 Partials 8920 8920
Impacted FilesCoverage ΔComplexity Δ
...e/admin/v2/models/ModifyColumnFamiliesRequest.java100% <ø> (ø)7 <0> (ø)⬇️

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 e5e6332...eb6dcde. Read the comment docs.

@sduskissduskis added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 26, 2019
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 26, 2019
@igorbernstein2
igorbernstein2 merged commit 667e026 into googleapis:masterJun 27, 2019
@igorbernstein2igorbernstein2 added the api: bigtable Issues related to the Bigtable API. label Jun 27, 2019
benjaminp pushed a commit to benjaminp/google-cloud-java that referenced this pull request Jul 13, 2026
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.

6 participants

@rahulKQL@sduskis@googlebot@igorbernstein2@ajaaym@yoshi-kokoro