Skip to content

refactor(common): re-use value extractors in synchronous polling loop#6823

Merged
coryan merged 1 commit intogoogleapis:mainfrom
coryan:refactor-common-extract-long-running
Jun 21, 2021
Merged

refactor(common): re-use value extractors in synchronous polling loop#6823
coryan merged 1 commit intogoogleapis:mainfrom
coryan:refactor-common-extract-long-running

Conversation

@coryan
Copy link
Copy Markdown
Contributor

@coryan coryan commented Jun 21, 2021

This is just a cleanup after some of the work for #6821


This change is Reviewable

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 21, 2021
@google-cloud-cpp-bot
Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 876c0eb47c697137da719748a317e172f15ee9b5

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 21, 2021

Codecov Report

Merging #6823 (876c0eb) into main (2038107) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6823      +/-   ##
==========================================
- Coverage   95.03%   95.02%   -0.01%     
==========================================
  Files        1264     1264              
  Lines      113172   113150      -22     
==========================================
- Hits       107553   107526      -27     
- Misses       5619     5624       +5     
Impacted Files Coverage Δ
google/cloud/internal/polling_loop.h 100.00% <100.00%> (ø)
...e/cloud/spanner/testing/cleanup_stale_instances.cc 42.42% <0.00%> (-12.13%) ⬇️
google/cloud/grpc_error_delegate.cc 95.83% <0.00%> (-4.17%) ⬇️
...le/cloud/internal/default_completion_queue_impl.cc 97.60% <0.00%> (+0.59%) ⬆️

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 2038107...876c0eb. Read the comment docs.

@coryan coryan marked this pull request as ready for review June 21, 2021 21:01
@coryan coryan requested a review from a team June 21, 2021 21:01
@coryan coryan merged commit c386028 into googleapis:main Jun 21, 2021
@coryan coryan deleted the refactor-common-extract-long-running branch June 21, 2021 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants