Advanced Data Governance Summit HOL: SQL bug fixes and content cleanup#3330
Open
sfc-gh-ankgupta wants to merge 11 commits into
Open
Advanced Data Governance Summit HOL: SQL bug fixes and content cleanup#3330sfc-gh-ankgupta wants to merge 11 commits into
sfc-gh-ankgupta wants to merge 11 commits into
Conversation
- Remove 'AI-Powered' from guide heading, descriptions, and SQL script - Flip order in SQL: run SYSTEM$CLASSIFY manually first, then attach profile - Update comments: clarify manual test vs auto-classification intent - Update Step 2.4 header to reflect new flow
- Remove Section 2.5 (CREDITCARD custom classifier) from SQL script - Renumber subsequent steps 2.6-2.11 to 2.5-2.10 - Update overview list in SQL script - Remove 'custom classifiers' from key features table in guide.md - Remove CREDITCARD classifier reference from teardown comments
- Remove 'AI-Powered' from SQL file header - Move USE block to just before Step 3.2 SQL (UI sections first) - Add auto-classification disclaimer to Step 3.1 UI section - Restructure Step 3.2: Information Schema (primary/immediate) + corrected ACCOUNT_USAGE queries using LATERAL FLATTEN on RESULT JSON - Remove Step 3.5 Governance Gap Analysis from SQL file and Key Takeaways - Remove Governance Gap Analysis section from guide.md - Fix broken inline SQL in guide.md (replace with pointer to script)
- Remove TABLE_CATALOG (not available in preview schema) - Use dynamic RUN_ID subquery (latest run) instead of hardcoded UUIDs - Replace ad-hoc queries with 3 focused lab-ready queries: list runs, who has access, role summary with ARRAY_AGG
- Remove AI-Powered from file header - Replace 3 broken DATA_CLASSIFICATION_LATEST queries with TAG_REFERENCES (custom DATA_CLASSIFICATION tag has flat columns; classification_latest is table-level JSON requiring FLATTEN) - Update OBSERVE/Key Takeaways comments to match new view references - Preserved AI terminology where it describes Cortex Code itself or AI-assisted governance value (lines 23, 153, 446)
Contributor
👋 Helpful Information💡 Non-Image Files in Assets FolderNon-image files found in /assets folders will NOT be uploaded to snowflake.com. If you are referencing them in your guide, you should link to them directly (e.g. using permanent GitHub links).
These are informational messages and will not block your PR from being merged. |
Contributor
🔗 Staging Preview ReadyQuickstart:
Generated by GitHub Actions |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Targeted fixes and content cleanup for the Advanced Data Governance Summit HOL quickstart. Focus areas: correct broken SQL queries against
DATA_CLASSIFICATION_LATESTandENTITLEMENT_REPORTviews, simplify lab content, and clean up terminology.Changes
Step 2 (Classification & Policies)
Step 3 (Trust Center)
Step 6 (Cortex Code skills)
Teardown
Cross-cutting
Test plan