-
Notifications
You must be signed in to change notification settings - Fork 432
feat(idempotency): adding redis as idempotency backend #2567
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
leandrodamascena
merged 123 commits into
aws-powertools:develop
from
roger-zhangg:redis/newbackend
Jan 10, 2024
Merged
Changes from all commits
Commits
Show all changes
123 commits
Select commit
Hold shift + click to select a range
3d5062c
(redis): initial commit
leandrodamascena aed821a
feat(redis/idempotency): creating redis connections
leandrodamascena ffcf16d
feat(redis/idempotency): creating redis connections
leandrodamascena b5c5791
feat(redis/idempotency): fixing import
leandrodamascena 6902e73
feat(redis/idempotency): adding base class
leandrodamascena 504d258
feat(redis/idempotency): adding logic to get record
leandrodamascena 0d1e3e9
feat(redis/idempotency): adding expiry timeout
leandrodamascena fce91e5
feat(redis) - merging from develop
leandrodamascena 3126885
feat(redis) - refactoring connection and fixing mypy errors
leandrodamascena f9c1875
feat(redis) - merging from develop
leandrodamascena 81c141e
feat(redis) - removing wrong print
leandrodamascena 5d6b0d0
feat(redis) - removing fields and adding additional logic to validate…
leandrodamascena 656096f
Merge branch 'develop' into redis/newbackend
leandrodamascena 038926c
feat(redis) - adding redis as dev dependency
leandrodamascena ca9b16d
Update idempotency.md
Vandita2020 b8bcfe4
Update idempotency.md
Vandita2020 cbdf968
merge updates
roger-zhangg cbfd890
resolve regarding to ruben's comment
roger-zhangg ecabaff
resolve regarding to ruben's comment
roger-zhangg 2352161
local test, minor fixes
roger-zhangg b813836
add redis to extra dep
roger-zhangg 1a9d8d9
fix git issue
roger-zhangg 9859896
fix docstring
roger-zhangg 6bfac34
Merge remote-tracking branch 'pt/develop' into redis/newbackend
roger-zhangg 8c38af1
fix poetry, address some Leandro's suggestion
roger-zhangg cceba13
change redis connection
roger-zhangg ad26e31
add mock redis, redis validation, sentinel support
roger-zhangg 9c36fc8
fix test
roger-zhangg 7f3aba0
Merge branch 'develop' of https://github.com/aws-powertools/powertool…
roger-zhangg 46fb65e
add redis
roger-zhangg ba6532a
merging from develop
leandrodamascena 0861f2a
add doc, few todos still need to address
roger-zhangg a405652
Merge branch 'redis/newbackend' of github.com:roger-zhangg/aws-lambda…
roger-zhangg 9775eaa
add docs, test. Removed Connection
roger-zhangg e6300a8
fix test on delete
roger-zhangg 01ea6a4
Merge branch 'develop' of https://github.com/aws-powertools/powertool…
roger-zhangg fcd2009
add types-redis to pyproj
roger-zhangg 5414d5e
fix Literal for 3.7
roger-zhangg a741622
Merge branch 'develop' of https://github.com/aws-powertools/powertool…
roger-zhangg 9cb5f3f
add comment for delete
roger-zhangg 12b76c4
add redis as dev dep
roger-zhangg 65ce480
Merge branch 'develop' of https://github.com/aws-powertools/powertool…
roger-zhangg 2a40609
fix poetry
roger-zhangg 21c77ea
merging from develop
leandrodamascena 8851d2e
Merge branch 'develop' into redis/newbackend
leandrodamascena 205c201
Simplifying DX
leandrodamascena 9ff5e48
remove redis-config in doc,test
roger-zhangg 25bb868
merging from develop
leandrodamascena 7ceb45e
handle race condition
roger-zhangg 181538c
Merge branch 'redis/newbackend' of github.com:roger-zhangg/aws-lambda…
roger-zhangg 602c393
remove todo
roger-zhangg 28af439
change to use redis.set
roger-zhangg 6644217
support decode_response=False
roger-zhangg ab12c29
fix mock redis
roger-zhangg a7b136a
Merge branch 'develop' of https://github.com/aws-powertools/powertool…
roger-zhangg 1ea6a21
add test for no decode
roger-zhangg adee7af
fix docstring
roger-zhangg 50f4dfa
Merge branch 'develop' of https://github.com/aws-powertools/powertool…
roger-zhangg b732dc6
Merge branch 'develop' into redis/newbackend
leandrodamascena 05b7ff0
fix coverage
roger-zhangg 75758de
Merge branch 'develop' of https://github.com/aws-powertools/powertool…
roger-zhangg c93026c
Merge branch 'develop' into redis/newbackend
leandrodamascena 0addabb
Merge branch 'develop' into redis/newbackend
leandrodamascena b8f938c
add a test case to demostrate race condition
roger-zhangg 2b1db18
Merge branch 'redis/newbackend' of github.com:roger-zhangg/aws-lambda…
roger-zhangg 23de153
add a race condition test for empty record
roger-zhangg a64ba69
Merge branch 'develop' of https://github.com/aws-powertools/powertool…
roger-zhangg 2b27099
Merge branch 'develop' into redis/newbackend
leandrodamascena 7cb4e1f
add abs_lambda_path, protocol for redis client
roger-zhangg 5c639da
Merge branch 'redis/newbackend' of github.com:roger-zhangg/aws-lambda…
roger-zhangg 514ed7d
fix typing
roger-zhangg bebd9c6
remove awaitable
roger-zhangg f00cd3c
optimize protocol
roger-zhangg b23bc5c
Fix Bandit issue
leandrodamascena 5ae98ce
Refactoring integration tests to use testcontainers
leandrodamascena 2e5067b
Merging from develop
leandrodamascena f8490d1
Removing code smell
leandrodamascena 8d6aec5
fix makefile, remove sentinel setup
roger-zhangg 2d13a3c
Adding e2e tests
leandrodamascena 15de5db
Merging from develop
leandrodamascena 9e4f929
Merge branch 'develop' into redis/newbackend
leandrodamascena 5a07420
Merge branch 'develop' into redis/newbackend
leandrodamascena c95e65c
Merge branch 'develop' into redis/newbackend
leandrodamascena 0779aad
Merge branch 'develop' into redis/newbackend
leandrodamascena d30a0bf
Merging from develop
leandrodamascena ab22176
merging develop
693d731
Testing pipeline
leandrodamascena 12af90e
Removing things
leandrodamascena 0899f68
Removing things
leandrodamascena c3674c0
Fixing docstring and removing old code
leandrodamascena 68fa0c4
Improving the documentation
leandrodamascena 09544c1
Highlights + code removal
leandrodamascena 791315d
Merging from develop
leandrodamascena 2bc7987
Removing unnecessary tests
leandrodamascena 41f353e
Removing unnecessary tests
leandrodamascena 9418aaa
Documentation
leandrodamascena 064ffac
Merge branch 'develop' into redis/newbackend
leandrodamascena 9387d29
Merging from develop
leandrodamascena 2ae69f8
Addressing initial Ruben's feedback
leandrodamascena d87c869
Addressing Ruben's feedback - documentation
leandrodamascena 2d19e24
Addressing Ruben's feedback - docstring
leandrodamascena a031ced
Addressing Ruben's feedback - SSL
leandrodamascena 3174800
Addressing Ruben's feedback - db_index
leandrodamascena 8d31fa5
Addressing Ruben's feedback - db_index
leandrodamascena 33025d4
Addressing Ruben's feedback - db_index
leandrodamascena 122947a
redis comment improvements, minor refactor.
roger-zhangg 1369cc1
fix example and docstring import
roger-zhangg aa63492
fix import in inegration test
roger-zhangg ea48dfd
Addressing Ruben's feedback - Redis name
leandrodamascena 0a15505
Merge branch 'develop' into redis/newbackend
leandrodamascena 442f36f
Addressing Ruben's feedback - Comments
leandrodamascena b09db82
Merge branch 'develop' into redis/newbackend
leandrodamascena 30ec360
Minor changes in the documentation
leandrodamascena 5f836b9
Minor changes in the documentation
leandrodamascena 2826845
Merge branch 'develop' into redis/newbackend
leandrodamascena dc07eaf
Removing Redis as builti-in dependency in our layer
leandrodamascena 1e9337c
Adding Redis to install when creating a new env
leandrodamascena f570723
Adressing Ruben's feedback
leandrodamascena 255a720
Making sonar happy
leandrodamascena be03cc0
Merge branch 'develop' into redis/newbackend
leandrodamascena 371bed1
Merge branch 'develop' into redis/newbackend
leandrodamascena 698d147
e2e failing due to wrong import
de7b77f
Merge branch 'develop' into redis/newbackend
leandrodamascena 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 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
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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.