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

feat: support authorized UDF entries in dataset ACL#875

Merged
stephaniewang526 merged 1 commit intogoogleapis:masterfrom
pmakani:authorize-routines
Oct 27, 2020
Merged

feat: support authorized UDF entries in dataset ACL#875
stephaniewang526 merged 1 commit intogoogleapis:masterfrom
pmakani:authorize-routines

Conversation

@pmakani
Copy link
Copy Markdown
Contributor

@pmakani pmakani commented Oct 27, 2020

No description provided.

@pmakani pmakani requested review from a team, shollyman and stephaniewang526 October 27, 2020 12:39
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 27, 2020
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 27, 2020

Codecov Report

Merging #875 into master will decrease coverage by 0.00%.
The diff coverage is 75.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #875      +/-   ##
============================================
- Coverage     80.38%   80.38%   -0.01%     
- Complexity     1268     1270       +2     
============================================
  Files            79       79              
  Lines          6547     6570      +23     
  Branches        749      756       +7     
============================================
+ Hits           5263     5281      +18     
- Misses          891      893       +2     
- Partials        393      396       +3     
Impacted Files Coverage Δ Complexity Δ
...y/src/main/java/com/google/cloud/bigquery/Acl.java 72.00% <72.22%> (+0.57%) 14.00 <1.00> (+1.00)
...in/java/com/google/cloud/bigquery/DatasetInfo.java 95.55% <83.33%> (-0.43%) 36.00 <0.00> (+1.00) ⬇️

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 ce6fd55...e957521. Read the comment docs.

Copy link
Copy Markdown
Contributor

@shollyman shollyman left a comment

Choose a reason for hiding this comment

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

Changes themselves look good, but let's update the description for this change to describe things a bit better. Perhaps "feat: support authorized UDF entries in dataset ACL" or similar.

@stephaniewang526 stephaniewang526 changed the title feat: add acl for routine feat: support authorized UDF entries in dataset ACL Oct 27, 2020
@stephaniewang526
Copy link
Copy Markdown
Contributor

okay -- thank you @shollyman!

@stephaniewang526 stephaniewang526 merged commit b031447 into googleapis:master Oct 27, 2020
@pmakani pmakani deleted the authorize-routines branch October 27, 2020 17:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants