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.

Sign up for GitHub

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

Jump to bottom

chore: Add notes about managing grants on hybrid table #3368

Merged
merged 4 commits into from
Jan 31, 2025
Merged

chore: Add notes about managing grants on hybrid table #3368

merged 4 commits into from
Jan 31, 2025

Conversation

sfc-gh-jmichalak
Copy link
Collaborator

  • Add a note about managing grants on hybrid tables. Currently, this fails in Snowflake and grants should be managed through TABLE.
    • It has not been confirmed whether this is a bug in Snowflake or just in the docs (probably the former). See DOC-4953 for more info.
    • Add a test proving this behavior.
  • Add hybrid table client helper.
  • This change does not fix the issue, but it will be done in SNOW-1022645.

Test Plan

  • acceptance tests
  • …

References

#3300

@sfc-gh-jmichalak
Copy link
Collaborator Author

I checked other "special" tables, and for the moment HYBRID is the only one failing.

@github-actions GitHub Actions
Copy link

Integration tests failure for 4fa03a3494cc2b056e4f0e1db57a4eed03789b8e

@github-actions GitHub Actions
Copy link

Integration tests failure for ab64c54d856ae2667f23c2c0ac31868223d1f3b0

@sfc-gh-jmichalak sfc-gh-jmichalak merged commit e5e98bd into dev Jan 31, 2025
8 of 9 checks passed
@sfc-gh-jmichalak sfc-gh-jmichalak deleted the grant-ownership-hybrid-table branch January 31, 2025 12:14
@snowflake-release-please snowflake-release-please bot mentioned this pull request Feb 5, 2025
Merged
sfc-gh-jcieslak pushed a commit that referenced this pull request Feb 5, 2025
@snowflake-release-please
chore(main): release 1.0.3 (#3380)
af5d1e4 
🤖 I have created a release *beep* *boop*
---


##
[1.0.3](v1.0.2...v1.0.3)
(2025-02-05)


### 🔧 **Misc**

* Add basic performance tests
([#3349](#3349))
([c57346b](c57346b))
* Add notes about managing grants on hybrid table
([#3368](#3368))
([e5e98bd](e5e98bd))
* Add preview and stable categories to resources and data sources in
docs
([#3370](#3370))
([b061434](b061434))
* Fix docs and update the protected users list
([#3365](#3365))
([f23e8cb](f23e8cb))
* Grant ownership common use cases
([#3356](#3356))
([97813b6](97813b6))
* Metric level fix with object renaming guide
([#3376](#3376))
([629ff92](629ff92))
* Move technical guides to the guide directory
([#3371](#3371))
([2b95809](2b95809))
* Prepare authentication methods guide
([#3364](#3364))
([c4a1c5f](c4a1c5f))
* Publish Performance tests summary
([#3359](#3359))
([d30d002](d30d002))
* Resolve issues and document account resource
([#3360](#3360))
([46b7a9d](46b7a9d))
* Upgrade tfplugindocs
([#3361](#3361))
([89f2b0a](89f2b0a))


### 🐛 **Bug fixes:**

* change EnforceNetworkRulesForInternalStages ddl to parameter
([#3343](#3343))
([beb01c7](beb01c7))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-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
Reviewers

@sfc-gh-jcieslak sfc-gh-jcieslak sfc-gh-jcieslak approved these changes

@sfc-gh-asawicki sfc-gh-asawicki sfc-gh-asawicki approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sfc-gh-jmichalak @sfc-gh-jcieslak @sfc-gh-asawicki