-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add support for DynamoDB and S3 registry #1483
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
Merged
feast-ci-bot
merged 38 commits into
feast-dev:master
from
leonid133:feature/online_dynamodb
Jul 3, 2021
Merged
Changes from 1 commit
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
9e0e7c7
Add support for DynamoDB and S3 registry
leonid133 44aadd8
rcu and wcu as a parameter of dynamodb online store
leonid133 6383791
fix linter
leonid133 73ff67a
aws dependency to extras
leonid133 aa6d0da
FEAST_S3_ENDPOINT_URL
leonid133 0a87050
tests
leonid133 3b8bb31
merge from master
leonid133 00e8675
fix signature, after merge
leonid133 6a99cd9
aws default region name configurable
leonid133 32dc799
merge from master
leonid133 db616c4
add offlinestore config type to test
leonid133 8dcbd5a
review changes
leonid133 fee93dd
merge from master
leonid133 2bbe268
Merge branch 'master' of https://github.com/feast-dev/feast into feat…
leonid133 5d33a79
Merge branch 'master' of https://github.com/feast-dev/feast into feat…
leonid133 24c44ee
merge latest from master
leonid133 7b99cde
review requested changes
leonid133 3a985b0
integration test for Dynamo
leonid133 6973581
change the rest of table_name to table_instance (where table_name is …
leonid133 e928424
fix DynamoDBOnlineStore commit
leonid133 59d7e4c
move client to _initialize_dynamodb
leonid133 594b932
rename document_id to entity_id and Row to entity_id
leonid133 15a787c
The default value is None
leonid133 7eaa654
Remove Datastore from the docstring.
leonid133 1468117
get rid of the return call from S3RegistryStore
leonid133 5dbe429
merge two exceptions
leonid133 986d45e
For ci requirement
leonid133 79d85c7
remove configuration from test
leonid133 f50b2fb
feast-integration-tests for tests
leonid133 509c521
change test path
leonid133 cd67973
add fixture feature_store_with_s3_registry to test
leonid133 5466d20
merge from master
leonid133 3d1b78c
region required
leonid133 ff8d635
Merge branch 'master' of https://github.com/feast-dev/feast into feat…
leonid133 57a607c
Address the rest of the comments
e9422ea
Merge branch 'master' into feature/online_dynamodb
3cd9597
Update to_table to to_arrow
124b337
Merge branch 'master' into feature/online_dynamodb
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
aws default region name configurable
Signed-off-by: lblokhin <lenin133@yandex.ru>
- Loading branch information
commit 6a99cd9e07a98559f7db2d5cddaf64994c480cb4
There are no files selected for viewing
This file contains hidden or 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 hidden or 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
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.
Uh oh!
There was an error while loading. Please reload this page.