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.

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

dbt: test updates to onchain metrics mart (v2) #1272

Closed
wants to merge 4 commits into from

Conversation

ccerv1
Copy link
Member

@ccerv1 ccerv1 commented Apr 23, 2024

No description provided.

Copy link

vercel bot commented Apr 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kariba-network ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2024 4:59am
oso-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2024 4:59am
oso-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 24, 2024 4:59am

@oso-prs
Copy link

oso-prs bot commented Apr 23, 2024

Test deployment for PR #1272 successfully deployed to oso-pull-requests.pr_1272.

@vercel vercel bot temporarily deployed to Preview – oso-frontend April 23, 2024 02:11 Inactive
@vercel vercel bot temporarily deployed to Preview – oso-docs April 23, 2024 02:12 Inactive
@ccerv1 ccerv1 enabled auto-merge April 23, 2024 16:09
@ccerv1 ccerv1 disabled auto-merge April 23, 2024 16:10
@ccerv1 ccerv1 enabled auto-merge April 23, 2024 16:10
auto-merge was automatically disabled April 23, 2024 19:46

Merge queue setting changed

@vercel vercel bot temporarily deployed to Preview – oso-docs April 24, 2024 04:55 Inactive
@vercel vercel bot temporarily deployed to Preview – oso-frontend April 24, 2024 04:57 Inactive
@oso-prs
Copy link

oso-prs bot commented Apr 24, 2024

Test deployment for PR #1272 failed on comment 97ed540c008b55c3c038342c55f65be8bade2253. With error:

Error: Command failed: /home/runner/work/oso/oso/ops/external-prs/pr-clone/.venv/bin/dbt run

    at ChildProcess.exithandler (node:child_process:422:12)
    at ChildProcess.emit (node:events:517:28)
    at ChildProcess.emit (node:domain:489:12)
    at maybeClose (node:internal/child_process:1098:16)
    at ChildProcess._handle.onexit (node:internal/child_process:303:5)

@ccerv1 ccerv1 changed the title fix: groupby on summary metrics dbt: test updates to onchain metrics mart (v2) Apr 24, 2024
@even-wei
Copy link
Contributor

Hi @ryscheng, @ccerv1 👋🏼
As CL shared with you, we are playing around Recce with OSO PRs.
Here is the Lineage Diff of this PR:
image

To get access, you only need to

  1. Install Recce (pip install recce)
  2. Download oso_pr1272_recce_state.json
  3. Open Recce server in review mode with the Recce state file (recce server --review oso_pr1272_recce_state.json)

Since this state file is generated in our BigQuery environment, you won't be able to do further diffing actions. But you can see the information within the state file.

We are happy to help you to setup Recce CI for your project, so the review process for dbt related PRs can be much better.
Feel free to let me know any issues and how can we make the project better together. 🙌🏼

@ccerv1 ccerv1 closed this May 20, 2024
@ccerv1 ccerv1 deleted the dbt/onchain-metrics-v2-fix branch May 20, 2024 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants