-
Notifications
You must be signed in to change notification settings - Fork 199
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] Add support for TTL and time/interval formats in actor timers/reminders #652 #658
base: master
Are you sure you want to change the base?
[WIP] Add support for TTL and time/interval formats in actor timers/reminders #652 #658
Conversation
8d55d05
to
2285239
Compare
@Giovds Is there any progress in this? |
@mukundansundar I haven't had the time to finish this yet, and I will still be unavailable next week. I expect to have time again afterwards. I think what is left is to write integration tests to prove that it works, but I couldn't get them to run them locally yet. |
e63203d
to
76945aa
Compare
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
There is no need to create an instance of the DurationUtils or to extend it. Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Since the class is package-private there is no need to make the getters public. Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
Signed-off-by: Giovanni van der Schelde <[email protected]>
76945aa
to
3f6b554
Compare
@Giovds nudge - is there any update on this PR ? |
Codecov Report
@@ Coverage Diff @@
## master #658 +/- ##
============================================
+ Coverage 77.62% 78.09% +0.47%
- Complexity 993 1028 +35
============================================
Files 91 92 +1
Lines 3128 3196 +68
Branches 342 349 +7
============================================
+ Hits 2428 2496 +68
- Misses 534 535 +1
+ Partials 166 165 -1
Continue to review full report at Codecov.
|
@mukundansundar #652 (comment) Mainly stuck at creating integration tests and running them locally. |
gentle ping - @Giovds. Please see README as to how to run the tests locally |
Description
This PR should add the following items:
ActorTimers
ActorReminders
Integration test
Issue reference
The issue this PR will close: #652
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: