Make OracleDbHelper.get_sqlalchemy_connection_string use oracle+oracledb instead of oracle

[leorudczenko]

Summary

When etlhelper was updated to v1.0.0, one of the major changes was replacing the use of cx_Oracle with oracledb. This is great, but the method OracleDbHelper.get_sqlalchemy_connection_string was not updated to reflect this change.

On this line, the string begins with oracle://, which will default to oracle+cx_oracle://, which is now deprecated in etlhelper.

Solution

To explicitly use oracledb, the connection string must start with oracle+oracledb:// instead. This was found from a stackoverflow post here, with a link to the official documentation on this being here.

Acceptance Criteria

• oracle:// is replaced with oracle+oracledb:// in the method OracleDbHelper.get_sqlalchemy_connection_string