Skip to content
Snippets Groups Projects

add entry for snowflake development databases, including retention policy

Merged Justin Stark requested to merge add-docs-for-dev-dbs into main
All threads resolved!

Why is this change being made?

Clarifies the use of development databases in snowflake, adds information and links about development and development databases cleaning

corresponding change to data project https://gitlab.com/gitlab-data/analytics/-/merge_requests/10434

Author and Reviewer Checklist

Please verify the check list and ensure to tick them off before the MR is merged.

  • Provided a concise title for this Merge Request (MR)
  • Added a description to this MR explaining the reasons for the proposed change, per say why, not just what
    • Copy/paste the Slack conversation to document it for later, or upload screenshots. Verify that no confidential data is added, and the content is SAFE
  • Assign reviewers for this MR to the correct Directly Responsible Individual/s (DRI)
    • If the DRI for the page/s being updated isn’t immediately clear, then assign it to one of the people listed in the Maintained by section on the page being edited
    • If your manager does not have merge rights, please ask someone to merge it AFTER it has been approved by your manager in #mr-buddies
    • The when to get approval handbook section explains the workflow in more detail
  • For transparency, share this MR with the audience that will be impacted.
    • Team: For changes that affect your direct team, share in your group Slack channel
    • Department: If the update affects your department, share the MR in your department Slack channel
    • Division: If the update affects your division, share the MR in your division Slack channel
    • Company: If the update affects all (or the majority of) GitLab team members, post an update in #whats-happening-at-gitlab linking to this MR

Commits

  • add entry for dev db

Edited by Justin Stark

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Justin Stark
  • Justin Stark
  • Justin Stark added 1 commit

    added 1 commit

    Compare with previous version

  • Justin Stark added 1 commit

    added 1 commit

    Compare with previous version

  • Justin Stark resolved all threads

    resolved all threads

  • Justin Stark added 1 commit

    added 1 commit

    Compare with previous version

  • Using combined codequality.json

    ️ Pipeline Failure - Linting Errors

    One of the linters has reported errors and as a result the pipeline has failed. Once the pipeline completes, you'll find the code quality report above which can link you to where the error is in your code. Additionally, below you'll find a table of the errors. The table has links to the lint rules so you can find more information on how to fix the issue(s).

    Rule File Line Error
    MD012 content/handbook/business-technology/data-team/platform/dbt-guide.md 53 Multiple consecutive blank lines [Expected: 1; Actual: 2]

    If you need more help please reach out on Slack in #mr-buddies.

  • Justin Stark added 1 commit

    added 1 commit

    Compare with previous version

  • Justin Stark changed the description

    changed the description

  • Justin Stark marked the checklist item Provided a concise title for this [Merge Request (MR)][mr] as completed

    marked the checklist item Provided a concise title for this [Merge Request (MR)][mr] as completed

  • Justin Stark marked the checklist item Added a description to this MR explaining the reasons for the proposed change, per [say why, not just what][say-why-not-just-what] as completed

    marked the checklist item Added a description to this MR explaining the reasons for the proposed change, per [say why, not just what][say-why-not-just-what] as completed

  • Justin Stark marked the checklist item Assign reviewers for this MR to the correct [Directly Responsible Individual/s (DRI)][dri] as completed

    marked the checklist item Assign reviewers for this MR to the correct [Directly Responsible Individual/s (DRI)][dri] as completed

  • Justin Stark marked the checklist item For transparency, share this MR with the audience that will be impacted. as completed

    marked the checklist item For transparency, share this MR with the audience that will be impacted. as completed

  • Using combined codequality.json

    ️ Pipeline Failure - Linting Errors

    One of the linters has reported errors and as a result the pipeline has failed. Once the pipeline completes, you'll find the code quality report above which can link you to where the error is in your code. Additionally, below you'll find a table of the errors. The table has links to the lint rules so you can find more information on how to fix the issue(s).

    Rule File Line Error
    MD012 content/handbook/business-technology/data-team/platform/dbt-guide.md 55 Multiple consecutive blank lines [Expected: 1; Actual: 2]

    If you need more help please reach out on Slack in #mr-buddies.

  • Justin Stark
  • Justin Stark resolved all threads

    resolved all threads

  • Justin Stark added 1 commit

    added 1 commit

    • 56330706 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Author Developer

    Post to slack once merged

    We've just merged an update to our weekly snowflake cleaning DAG that drops all tables in development databases that are 80+ days stale. Additionally, we've updated the handbook to clarify the expected use of development databases. You can query PROD.DATA_QUALITY.STALE_DEV_DB_TABLES to see if any tables in your development database will be affected.

  • Justin Stark changed the description

    changed the description

  • Justin Stark changed title from add entry for dev db to add entry for snowflake development databases, including retention policy

    changed title from add entry for dev db to add entry for snowflake development databases, including retention policy

  • Peter Empey approved this merge request

    approved this merge request

  • Justin Stark mentioned in merge request !7094 (merged)

    mentioned in merge request !7094 (merged)

  • Dennis van Rooijen
  • Justin Stark resolved all threads

    resolved all threads

  • Justin Stark added 1 commit
  • Using combined codequality.json

    ️ Pipeline Failure - Linting Errors

    One of the linters has reported errors and as a result the pipeline has failed. Once the pipeline completes, you'll find the code quality report above which can link you to where the error is in your code. Additionally, below you'll find a table of the errors. The table has links to the lint rules so you can find more information on how to fix the issue(s).

    Rule File Line Error
    MD022 content/handbook/business-technology/data-team/platform/dbt-guide.md 56 Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Above]

    If you need more help please reach out on Slack in #mr-buddies.

  • Justin Stark
  • Justin Stark resolved all threads

    resolved all threads

  • Justin Stark added 1 commit

    added 1 commit

    • aec30074 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Justin Stark added this merge request to the merge train at position 2

    added this merge request to the merge train at position 2

  • merged

  • Justin Stark mentioned in commit 6fa7cea2

    mentioned in commit 6fa7cea2

  • Please register or sign in to reply
    Loading