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.

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

Swapped in the new ING JDBC wrapper and the DataStax OSS Driver #205

Merged
merged 7 commits into from
Oct 6, 2023

Conversation

kevin-atx
Copy link
Contributor

No description provided.

@maximevw
Copy link
Contributor

maximevw commented Sep 3, 2023

Hello @kevin-atx,

A new version of Cassandra JDBC wrapper (4.9.1) has been published and solved issues preventing it to work correctly with Liquibase, it should be better to use this version instead of 4.9.0.

Updated to the latest version of the JDBC wrapper since it was updated to work better with Liquibase.
@kevin-atx
Copy link
Contributor Author

@maximevw Thank you for the update! I made the pom file changes and will continue to work on getting this branch working.

@maximevw
Copy link
Contributor

maximevw commented Sep 5, 2023

@kevin-atx The bundle jar in lib folder should be updated as well because the dependency in pom references a system path, so just updating the version will have no effect. 😉

@kevin-atx
Copy link
Contributor Author

Thanks @maximevw - updated the new library in the lib folder as well. :)

@filipelautert
Copy link
Contributor

filipelautert commented Oct 5, 2023

@sfinias - hey, we discussed cassandra driver (incorrect) change some time ago. What do you think of this one?

# Conflicts:
#	src/main/java/liquibase/ext/cassandra/database/CassandraDatabase.java
@filipelautert filipelautert self-assigned this Oct 5, 2023
@sfinias
Copy link
Contributor

sfinias commented Oct 6, 2023

@filipelautert Looks good to me! 🚀

@filipelautert filipelautert merged commit eda9b1f into main Oct 6, 2023
5 checks passed
@filipelautert filipelautert deleted the kevin-new-driver-test branch April 15, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants