-
Notifications
You must be signed in to change notification settings - Fork 1k
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
feat: Enable Vector database and retrieve_online_documents API #4061
Merged
Merged
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
713768e
feat: add document store
HaoXuAI 58d5d94
feat: add document store
HaoXuAI 2cd73d1
feat: add document store
HaoXuAI d2e0a59
feat: add document store
HaoXuAI 7079e7f
remove DocumentStore
HaoXuAI 8c9ee97
format
HaoXuAI 513dd39
Merge branch 'master' into feat-documentstore
HaoXuAI 29d98cd
format
HaoXuAI 11eb97f
format
HaoXuAI 865baf2
format
HaoXuAI 47cd117
format
HaoXuAI 3f9f59f
format
HaoXuAI 7935071
remove unused vars
HaoXuAI ba39f93
add test
HaoXuAI cf53c71
add test
HaoXuAI 92046af
format
HaoXuAI d0acd2d
format
HaoXuAI cc45f73
format
HaoXuAI 006b5c6
format
HaoXuAI 6e0ba03
format
HaoXuAI a2302be
fix not implemented issue
HaoXuAI 2e6fc55
fix not implemented issue
HaoXuAI 3cbbf21
fix test
HaoXuAI ec32764
format
HaoXuAI e2d8008
format
HaoXuAI 523d20f
format
HaoXuAI 5cd085d
format
HaoXuAI 795699e
format
HaoXuAI 67b007f
format
HaoXuAI 33b46bd
update testcontainer
HaoXuAI 82fe5f1
format
HaoXuAI 0618378
fix postgres integration test
HaoXuAI 7de2016
format
HaoXuAI 92fed1d
fix postgres test
HaoXuAI d4f2639
fix postgres test
HaoXuAI 396d7de
fix postgres test
HaoXuAI 6c38b92
fix postgres test
HaoXuAI f763dc9
fix postgres test
HaoXuAI 818c055
format
HaoXuAI a51b555
format
HaoXuAI 2624b22
format
HaoXuAI File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
16 changes: 12 additions & 4 deletions
16
sdk/python/feast/infra/online_stores/contrib/postgres_repo_configuration.py
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,18 @@ | ||
from feast.infra.offline_stores.contrib.postgres_offline_store.tests.data_source import ( | ||
PostgreSQLDataSourceCreator, | ||
) | ||
from tests.integration.feature_repos.integration_test_repo_config import ( | ||
IntegrationTestRepoConfig, | ||
) | ||
from tests.integration.feature_repos.universal.online_store.postgres import ( | ||
PGVectorOnlineStoreCreator, | ||
PostgresOnlineStoreCreator, | ||
) | ||
|
||
FULL_REPO_CONFIGS = [ | ||
IntegrationTestRepoConfig(online_store_creator=PostgreSQLDataSourceCreator), | ||
IntegrationTestRepoConfig( | ||
online_store="postgres", online_store_creator=PostgresOnlineStoreCreator | ||
), | ||
IntegrationTestRepoConfig( | ||
online_store="pgvector", online_store_creator=PGVectorOnlineStoreCreator | ||
), | ||
] | ||
|
||
AVAILABLE_ONLINE_STORES = {"pgvector": PGVectorOnlineStoreCreator} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's probably something to be said about having a configurable distance metric to let the user choose which way to get the
top_k
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, there are a bunch of different algorithms/configs for Postgresql to retrieve the documents. We can support it in the future after this PR