Skip to content

Optimized Python Models #92

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jul 11, 2023
Merged

Optimized Python Models #92

merged 3 commits into from
Jul 11, 2023

Conversation

@aosingh
Copy link
Member

@aosingh aosingh commented Jul 11, 2023

Changed implementation to run Python models using ADB-S OML REST API

  • Changed profile parameter name from oml_auth_token_uri to oml_cloud_service_url
  • REST API supports parameters which SQL API pyqeval doesn't for example async, timeout and service
  • Better error messages shown to the user in case of user-script failures
  • Easy to start a job in async mode and poll job status

aosingh added 3 commits July 6, 2023 16:08
@aosingh
ADBS OML REST API and support for async execution of Python models
c58329d 
- Switched to REST API for remote Python execution
- Introduced config parameters: async_flag, timeout and service
- Better error reporting for Python models
- Tested with 10M data records
@aosingh
Changed signature of oracle__get_empty_subquery_sql based on dbt-core…
79c3552 
… upgrade
@aosingh
Test Case fix: Removed round brackets around constraint 'check' expre…
6c6cd59 
…ssion
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jul 11, 2023
@aosingh aosingh merged commit 45c7a27 into main Jul 11, 2023
@aosingh aosingh deleted the dev/async_python_models branch July 24, 2023 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ramapemmaraju ramapemmaraju ramapemmaraju approved these changes

@rkope99 rkope99 Awaiting requested review from rkope99

@spanchum spanchum Awaiting requested review from spanchum

Assignees

No one assigned

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aosingh @ramapemmaraju