Skip to content

Conversation

@jiwhiz
Copy link

@jiwhiz jiwhiz commented Oct 8, 2024

Pinecone Java Client released 2.1.0, and currently spring-ai still uses 0.8.0 version.

@markpollack
Copy link
Member

markpollack commented Oct 8, 2024

Thanks! Will review post 1.0 M3 as this will bring in a new version of the protobuf library and could conflict with milvus and others that use it since typically version mismatches in this library are breaking changes.

this.namespace = builder.namespace;
this.contentFieldName = builder.contentFieldName;
this.distanceMetadataFieldName = builder.distanceMetadataFieldName;

Copy link
Member

Choose a reason for hiding this comment

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

@jiwhiz Thanks for the PR. Could you explain the reasoning for this change - removal of the some of the existing properties, client configuration and the usage of Index as part of this client version update?

@eddumelendez
Copy link
Contributor

it would be great to use 3.1.0 instead. Client can be configured to use a local emulator for testing.

@ilayaperumalg ilayaperumalg modified the milestones: 1.0.0-M6, 1.0.0-M7 Feb 6, 2025
@eddumelendez
Copy link
Contributor

BTW, Testcontainers provides a PineconeLocalContainer in version 1.20.5. See https://java.testcontainers.org/modules/pinecone/

@ilayaperumalg
Copy link
Member

@jiwhiz Thanks for your PR. After going through the pinecone java client changes, I understand the reason why you made these changes. I have submitted a separate PR to upgrade the pinecone java client to 4.0.1: #2328 and hence, closing this PR. Thank you for your contribution!

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