New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 793489 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Feb 2018
Components:
EstimatedDays: 2
NextAction: ----
OS: ----
Pri: 1
Type: Feature



Sign in to add a comment

Modify bqschemaupdater to update documentation to make BigQuery tables easier to find

Project Member Reported by katthomas@chromium.org, Dec 8 2017

Issue description

we can update bqschemaupdater to include path to the proto file in the table description, so it shows up in  https://bigquery.cloud.google.com/table/chrome-infra-events:aggregated.cq_attempts?tab=details   a list of tables could be in a markdown file indeed. We don't have to list schemas in that file, only links to bq table (https://bigquery.cloud.google.com/table/chrome-infra-events:aggregated.cq_attempts) and high-level description (that does not change and so does not need to be updated by bqschemaupdater) <-- thanks nodir
 
EstimatedDays: 2
Labels: -Pri-2 Pri-1
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 22 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/e9b55a03cf051bc89eabf6626f9e92f7c47ca36b

commit e9b55a03cf051bc89eabf6626f9e92f7c47ca36b
Author: Katie Thomas <katthomas@google.com>
Date: Mon Jan 22 14:11:53 2018

[doc] add documentation for bigquery tables

R=nodir@chromium.org

Bug:  793489 
Change-Id: Id08f00c559243c20128a5962cd80a1eda5e8ddf4
Reviewed-on: https://chromium-review.googlesource.com/871524
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Commit-Queue: Katie Thomas <katthomas@google.com>

[add] https://crrev.com/e9b55a03cf051bc89eabf6626f9e92f7c47ca36b/doc/bigquery_tables.md
[modify] https://crrev.com/e9b55a03cf051bc89eabf6626f9e92f7c47ca36b/doc/users/event_pipeline.md

Project Member

Comment 4 by bugdroid1@chromium.org, Feb 7 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/luci/luci-go.git/+/92449ccf20a21b4cbf431b15be7c28a6388b0ade

commit 92449ccf20a21b4cbf431b15be7c28a6388b0ade
Author: Katie Thomas <katthomas@google.com>
Date: Wed Feb 07 18:46:13 2018

[bqschemaupdater] add proto location to table desc

Also fix the headers

R=nodir@chromium.org

Bug:  793489 
Change-Id: I887b6fb3596e728ad91e5a1e45d5352bf9761b39
Reviewed-on: https://chromium-review.googlesource.com/871498
Commit-Queue: Katie Thomas <katthomas@google.com>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>

[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/README.md
[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/dry_run_table_store.go
[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/dry_run_table_store_test.go
[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/local_table_store.go
[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/local_table_store_test.go
[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/main.go
[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/main_test.go
[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/schema.go
[modify] https://crrev.com/92449ccf20a21b4cbf431b15be7c28a6388b0ade/tools/cmd/bqschemaupdater/table_store.go

Status: Fixed (was: Started)

Sign in to add a comment