@javornikolov Is there anything I need to change for this PR?

@javornikolov Is there anything I need to change for this PR?

Hi @nickpileggi947, thanks for your contribution! I need to look closer. One thing at first glimpse is that custom_libs content is supposed to be empty in the repo (it's intended for local tweaking). If a driver is not available in public artifact repo, we can upload it to our dedicated s3 place (I can take care of that).
Also - I am not sure we really need to change a Fitnesse properties file (I saw it in one of the commits).

@javornikolov Appreciate the first set of feedback, I have pushed another commit that should fix those. Let me know if there is anything else you see that needs to be updated.

@javornikolov just checking if there was anything else I needed to change on this PR?

@javornikolov Just checking in again to see if you (or any other approver) is happy with this PR?

@javornikolov All of the reviews have been closed and updated.

@nickpileggi947, I added a few comments which are relatively small. The rest looks good, a few outstanding things:

• May we squash the multiple commits into a single or few (we don't need the history of all small fixups).
• I wonder if it would be possible to somehow run snowflake integration tests on TravisCI. (We might postpone that if it's a pain)

@javornikolov I'll review and update these later today. But yea, we can squash these down to one large commit. And I would love to run the snowflake integration tests on TravisCI, but there is no free way to get a DB instance.

@javornikolov Sorry for the late response, I have made all of the updates and squashed the commits to one. Let me know if you need anything else.

@javornikolov Sorry for the late response, I have made all of the updates and squashed the commits to one. Let me know if you need anything else.

Thank you @nickpileggi947! Looks good to me 👍 I am merging it.