Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

bigquery: show base64 encode and records in insertAll snippets.#1407

Merged
tswast merged 1 commit into
googleapis:mastergoogleapis/google-cloud-java:masterfrom
tswast:bigquery-insertalltswast/google-cloud-java:bigquery-insertallCopy head branch name to clipboard
Nov 17, 2016
Merged

bigquery: show base64 encode and records in insertAll snippets.#1407
tswast merged 1 commit into
googleapis:mastergoogleapis/google-cloud-java:masterfrom
tswast:bigquery-insertalltswast/google-cloud-java:bigquery-insertallCopy head branch name to clipboard

Conversation

@tswast

@tswast tswast commented Nov 16, 2016

Copy link
Copy Markdown
Contributor

Showing a raw base64 string isn't great. Instead, show a way to encode
the bytes to a base64 string using Guava's BaseEncoding class.

Also, it wasn't clear how to insert values of type record, so show that,
too.

@lesv @mziccard PTAL

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 16, 2016
@tswast tswast force-pushed the bigquery-insertall branch from 96aede3 to a6cc3f1 Compare November 16, 2016 23:06
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.005%) to 84.22% when pulling a6cc3f1 on tswast:bigquery-insertall into da12d8f on GoogleCloudPlatform:master.

Showing a raw base64 string isn't great. Instead, show a way to encode
the bytes to a base64 string using Guava's BaseEncoding class.

Also, it wasn't clear how to insert values of type record, so show that,
too.
@tswast tswast force-pushed the bigquery-insertall branch from a6cc3f1 to 063f902 Compare November 17, 2016 17:21
@tswast tswast merged commit e2bf4d4 into googleapis:master Nov 17, 2016
@tswast tswast deleted the bigquery-insertall branch November 17, 2016 17:33
@mziccard

Copy link
Copy Markdown
Contributor

Using Guava in snippets is generally a bad idea. We shouldn't assume that users are familiar with it nor we should show code that depends on it to work

@tswast

tswast commented Nov 17, 2016

Copy link
Copy Markdown
Contributor Author

Okay. I wasn't sure about that. I wish we could show imports in snippets.

What about using classes that are Java 8, only? http://docs.oracle.com/javase/8/docs/api/java/util/Base64.html There wasn't a built-in way to encode Base64 until then.

chingor13 pushed a commit that referenced this pull request Jan 6, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
chingor13 pushed a commit that referenced this pull request Jan 22, 2026
🤖 I have created a release *beep* *boop*
---


### Updating meta-information for bleeding-edge SNAPSHOT release.

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
chingor13 pushed a commit that referenced this pull request Feb 24, 2026
)

* feat: [java] allow passing libraries_bom_version from env

* reformat
Source-Link: googleapis/synthtool@e36d2f1
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:31aa2ef27b071c2e7844b0eb1d5a24254daff06615b1b138b994dd6345c0b0ea

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Diego Marquez <diegomarquezp@google.com>
chingor13 pushed a commit that referenced this pull request Mar 12, 2026
)

* feat: [java] allow passing libraries_bom_version from env

* reformat
Source-Link: googleapis/synthtool@e36d2f1
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:31aa2ef27b071c2e7844b0eb1d5a24254daff06615b1b138b994dd6345c0b0ea

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Diego Marquez <diegomarquezp@google.com>
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026
suztomo pushed a commit that referenced this pull request Mar 30, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
lqiu96 pushed a commit that referenced this pull request Apr 1, 2026
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
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.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.