KEMBAR78
chore: refactor unary callable chains to avoid code duplication by igorbernstein2 · Pull Request #2394 · googleapis/java-bigtable · GitHub
Skip to content

Conversation

@igorbernstein2
Copy link
Contributor

@igorbernstein2 igorbernstein2 commented Nov 1, 2024

This is primarily a code clean up change that extracts all of the unary chain building logic into a helper method and removes a significant amount of code duplication.

Change-Id: Idbb2727108857daa662f2e4690053cb6065ad054
Change-Id: I22910ebd8024eecf623fee595596fa53e4ec59f5
@igorbernstein2 igorbernstein2 requested review from a team as code owners November 1, 2024 22:35
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: bigtable Issues related to the googleapis/java-bigtable API. labels Nov 1, 2024
@igorbernstein2 igorbernstein2 changed the title Simplify unary chore: refactor unary callable chains to avoid code duplication Nov 1, 2024
@igorbernstein2 igorbernstein2 added the automerge Merge the pull request once unit tests and other checks pass. label Nov 4, 2024
Copy link
Contributor

@mutianf mutianf left a comment

Choose a reason for hiding this comment

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

Can we add the unary tests to EnhancedBigtableStubTest?

Change-Id: Id28fad823163c00c4d8a30441b9b1ce849e45b9c
Change-Id: I02d2e06e5c93cf5157d26389a7a41ff53cef7ea9
Change-Id: I0ae44d5d286fae4209dedc89f09eb30fde22af24
igorbernstein2 and others added 2 commits November 4, 2024 14:15
Change-Id: Ia7af9eaab0f957129354d896d36643665d5703ac
@gcf-merge-on-green
Copy link

Your PR has conflicts that you need to resolve before merge-on-green can automerge

Change-Id: I644d73fc85c3af024667d0faa3a5a99cbf497c75

# Conflicts:
#	google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/stub/EnhancedBigtableStub.java
@gcf-merge-on-green gcf-merge-on-green bot merged commit b40828c into main Nov 4, 2024
22 of 23 checks passed
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Nov 4, 2024
@gcf-merge-on-green gcf-merge-on-green bot deleted the simplify-unary branch November 4, 2024 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the googleapis/java-bigtable API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants