Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

RM-13-fix-vertica-no-s-3-integration-test #204

Merged
merged 5 commits into from
Feb 3, 2023
Merged

RM-13-fix-vertica-no-s-3-integration-test #204

merged 5 commits into from
Feb 3, 2023

Conversation

ryantimjohn
Copy link
Contributor

No description provided.

@ryantimjohn ryantimjohn requested a review from Brunope February 2, 2023 11:23
@ryantimjohn
Copy link
Contributor Author

@Brunope well, uh... this just works now. =)
Assume this is because of the changes we made for other DBs fixing whatever was wrong with Vertica.
I pulled in the changes from #203 just to make the tests run quicker, we can merge that one first.

@ryantimjohn ryantimjohn marked this pull request as ready for review February 2, 2023 11:25
@ryantimjohn ryantimjohn mentioned this pull request Feb 2, 2023
Merged
@ryantimjohn ryantimjohn merged commit 5025f8c into master Feb 3, 2023
@ryantimjohn ryantimjohn deleted the RM-13-fix-vertica-no-s-3-integration-test branch February 3, 2023 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Brunope Brunope Brunope approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ryantimjohn @Brunope