Skip to content

v2: Cooler ❄️ #45

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.

Sign up for GitHub

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

Jump to bottom
Merged
PierreMardon merged 25 commits into from
Feb 6, 2025
Merged

v2: Cooler ❄️ #45

PierreMardon merged 25 commits into from
Feb 6, 2025

Conversation

@PierreMardon
Copy link
Contributor

@PierreMardon PierreMardon commented Feb 6, 2025

Closes #44

  • Paradigm change: retriers are now cold publishers
  • Core full rewriting: underlying publishers have been 100% rewritten at a lower level to stop using a combination of operators.
  • Demand and backpressure are now properly handled. You won't miss an event unless you want to.

@PierreMardon
Implement 100% cold retriers
8f379b3
@PierreMardon
Cleanup main code
daeef09
@PierreMardon
Add handleRetrierEvents modifier on JobRepeater
f2c4e7b
@PierreMardon
Delete all tests, minor adjustments
6303d16
@PierreMardon
Some adjustements, first tests, fixes
11c8e4e
@PierreMardon
Enhance DSL, add some protocols
1a8f9d3
@PierreMardon
Still tests
28b5505
@PierreMardon
Some more tests and fixes
3483db8
@PierreMardon
Some more tests
a0840ec
@PierreMardon
Edit README, add event handler on publishers
3b1cb1a
@PierreMardon
README edit
c81c498
@PierreMardon
Linting fix, README edit
0351cff
@PierreMardon
Fix async value
5b11a57
@PierreMardon
Minor fixes
27ec393
@PierreMardon
Fix condition publisher cancelling the retrier
bb243f6
@PierreMardon
Implement actual TrialPublisher and ConditionalRetrierPublisher
6100bd5
@PierreMardon
Renaming, move files, fix handleRetrierEvents and add test
539af8c
@PierreMardon
Create RepeatingTrialPublisher, first cleanup
f9bbea5
@PierreMardon
Lint fixes, some comments
e36c931
@PierreMardon
Cleanup TrialPublisher code and comment, add attempt failure index test
e9cc245
@PierreMardon
Add backpressure in README, optimize TrialPublisher end
aecda95
@PierreMardon
Minor renaming, remove useless call in ConditionalTrialPublisher
6de66bd
@PierreMardon PierreMardon self-assigned this Feb 6, 2025
@PierreMardon PierreMardon merged commit 3740fa9 into main Feb 6, 2025
2 checks passed
@PierreMardon PierreMardon deleted the feature/44-v2-cooler branch February 6, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@PierreMardon PierreMardon

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

v2: Cooler

2 participants

@PierreMardon