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

V3 cleanup and signature changes #169

Merged
merged 24 commits into from
Aug 25, 2022
Merged

V3 cleanup and signature changes #169

merged 24 commits into from
Aug 25, 2022

Conversation

bartelink
Copy link
Collaborator

@bartelink bartelink commented Aug 18, 2022

Lots of restructuring prompted by
a) signature changes driven by profiling
b) implementation of FeedSourceBase.AwaitCompletion

See also #116:

  • cleanup naming and signatures for Sinks
    • rename ProjectorPipeline<'T> -> Sink
    • name Ingester batch tuples
  • Restructure Scheduler modules
  • rename Feed.Internal to Feed.Core
  • convert all Tuples to struct tuples
  • change StreamSpan from a record to a type alias
  • Linting e.g. remove ., ref etc
  • Extensive profiling-driven impl changes

Make Scheduler Generic
Add/Change Default EventBody to ROM
@bartelink bartelink changed the title Prep for AwaitSink V3 cleanup Aug 23, 2022
@bartelink bartelink mentioned this pull request Aug 25, 2022
22 tasks
@bartelink bartelink changed the title V3 cleanup V3 cleanup and signature changes Aug 25, 2022
@bartelink bartelink marked this pull request as ready for review August 25, 2022 12:38
@bartelink bartelink merged commit 8ed429c into master Aug 25, 2022
@bartelink bartelink deleted the cleanup branch August 25, 2022 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant