-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
feat: add python model support #188
feat: add python model support #188
Conversation
@nicor88 Also, I want to use |
@Avinash-1394 have a lock at monkeypatch to mock specific calls from a function - https://www.patricksoftwareblog.com/monkeypatching-with-pytest/ Regarding the issue with cached_property I left you a suggestion, it's should work, but I didn't test it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM, we should add some documentation on:
- the requirements
- how it works
@Jrmyy I have updated the README with instructions and also provided docstrings to explain what happens behind the scenes. Let me know if I need to add more details. |
@nicor88 I have added one test but I am going to try and contribute to moto for these. But can we merge this and do that in a future MR? Or should we try to add more tests before we can merge this? |
@Avinash-1394 what you did here is an amazing feature, regarding mocking the requests could be done with mock patch, but let's consider to open an issue in moto and add a Todo in the tests. I want to take some more time to review the changes proposed here, also @Jrmyy @mattiamatrix could you have a look too? Ideally we include this in the next release :) |
Description
Support dbt python models by using Spark
Docs - https://docs.aws.amazon.com/athena/latest/ug/notebooks-spark.html
Bugs currently identified:
Prerequisites
~/.dbt/profiles.yml
file and the profile is referenced indbt_project.yml
Models used to test - Optional
You can add the below models to your dbt project or you can clone this repository and do poetry installation & run
model
python_table
python_incremental
Build
dbt output
Tests added
TODO:
Checklist