Skip to content

BigQuery: Augment BQ Model metadata. - #5214

Merged
shollyman merged 2 commits into
googleapis:masterfrom
shollyman:model-stats
May 24, 2019
Merged

BigQuery: Augment BQ Model metadata.#5214
shollyman merged 2 commits into
googleapis:masterfrom
shollyman:model-stats

Conversation

@shollyman

@shollymanshollyman commented May 23, 2019

Copy link
Copy Markdown
Contributor

This change introduces extended ML model metadata into the existing CRUD
implementation for BigQuery ML models. This implementation exposes the
underlying discovery types directly without an abstraction for the ML metadata.

Methods that provide this metadata are all marked @BetaApi to indicate the getters may change.

This change introduces extended ML model metadata into the existing CRUD
implementation for BigQuery ML models. This implementation exposes the
underlying discovery types directly without an abstraction for the ML metadata.
Methods that provide this metadata are all marked @BetaApi to indicate the getters may change.
@shollyman
shollyman requested a review from a teamMay 23, 2019 09:51
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label May 23, 2019
@shollyman
shollyman requested a review from tswastMay 23, 2019 10:05
@codecov

codecovBot commented May 23, 2019

Copy link
Copy Markdown

Codecov Report

Merging #5214 into master will increase coverage by <.01%.
The diff coverage is 58.97%.

Impacted file tree graph

@@ Coverage Diff @@## master #5214 +/- ##
============================================
+ Coverage 50.41% 50.42% +<.01% - Complexity 23796 23799 +3 
============================================
Files 2251 2251 Lines 226836 226874 +38 Branches 24966 24969 +3 ============================================
+ Hits 114370 114392 +22 - Misses 103858 103871 +13 - Partials 8608 8611 +3
Impacted FilesCoverage ΔComplexity Δ
...src/main/java/com/google/cloud/bigquery/Model.java67.27% <0%> (-8.24%)12 <0> (ø)
.../main/java/com/google/cloud/bigquery/BigQuery.java78.44% <100%> (+0.26%)0 <0> (ø)⬇️
...main/java/com/google/cloud/bigquery/ModelInfo.java76.11% <66.66%> (-2.73%)21 <3> (+3)

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 03bf10c...1f817f2. Read the comment docs.

@tswasttswast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. This is so much nicer than if we had to duplicate all those properties manually.

@shollyman

Copy link
Copy Markdown
ContributorAuthor

LGTM. This is so much nicer than if we had to duplicate all those properties manually.

The combination of the annotation, that the representations are effectively read only, and the fact that we pin the discovery artifacts in the maven dependencies makes me much more comfortable here. Assuming no other comments come up, I'll merge it when I'm back in the office tomorrow my time.

@shollyman
shollyman merged commit 6636539 into googleapis:masterMay 24, 2019
@shollyman
shollyman deleted the model-stats branch May 24, 2019 10:29
shollyman added a commit to shollyman/google-cloud-java that referenced this pull request Jul 12, 2019
This is a breaking change to a @BetaApi within BigQuery.
It swaps in a new veneer StandardSQLField type in place of the underlying
discovery type for getting information about the feature and label columns
from ML models, which was functionality introduced in:
googleapis#5214
shollyman added a commit that referenced this pull request Jul 18, 2019
…ns. (#5732)
This is a breaking change to a @BetaApi within BigQuery.
It swaps in a new veneer StandardSQLField type in place of the underlying
discovery type for getting information about the feature and label columns
from ML models, which was functionality introduced in:
#5214
stephaniewang526 pushed a commit to googleapis/java-bigquery that referenced this pull request Dec 6, 2019
…ns. (#5732)
This is a breaking change to a @BetaApi within BigQuery.
It swaps in a new veneer StandardSQLField type in place of the underlying
discovery type for getting information about the feature and label columns
from ML models, which was functionality introduced in:
googleapis/google-cloud-java#5214
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@shollyman@sduskis@tswast@googlebot