Skip to content

Conversation

@michaelsembwever
Copy link
Contributor

DRAFT until 4.18.2 is released.

ref: JAVA-3118 – apache/cassandra-java-driver#1931

Thank you for taking time to contribute this pull request!
You might have already read the [contributor guide][1], but as a reminder, please make sure to:

  • Sign the contributor license agreement
  • Rebase your changes on the latest main branch and squash your commits
  • Add/Update unit tests as needed
  • Run a build and make sure all tests pass prior to submission

<oracle.version>23.4.0.24.05</oracle.version>
<postgresql.version>42.7.2</postgresql.version>
<cassandra.java-driver.version>4.18.1</cassandra.java-driver.version>
<cassandra.java-driver.version>4.18.2-SNAPSHOT</cassandra.java-driver.version>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

todo

@michaelsembwever michaelsembwever force-pushed the mck/vector-support__JAVA-3118 branch from ab8463f to 3d10e48 Compare November 25, 2024 12:24
@markpollack
Copy link
Member

@michaelsembwever Thanks! Any timeline you are aware of for the new release of the java driver? It isn't scheduled at part of a release in the github issue.

@michaelsembwever
Copy link
Contributor Author

Any timeline you are aware of for the new release of the java driver?

I think as soon as apache/cassandra-java-driver#1931 lands then we'll start the 4.18.2 release. The release process itself takes 3-5 days, and ball-parking the PR maybe is a week or two away.

@michaelsembwever
Copy link
Contributor Author

btw, the -Dskip.vectorstore.cassandra=false is a real pita.

It creeps into everything :(
and when you explicitly specify a test it's not intuitive behaviour

# this should work, it's pretty explicit in what it's asking for…
mvn failsafe:integration-test -Dit.test=CassandraVectorStoreIT

@ericbottard
Copy link
Member

Manually cherry-picked changes and implemented as 3fc1ed6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants