Conversation
Added Oracle adapter support
There was a problem hiding this comment.
Thank you for opening this PR @vinodsugur, and sorry for the delay in getting to review this. Oracle support would definitely be a nice-to-have feature. The code in general looks good and I left a few comments. Can you address those? Can you also make sure that Oracle gets added to the supported adapters list in the README?
I have resolved open queries.
vinodsugur
left a comment
There was a problem hiding this comment.
I have resolved open queries. Please review.
|
|
I have closed on all open points. |
Added Oracle adapter support
Description & motivation
Checklist
integration_tests/models, dbt data tests inintegration_tests/testsor integration tests in the CI workflow)