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

update to handle motherduck #87

Closed
matsonj opened this issue Jun 23, 2023 · 7 comments · Fixed by #94
Closed

update to handle motherduck #87

matsonj opened this issue Jun 23, 2023 · 7 comments · Fixed by #94

Comments

@matsonj
Copy link
Owner

matsonj commented Jun 23, 2023

No description provided.

@Alex-Monahan
Copy link
Contributor

Woot! Following... 😄

@matsonj
Copy link
Owner Author

matsonj commented Jun 24, 2023

@Alex-Monahan can you poke internally for a key 😇

@Alex-Monahan
Copy link
Contributor

@Alex-Monahan can you poke internally for a key 😇

Happy to ask! Did you already apply for an invite on our site? https://motherduck.com/

@matsonj
Copy link
Owner Author

matsonj commented Jun 24, 2023

Yes I am in the queue!

@Alex-Monahan
Copy link
Contributor

Invite sent! Let me know if you have any questions or issues. Psyched to see what you build and happy to help along the way!

@matsonj
Copy link
Owner Author

matsonj commented Jun 27, 2023

Ok small issue, from the dbt-duckdb repo:

MotherDuck databases do not suppport transactions, so there is a new disable_transactions profile option that will be automatically enabled if you are connecting to a MotherDuck database in your path.

I am indeed using transactions to calculate ELO changes HOWEVER, it is deprecated and replaced in the "v2" version of this project. I suggest you follow along there, as I will handle it there.

@matsonj matsonj closed this as completed Jun 27, 2023
@matsonj matsonj reopened this Aug 15, 2023
@matsonj
Copy link
Owner Author

matsonj commented Aug 15, 2023

👀 👀

"node_modules/duckdb": {
"version": "0.8.1",
"resolved": "https://registry.npmjs.org/duckdb/-/duckdb-0.8.1.tgz",
"integrity": "sha512-a2SJDuvBVKy5muYFxXTANlqdNX1daF3NHzpqRdrk0Qx5n3Sh7BxL66O+WY9epaDFukiXEpz45sds5T1LaPaHog==",
"hasInstallScript": true,
"peer": true,
"dependencies": {
"@mapbox/node-pre-gyp": "^1.0.0",
"node-addon-api": "*",
"node-gyp": "^9.3.0"
}
},
"node_modules/duckdb-async": {
"version": "0.8.1",
"resolved": "https://registry.npmjs.org/duckdb-async/-/duckdb-async-0.8.1.tgz",
"integrity": "sha512-/g5/zxC5XXXeQVEs86AtWVnzOmS7FBgVm976sJ75Tvrovms+QC5V+0SLNOl6VJQUT55NvvlrwX1zQybQfa+48w==",
"peer": true,
"dependencies": {
"duckdb": "0.8.1"

@matsonj matsonj linked a pull request Aug 16, 2023 that will close this issue
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 a pull request may close this issue.

2 participants