Skip to content

Refine Orleans log consistency provider configuration to support mixed usage #6

@coderabbitai

Description

@coderabbitai

Description

The current implementation in HostApplicationBuilderExtensions.Orleans.cs registers two default Orleans log consistency providers sequentially:

  • AddLogStorageBasedLogConsistencyProviderAsDefault()
  • AddStateStorageBasedLogConsistencyProviderAsDefault()

Since only one provider can be the default, the second call overrides the first. This needs to be revisited to properly configure a mixed approach where StateStorage is used for snapshots.

Context

This issue was identified during the code review of PR #5. The configuration needs refinement to support the intended mixed usage of both providers.

References

Suggested Resolution

Consider:

  1. Setting one provider as the default
  2. Registering the other as a named provider
  3. Configuring StateStorage specifically for snapshot functionality

Requested by: @snovak7

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

Status

Backlog

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions