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

add dflow to dex_solana.trades #7405

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Conversation

smyyguy
Copy link
Contributor

@smyyguy smyyguy commented Dec 31, 2024

add dflow to dex_solana.trades

@github-actions github-actions bot marked this pull request as draft December 31, 2024 18:12
@github-actions github-actions bot added WIP work in progress dbt: solana covers the Solana dbt subproject labels Dec 31, 2024
@jeff-dude
Copy link
Member

i believe you're hitting a memory error on the cross-project base trades spell, due to large data sizes on each. can you try to hardcode a short timeframe in that spell, to see if we can get it to run on subset of the data as expected?

@jeff-dude jeff-dude self-assigned this Jan 2, 2025
@jeff-dude jeff-dude added the in review Assignee is currently reviewing the PR label Jan 2, 2025
@smyyguy
Copy link
Contributor Author

smyyguy commented Jan 3, 2025

e

hmm not really sure why that didnt take if that was the issue

@jeff-dude
Copy link
Member

hmm not really sure why that didnt take if that was the issue

we need to put timestamp in front of hardcoded date/times, it's a dunesql syntax requirement

@jeff-dude jeff-dude removed the in review Assignee is currently reviewing the PR label Jan 21, 2025
@jeff-dude jeff-dude marked this pull request as ready for review January 23, 2025 19:29
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dbt: solana covers the Solana dbt subproject ready-for-review this PR development is complete, please review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants