Skip to content

Comments

Use high-resolution timer for cache TTL on PHP 7.3+#35

Merged
WyriHaximus merged 1 commit intoreactphp:masterfrom
clue-labs:hrtime
Apr 27, 2019
Merged

Use high-resolution timer for cache TTL on PHP 7.3+#35
WyriHaximus merged 1 commit intoreactphp:masterfrom
clue-labs:hrtime

Conversation

@clue
Copy link
Member

@clue clue commented Apr 27, 2019

This project suggests that cache implementations SHOULD use a
monotonic time source if available. Given that a monotonic time source is
only available as of PHP 7.3 by default, event loop implementations MAY
fall back to using wall-clock time. http://php.net/manual/en/function.hrtime.php

Builds on top of reactphp/event-loop#182 and reactphp/event-loop#130

@clue clue added this to the v0.6.0 milestone Apr 27, 2019
@clue
Copy link
Member Author

clue commented Apr 27, 2019

Unrelated test failure on HHVM is being addressed in #36, all other tests pass 👍

@clue
Copy link
Member Author

clue commented Apr 27, 2019

Rebased on master to resolve merge conflict and test failure now that #36 is in :shipit:

@WyriHaximus WyriHaximus self-requested a review April 27, 2019 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants