rosetta icon indicating copy to clipboard operation
rosetta copied to clipboard

Adding Redshift <-> MySql support

Open ToniIsmaili opened this issue 1 year ago • 2 comments

@jetroni While resolving a merge conflict the other PR got closed, here is the new and improved one!

Could you please review the new tests for Redshift?

ToniIsmaili avatar Aug 08 '24 12:08 ToniIsmaili

No worries. Tests are looking good now. Another small change I need you to do is inside the cli module there is a file named drivers.yaml, I need you to add the Redshift download link for the jar file. After this we will enable the tests (ci/cd) and go on from there.

jetroni avatar Aug 09 '24 12:08 jetroni

I went ahead and made the unit tests as discussed. I used a Postgres image since there's no official Redshift image available, but it should still cover what we need.

Let me know if there's anything that needs tweaking

ToniIsmaili avatar Sep 10 '24 11:09 ToniIsmaili