-
Notifications
You must be signed in to change notification settings - Fork 68
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
V5 checklist #232
Milestone
Comments
Merged
This was referenced Jan 14, 2022
Merged
Closed
bartelink
added a commit
that referenced
this issue
Sep 2, 2022
Added: - `Equinox.Core`: `Category` base class, with `Decider` and `Stream` helper `module`s Changed: - Performance: Switch surface APIs to `struct` Tuples and Options where relevant, some due to `struct` changes in jet/FsCodec#82, and use `task` in hot paths - `Equinox.Decider`: `log` is now supplied via `Equinox.Category` - `Equinox.Decider`: `maxAttempts` with a default policy and an optional argument on `Transact*` APIs re #232 - `Equinox`: push `Serilog` dependency from `Equinox` out to `Equinox.Core`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Breaking change/things that are ugly queue:
CosmosStore
: will shift to usingAzure.Cosmos
when that happens. When that work might happen is subject to change. Here's an example of some teaser info.Cosmos
: Check if newer query engine optimizations might yield a better indexing strategy, see linked article in CosmosStore: Evaluate potential indexing strategy change re new query optimizations #274 (perhaps it can borrow the one that works in DynamoDB now given this enhancement?)Version
(or force a full Reload?)V4 vs V3:
IAsyncEnumerable
maxAttempts
and associated throwing behavior fromDecider
ctor toTransact
'sattempts
argument Remove net461; target net60; reformat #310task
; expose same (but retainingAsync
as the primary interface in order to avoid leaking continuation token mess) Performance: Tasks / Struct #337 V4 store alignment; DeciderCore #338EventStoreDb
: inCosmosStore
,BatchingPolicy
/QueryMaxItems
etc are de-emphasized - perhaps same should be done for the EventStore equivalents? (e.g.EventStore.Context
has an overload that requires one) V4 store alignment; DeciderCore #338netstandard2.1
/net6.0
Remove net461; target net60; reformat #310net6.0
Remove net461; target net60; reformat #310"ssEvt"
EventStore/SqlStreamStore: Log consistency #311Transact(interpret : 'state -> Async<'event list>)
Rename TransactAsync -> Transact #314Decider.TransactAsync
toTransact
Rename TransactAsync -> Transact #314replaceslater removedXXXStoreCategory.FromMemento
stuff with aLoadOption.FromMemento
(The system that previously used this has been decommissioned, so we may consider removing it until we have realistic use case justifying its continued existence) ResolveOption/FromMemento -> LoadOption #308AsyncCacheCell
->TaskCell
(andAsyncLazy
->LazyTask
) Rename AsyncCacheCell to TaskCell #432V3 breaking changes compared to V2:
will depend on Azure.Cosmos (aka V4)targetsMicrosoft.Azure.Cosmos
for now, as MS have parkedAzure.Cosmos
(aka V4) for the foreseeablenetcoreapp3.1
Direct
to match V3 SDK CosmosStore: Flip default Connection mode to Direct #281maxItemsInTip
is mandatory in CosmosStoreContext ctor Cosmos: Provide a clean way to examine endpoint Uri when working from connection string #284Equinox.Cosmos V4 does not currently implement support for ignoring cert validation for e.g. use with Cosmos Simulator. NowCosmos: Port Connector bypassCertificateValidation to V3 #236 resolves this; presumably V4 will offer similar when rebasedMicrosoft.Azure.Cosmos
V 3.11 adds a custom HTTP factory, this could be implemented)The text was updated successfully, but these errors were encountered: