Skip to content
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

Remove saga migration sample #6409

Merged
merged 3 commits into from Jan 19, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 0 additions & 1 deletion persistence/sql/index.md
Expand Up @@ -6,7 +6,6 @@ related:
- samples/sql-persistence/simple
- samples/sql-persistence/transitioning-correlation-ids
- samples/saga/sql-sagafinder
- samples/saga/migration
- persistence/upgrades/sql-2to3
- persistence/upgrades/sql-1to2
- persistence/upgrades/sql-1.0.0-1.0.1
Expand Down
16 changes: 2 additions & 14 deletions persistence/sql/migrating-from-nhibernate.md
Expand Up @@ -33,7 +33,7 @@ NHibernate has an `OutboxRecord` table shared by all endpoints, where in later v

## Outbox retention

If the outbox retention period is set to a very large period and the message throughput is high then such a migration will take a while to complete. It is recommended to keep the retention period as low as possible.
If the outbox retention period is set to a very large period and the message throughput is high then such a migration will take a while to complete. It is recommended to keep the retention period as low as possible.

- [NHibernate Deduplication record lifespan](/persistence/nhibernate/outbox.md#deduplication-record-lifespan)
- [SQL Persistence Deduplication record lifespan](/persistence/sql/outbox.md#deduplication-record-lifespan)
Expand All @@ -54,16 +54,4 @@ Downtime migration that uses custom scripting with deep knowledge on saga state
- Requires custom saga migration mappings
- Downtime is relative to the size of the data set
- Pretty easy if the saga state schema is fairly flat
- Required for outbox, timeouts, and subscriptions

### Runtime migration

Runtime migration means that both NHibernate and SQL Persistence are used. New saga instances are created via SQL Persistence while legacy instances stored using NHibernate eventually will be removed once these saga instance complete.

The [Saga migration from NHibernate to SQL Persistence sample](/samples/saga/migration/) shows how this can be achieved.

- Complex deployment
- Requires saga logic to be mainted for both persisters
- Does not require serialization knowledge
- No downtime
- Not very suitable for saga instances that never complete or when instances complete far into the future
- Required for outbox, timeouts, and subscriptions
3 changes: 1 addition & 2 deletions persistence/sql/saga.md
Expand Up @@ -6,7 +6,6 @@ related:
- samples/sql-persistence/simple
- samples/sql-persistence/transitioning-correlation-ids
- samples/saga/sql-sagafinder
- samples/saga/migration
- persistence/sql/saga-concurrency
- persistence/sql/sqlsaga
redirects:
Expand Down Expand Up @@ -37,7 +36,7 @@ NOTE: Using [delimited identifiers](https://technet.microsoft.com/en-us/library/
### Columns


#### ID
#### ID

The value of `IContainSagaData.Id`. Primary Key.

Expand Down
10 changes: 0 additions & 10 deletions samples/saga/migration/Core_7/Client/Client.csproj

This file was deleted.

54 changes: 0 additions & 54 deletions samples/saga/migration/Core_7/Client/Program.cs

This file was deleted.

15 changes: 0 additions & 15 deletions samples/saga/migration/Core_7/Client/ReplyHandler.cs

This file was deleted.

33 changes: 0 additions & 33 deletions samples/saga/migration/Core_7/PersistenceMigration.sln

This file was deleted.

This file was deleted.

45 changes: 0 additions & 45 deletions samples/saga/migration/Core_7/Server.New/Program.cs

This file was deleted.

12 changes: 0 additions & 12 deletions samples/saga/migration/Core_7/Server.New/Server.New.csproj

This file was deleted.

60 changes: 0 additions & 60 deletions samples/saga/migration/Core_7/Server.New/TestSaga.cs

This file was deleted.

7 changes: 0 additions & 7 deletions samples/saga/migration/Core_7/Server/DummyMessage.cs

This file was deleted.

14 changes: 0 additions & 14 deletions samples/saga/migration/Core_7/Server/NotFoundHandler.cs

This file was deleted.