KEMBAR78
chore(test): log resource prefix to make debugging easier by igorbernstein2 · Pull Request #806 · googleapis/java-bigtable · GitHub
Skip to content

Conversation

@igorbernstein2
Copy link
Contributor

Follow up to #801

  • move prefix generation to its own instance rule
  • log whenever a new prefix generated, which should make it easier to debug test failures

@igorbernstein2 igorbernstein2 requested a review from a team as a code owner May 13, 2021 15:48
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label May 13, 2021
@product-auto-label product-auto-label bot added the api: bigtable Issues related to the googleapis/java-bigtable API. label May 13, 2021
@codecov
Copy link

codecov bot commented May 13, 2021

Codecov Report

Merging #806 (639fe96) into master (f5e5d5a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #806   +/-   ##
=========================================
  Coverage     83.47%   83.47%           
  Complexity     1315     1315           
=========================================
  Files           114      114           
  Lines          7806     7806           
  Branches        446      446           
=========================================
  Hits           6516     6516           
  Misses         1034     1034           
  Partials        256      256           

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 f5e5d5a...639fe96. Read the comment docs.

@igorbernstein2 igorbernstein2 merged commit 167d10d into googleapis:master May 14, 2021
@igorbernstein2 igorbernstein2 deleted the admin-it-log-prefix branch May 14, 2021 17:55
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. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants