Skip to content

Add filter query support for GemFireVectorStore #2753

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jhuynh1
Copy link

@jhuynh1 jhuynh1 commented Apr 15, 2025

No description provided.

@jhuynh1 jhuynh1 force-pushed the prefilter-gemfire-vdb branch from 59e9a2a to 3e713b8 Compare April 15, 2025 17:06
@jhuynh1
Copy link
Author

jhuynh1 commented Apr 15, 2025

Hi @markpollack , this is a PR for adding filter queries to GemFireVectorStore. Thanks!

@sobychacko
Copy link
Contributor

@jhuynh1 I am reviewing this PR. The new test in GemFireVectorStoreIT is failing for me locally. The test is disabled by default; you need to enable it. Some other tests in the class are failing as well. It would help with the review/merge if you could look at that. Thank you!

@nabarunnag
Copy link

nabarunnag commented Jun 17, 2025

Hi, @sobychacko we have pushed a commit to the PR to fix the failing test. The failing test is passing now. Please do test it and let us know, if there is anything else we can do to get this merged.

Thank you again

@nabarunnag nabarunnag force-pushed the prefilter-gemfire-vdb branch 2 times, most recently from 5bd5d16 to b2c7baa Compare June 17, 2025 18:38
Signed-off-by: Nabarun Nag <[email protected]>
@nabarunnag nabarunnag force-pushed the prefilter-gemfire-vdb branch from b2c7baa to 2ed72a3 Compare June 17, 2025 18:41
@nabarunnag
Copy link

Tests are passing locally
Screenshot 2025-06-17 at 11 40 21 AM
Screenshot 2025-06-17 at 11 42 06 AM

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