The Wayback Machine - https://web.archive.org/web/20201117190115/https://github.com/eventflow/EventFlow/pull/317
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

WIP: Event archives #317

Open
wants to merge 30 commits into
base: develop
from
Open

WIP: Event archives #317

wants to merge 30 commits into from

Conversation

@rasmus
Copy link
Member

@rasmus rasmus commented Apr 15, 2017

When a event store becomes too large, we would like the option to move some of the events to a different location.

@rasmus rasmus mentioned this pull request Apr 15, 2017
@codecov-io
Copy link

@codecov-io codecov-io commented Apr 15, 2017

Codecov Report

Merging #317 into develop will increase coverage by 11.24%.
The diff coverage is 69.93%.

Impacted file tree graph

@@             Coverage Diff              @@
##           develop     #317       +/-   ##
============================================
+ Coverage    73.59%   84.84%   +11.24%     
============================================
  Files          258      250        -8     
  Lines         6495     5338     -1157     
  Branches       537      427      -110     
============================================
- Hits          4780     4529      -251     
+ Misses        1485      585      -900     
+ Partials       230      224        -6
Impacted Files Coverage Δ
Source/EventFlow/Provided/Jobs/ArchiveEventsJob.cs 0% <0%> (ø)
Source/EventFlow/EventArchives/EventArchive.cs 100% <100%> (ø)
Source/EventFlow/EventFlowOptions.cs 78.7% <100%> (-0.91%) ⬇️
...rce/EventFlow/EventArchives/EventArchiveDetails.cs 100% <100%> (ø)
...ntFlow/Provided/ProvidedJobsConfigurationModule.cs 100% <100%> (ø) ⬆️
Source/EventFlow/Core/FileSystem.cs 41.66% <41.66%> (ø)
...entArchives/Files/FileEventArchiveConfiguration.cs 60% <60%> (ø)
.../EventArchives/Null/NullEventArchivePersistance.cs 66.66% <66.66%> (ø)
...ce/EventFlow/EventArchives/EventArchiveStreamer.cs 84.21% <84.21%> (ø)
...EventArchives/Files/FileEventArchivePersistance.cs 85.71% <85.71%> (ø)
... and 102 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7146edf...8448bc8. Read the comment docs.

rasmus added 20 commits Apr 15, 2017
# Conflicts:
# Source/EventFlow.TestHelpers/EventFlow.TestHelpers.csproj
# Source/EventFlow.Tests/EventFlow.Tests.csproj
# Source/EventFlow/EventFlow.csproj
rasmus added 4 commits Jul 9, 2017
# Conflicts:
# Source/EventFlow/Provided/ProvidedJobsConfigurationModule.cs
# Conflicts:
#	Source/EventFlow.Sql/Connections/ISqlConfiguration.cs
#	Source/EventFlow.Sql/Connections/SqlConfiguration.cs
rasmus added 2 commits Jul 11, 2017
@rasmus rasmus changed the title Event archives WIP: Event archives Sep 11, 2017
 Conflicts:
	Documentation/Jobs.rst
	Source/EventFlow/EventStores/Files/FilesEventPersistence.cs
@rasmus
Copy link
Member Author

@rasmus rasmus commented Oct 26, 2017

It seems like it would be optimal to wait for async streams and IAsyncEnumerable in C# 8 instead of inventing a wrapper

@rasmus rasmus removed the in progress label Mar 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.