Skip to content

v1.0.1b1 - Bugfix: https://github.com/oracle/dbt-oracle/issues/4 #6

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
May 16, 2022

Conversation

aosingh
Copy link
Member

@aosingh aosingh commented May 13, 2022

  1. database param is optional. You can drop it from profiles.yml
  2. Introduced a new macro get_database_name() which can be used if database param is missing
  3. Overrides adapter methods - list_relations_without_caching(), get_relation()
  4. Override catalog specific method - _get_one_catalog()
  5. Modified snapshot macros to invoke get_database_name() if database param is missing

aosingh added 3 commits May 12, 2022 17:41
1. database param is optional. You can drop it from profiles.yml
2. Introduced a new macro get_database_name() which can be used if database param is missing
3. Overrides adapter methods - list_relations_without_caching(), get_relation()
4. Override catalog specific method - _get_one_catalog()
5. Modified snapshot macros to invoke get_database_name() if database param is missing
1. Removed unnecessary imports from oracle/impl.py
2. Removed UPPER() from get_database_name() macro. This is not needed
1. Updated sample_profiles.yml and profile_template.yml to remove database
@ThoSap
Copy link
Contributor

ThoSap commented May 17, 2022

Fixes #4

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants