Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

docs(samples): add query external bigtable using permanent table#764

Merged
stephaniewang526 merged 3 commits intogoogleapis:masterfrom
pmakani:query-external-bigtable-perm
Oct 7, 2020
Merged

docs(samples): add query external bigtable using permanent table#764
stephaniewang526 merged 3 commits intogoogleapis:masterfrom
pmakani:query-external-bigtable-perm

Conversation

@pmakani
Copy link
Copy Markdown
Contributor

@pmakani pmakani commented Sep 30, 2020

No description provided.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 30, 2020
@stephaniewang526 stephaniewang526 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@stephaniewang526
Copy link
Copy Markdown
Contributor

Hi @billyjacobson -- PTAL as well! Thank you.

@stephaniewang526 stephaniewang526 marked this pull request as ready for review September 30, 2020 15:55
@stephaniewang526 stephaniewang526 requested a review from a team September 30, 2020 15:55
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@pmakani pmakani added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 30, 2020
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 30, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #764   +/-   ##
=========================================
  Coverage     80.41%   80.41%           
  Complexity     1265     1265           
=========================================
  Files            79       79           
  Lines          6530     6530           
  Branches        742      743    +1     
=========================================
  Hits           5251     5251           
  Misses          889      889           
  Partials        390      390           

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 038e55d...4905230. Read the comment docs.

Copy link
Copy Markdown

@lesv lesv left a comment

Choose a reason for hiding this comment

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

LGTM for java-samples

// once, and can be reused for multiple requests.
BigQuery bigquery = BigQueryOptions.getDefaultInstance().getService();

BigtableColumn name =
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

same comments from #763 regarding example bigtable schema

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@product-auto-label product-auto-label bot added api: bigquery Issues related to the googleapis/java-bigquery API. samples Issues that are directly related to samples. labels Oct 1, 2020
@pmakani pmakani force-pushed the query-external-bigtable-perm branch from d2cc43b to 4905230 Compare October 7, 2020 05:44
@pmakani
Copy link
Copy Markdown
Contributor Author

pmakani commented Oct 7, 2020

@stephaniewang526 PTAL.

@stephaniewang526 stephaniewang526 merged commit 257edfc into googleapis:master Oct 7, 2020
@pmakani pmakani deleted the query-external-bigtable-perm branch October 7, 2020 16:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquery Issues related to the googleapis/java-bigquery API. cla: yes This human has signed the Contributor License Agreement. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants