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

Pc driver adapter #1280

Merged
merged 45 commits into from
May 30, 2023
Merged

Pc driver adapter #1280

merged 45 commits into from
May 30, 2023

Conversation

MarkWolters
Copy link
Collaborator

First cut of the pinecone driver adapter implementation.
Wraps the driver available at https://github.com/pinecone-io/pinecone-java-client/tree/main
Ops and Dispensers implemented for all types specified in https://github.com/pinecone-io/pinecone-java-client/blob/main/src/main/proto/vector_service.proto

Copy link
Contributor

@jeffbanks jeffbanks left a comment

Choose a reason for hiding this comment

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

A few nits, but looks good overall. I like the tests and test coverage you've added. Well done!

@jeffbanks
Copy link
Contributor

@MarkWolters one other suggestion. For the pinecone.md and pinecone-example.yaml files the convention is to place the markdown file in the resources folder itself and include an activities folder to contain the *.yaml files. This would remove the need for the docs folder in your case.

.vscode/

### Mac OS ###
.DS_Store
Copy link
Contributor

Choose a reason for hiding this comment

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

If you prefix it with a *, it would happily take care of it on all directories

Suggested change
.DS_Store
*.DS_Store

Copy link
Contributor

Choose a reason for hiding this comment

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

Why not update the core .gitignore at the root of the project instead?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm not clear if there is a change being requested here or not, please update to let me know?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, change requested. There is a .gitignore at the root of the project, and it should be adjusted rather than propogage more .gitignore files to manage.

</dependency>
</dependencies>

<build>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: empty build section can be removed

Copy link
Contributor

@jshook jshook left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for the improvements!

MarkWolters and others added 20 commits May 26, 2023 14:10
… 0.64.0 to 0.64.2 (#1288)

Snyk has created this PR to upgrade com.vladsch.flexmark:flexmark-ext-yaml-front-matter from 0.64.0 to 0.64.2.

See this package in Maven Repository:
https://mvnrepository.com/artifact/com.vladsch.flexmark/flexmark-ext-yaml-front-matter/

See this project in Snyk:
https://app.snyk.io/org/nosqlbench/project/024e32ec-7f80-485c-b7bf-f69d45f933ce?utm_source=github&utm_medium=referral&page=upgrade-pr
- introduce a new WriteOp to write directly to Venice
- support AVRO keys for reads and for writes
- remove useless metrics
….6.3 (#1294)

Snyk has created this PR to upgrade org.apache.tinkerpop:tinkergraph-gremlin from 3.6.2 to 3.6.3.

See this package in Maven Repository:
https://mvnrepository.com/artifact/org.apache.tinkerpop/tinkergraph-gremlin/

See this project in Snyk:
https://app.snyk.io/org/nosqlbench/project/73081f87-c1b7-4c91-b407-1cb299565642?utm_source=github&utm_medium=referral&page=upgrade-pr

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
…4.2 to 0.64.4 (#1293)

Snyk has created this PR to upgrade com.vladsch.flexmark:flexmark-html2md-converter from 0.64.2 to 0.64.4.

See this package in Maven Repository:
https://mvnrepository.com/artifact/com.vladsch.flexmark/flexmark-html2md-converter/

See this project in Snyk:
https://app.snyk.io/org/nosqlbench/project/453e17d8-dd56-4df1-a6f6-f11fbe9f6022?utm_source=github&utm_medium=referral&page=upgrade-pr

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
… 0.64.0 to 0.64.4 (#1292)

Snyk has created this PR to upgrade com.vladsch.flexmark:flexmark-ext-yaml-front-matter from 0.64.0 to 0.64.4.

See this package in Maven Repository:
https://mvnrepository.com/artifact/com.vladsch.flexmark/flexmark-ext-yaml-front-matter/

See this project in Snyk:
https://app.snyk.io/org/nosqlbench/project/453e17d8-dd56-4df1-a6f6-f11fbe9f6022?utm_source=github&utm_medium=referral&page=upgrade-pr

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
… 0.64.2 to 0.64.4

Snyk has created this PR to upgrade com.vladsch.flexmark:flexmark-ext-yaml-front-matter from 0.64.2 to 0.64.4.

See this package in Maven Repository:
https://mvnrepository.com/artifact/com.vladsch.flexmark/flexmark-ext-yaml-front-matter/

See this project in Snyk:
https://app.snyk.io/org/nosqlbench/project/024e32ec-7f80-485c-b7bf-f69d45f933ce?utm_source=github&utm_medium=referral&page=upgrade-pr
@MarkWolters MarkWolters marked this pull request as ready for review May 26, 2023 14:15
@jshook jshook merged commit 8381e5a into main May 30, 2023
6 checks passed
@msmygit
Copy link
Contributor

msmygit commented Jul 6, 2023

Fixes #1252

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

Successfully merging this pull request may close these issues.

None yet

7 participants