Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Deprecate format specific row_count field in Read API #249

Merged
merged 22 commits into from
Apr 11, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
707718c
fix!: remove bigquery.readonly auth scope
gcf-owl-bot[bot] Feb 17, 2022
a127647
🦉 Updates from OwlBot
gcf-owl-bot[bot] Feb 17, 2022
787bc75
docs(samples): include metadata file, add exclusions for samples to h…
gcf-owl-bot[bot] Feb 18, 2022
cf4d8eb
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Feb 18, 2022
fd7797b
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Feb 18, 2022
bc100cb
Merge branch 'owl-bot-copy' of https://github.com/googleapis/nodejs-b…
gcf-owl-bot[bot] Feb 18, 2022
85ab2dc
feat: add trace_id for Read API
gcf-owl-bot[bot] Feb 20, 2022
5624cb1
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Feb 20, 2022
90dee96
Merge branch 'main' into owl-bot-copy
steffnay Mar 1, 2022
52e158b
docs: improve documentation for write client
gcf-owl-bot[bot] Mar 2, 2022
cb1c1c6
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Mar 2, 2022
d38c667
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Mar 2, 2022
699d764
Merge branch 'owl-bot-copy' of https://github.com/googleapis/nodejs-b…
gcf-owl-bot[bot] Mar 2, 2022
5dbc9f4
Merge branch 'main' into owl-bot-copy
bcoe Mar 7, 2022
fb86843
chore: update v2.14.1 gapic-generator-typescript
gcf-owl-bot[bot] Mar 8, 2022
c982edc
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Mar 8, 2022
021adb3
chore: update v2.14.2 gapic-generator-typescript
gcf-owl-bot[bot] Mar 16, 2022
f552288
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Mar 16, 2022
92726b0
feat: Deprecate format specific `row_count` field in Read API
gcf-owl-bot[bot] Mar 31, 2022
448432b
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Mar 31, 2022
eddd322
Merge branch 'main' into owl-bot-copy
SurferJeffAtGoogle Apr 7, 2022
2d86059
Merge branch 'main' into owl-bot-copy
steffnay Apr 11, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions linkinator.config.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,14 @@
"skip": [
"https://codecov.io/gh/googleapis/",
"www.googleapis.com",
"img.shields.io"
"img.shields.io",
"https://console.cloud.google.com/cloudshell",
"https://support.google.com"
],
"silent": true,
"concurrency": 5
"concurrency": 5,
"retry": true,
"retryErrors": true,
"retryErrorsCount": 5,
"retryErrorsJitter": 3000
}
7 changes: 4 additions & 3 deletions protos/google/cloud/bigquery/storage/v1/arrow.proto
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright 2021 Google LLC
// Copyright 2022 Google LLC
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -39,8 +39,9 @@ message ArrowRecordBatch {
// IPC-serialized Arrow RecordBatch.
bytes serialized_record_batch = 1;

// The count of rows in `serialized_record_batch`.
int64 row_count = 2;
// [Deprecated] The count of rows in `serialized_record_batch`.
// Please use the format-independent ReadRowsResponse.row_count instead.
int64 row_count = 2 [deprecated = true];
}

// Contains options specific to Arrow Serialization.
Expand Down
7 changes: 4 additions & 3 deletions protos/google/cloud/bigquery/storage/v1/avro.proto
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright 2021 Google LLC
// Copyright 2022 Google LLC
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -35,6 +35,7 @@ message AvroRows {
// Binary serialized rows in a block.
bytes serialized_binary_rows = 1;

// The count of rows in the returning block.
int64 row_count = 2;
// [Deprecated] The count of rows in the returning block.
// Please use the format-independent ReadRowsResponse.row_count instead.
int64 row_count = 2 [deprecated = true];
}
2 changes: 1 addition & 1 deletion protos/google/cloud/bigquery/storage/v1/protobuf.proto
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright 2021 Google LLC
// Copyright 2022 Google LLC
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
Expand Down
27 changes: 22 additions & 5 deletions protos/google/cloud/bigquery/storage/v1/storage.proto
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright 2021 Google LLC
// Copyright 2022 Google LLC
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -47,7 +47,6 @@ service BigQueryRead {
option (google.api.default_host) = "bigquerystorage.googleapis.com";
option (google.api.oauth_scopes) =
"https://www.googleapis.com/auth/bigquery,"
"https://www.googleapis.com/auth/bigquery.readonly,"
"https://www.googleapis.com/auth/cloud-platform";

// Creates a new read session. A read session divides the contents of a
Expand Down Expand Up @@ -168,6 +167,13 @@ service BigQueryWrite {
// * For PENDING streams, data is not made visible until the stream itself is
// finalized (via the `FinalizeWriteStream` rpc), and the stream is explicitly
// committed via the `BatchCommitWriteStreams` rpc.
//
// Note: For users coding against the gRPC api directly, it may be
// necessary to supply the x-goog-request-params system parameter
// with `write_stream=<full_write_stream_name>`.
//
// More information about system parameters:
// https://cloud.google.com/apis/docs/system-parameters
rpc AppendRows(stream AppendRowsRequest) returns (stream AppendRowsResponse) {
option (google.api.http) = {
post: "/v1/{write_stream=projects/*/datasets/*/tables/*/streams/*}"
Expand Down Expand Up @@ -409,10 +415,12 @@ message AppendRowsRequest {
// request.
//
// For explicitly created write streams, the format is:
// `projects/{project}/datasets/{dataset}/tables/{table}/streams/{id}`
//
// * `projects/{project}/datasets/{dataset}/tables/{table}/streams/{id}`
//
// For the special default stream, the format is:
// `projects/{project}/datasets/{dataset}/tables/{table}/_default`.
//
// * `projects/{project}/datasets/{dataset}/tables/{table}/streams/_default`.
string write_stream = 1 [
(google.api.field_behavior) = REQUIRED,
(google.api.resource_reference) = {
Expand Down Expand Up @@ -499,7 +507,10 @@ message BatchCommitWriteStreamsRequest {
// Required. Parent table that all the streams should belong to, in the form of
// `projects/{project}/datasets/{dataset}/tables/{table}`.
string parent = 1 [
(google.api.field_behavior) = REQUIRED
(google.api.field_behavior) = REQUIRED,
(google.api.resource_reference) = {
type: "bigquery.googleapis.com/Table"
}
];

// Required. The group of streams that will be committed atomically.
Expand Down Expand Up @@ -594,6 +605,12 @@ message StorageError {
// There is a schema mismatch and it is caused by user schema has extra
// field than bigquery schema.
SCHEMA_MISMATCH_EXTRA_FIELDS = 7;

// Offset already exists.
OFFSET_ALREADY_EXISTS = 8;

// Offset out of range.
OFFSET_OUT_OF_RANGE = 9;
}

// BigQuery Storage specific error code.
Expand Down
10 changes: 9 additions & 1 deletion protos/google/cloud/bigquery/storage/v1/stream.proto
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright 2021 Google LLC
// Copyright 2022 Google LLC
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -132,6 +132,14 @@ message ReadSession {
// all streams are completely consumed. This estimate is based on
// metadata from the table which might be incomplete or stale.
int64 estimated_total_bytes_scanned = 12 [(google.api.field_behavior) = OUTPUT_ONLY];

// Optional. ID set by client to annotate a session identity. This does not need
// to be strictly unique, but instead the same ID should be used to group
// logically connected sessions (e.g. All using the same ID for all sessions
// needed to complete a Spark SQL query is reasonable).
//
// Maximum length is 256 bytes.
string trace_id = 13 [(google.api.field_behavior) = OPTIONAL];
}

// Information about a single stream that gets data out of the storage system.
Expand Down
2 changes: 1 addition & 1 deletion protos/google/cloud/bigquery/storage/v1/table.proto
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright 2021 Google LLC
// Copyright 2022 Google LLC
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ service BigQueryStorage {
option (google.api.default_host) = "bigquerystorage.googleapis.com";
option (google.api.oauth_scopes) =
"https://www.googleapis.com/auth/bigquery,"
"https://www.googleapis.com/auth/bigquery.readonly,"
"https://www.googleapis.com/auth/cloud-platform";

// Creates a new read session. A read session divides the contents of a
Expand Down
10 changes: 9 additions & 1 deletion protos/protos.d.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

36 changes: 36 additions & 0 deletions protos/protos.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

28 changes: 22 additions & 6 deletions protos/protos.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading