Add Dapr session storage option #1937
Conversation
…akefile for test commands - Added DaprSession class for managing sessions with Dapr state stores, supporting various backends like Redis and PostgreSQL. - Updated Makefile to include separate targets for unit tests, integration tests, and all tests. Signed-off-by: Filinto Duran <[email protected]>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Signed-off-by: Filinto Duran <[email protected]>
…istency values Signed-off-by: Filinto Duran <[email protected]>
Signed-off-by: Filinto Duran <[email protected]>
…ge test cases Signed-off-by: Filinto Duran <[email protected]>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Signed-off-by: Filinto Duran <[email protected]>
Signed-off-by: Filinto Duran <[email protected]>
Signed-off-by: Filinto Duran <[email protected]>
Signed-off-by: Filinto Duran <[email protected]>
…o reduce grpc warning Signed-off-by: Filinto Duran <[email protected]>
Signed-off-by: Filinto Duran <[email protected]>
Signed-off-by: Filinto Duran <[email protected]>
Signed-off-by: Filinto Duran <[email protected]>
|
@seratch thanks, I'll take care of the docs notes. In relation to the integration tests, I could move to a separate folder just for integration tests so running them can be done by specifying the folder, or I could add running guards, like if import dapr/testcontainers does not fail and and docker is present run it. Let me know what would you prefer |
Just skipping the integration tests when the prerequisites are not ready should be good to go 👍 |
Signed-off-by: Filinto Duran <[email protected]>
|
@seratch all items requested completed for when you have time. thanks |
Signed-off-by: Filinto Duran <[email protected]>
…o-close Signed-off-by: Filinto Duran <[email protected]>
This comment was marked as off-topic.
This comment was marked as off-topic.
Signed-off-by: Filinto Duran <[email protected]>
…containers, also show postgresql and redis at the same time Signed-off-by: Filinto Duran <[email protected]>
4 participants
Implement DaprSession for distributed session management and update makefile for test commands.
Dapr makes it easy to integrate with many different DB as shown in https://docs.dapr.io/reference/components-reference/supported-state-stores