Skip to content

Conversation

@palant
Copy link
Contributor

@palant palant commented Jun 28, 2024

This replaces BoxFuture by the type it aliases while adding the Sync trait. See #311 for the rationale behind this change.

@palant
Copy link
Contributor Author

palant commented Jun 28, 2024

And once again intermittent test failures in unrelated code. This time it’s the random number generator: you cannot reliably test distribution of random results, there will be occasional outliers. 🙄

@eaufavor eaufavor self-assigned this Jul 12, 2024
@eaufavor eaufavor added enhancement New feature or request WIP We are working on this feature internally labels Jul 12, 2024
@andrewhavck andrewhavck added Accepted This change is accepted by us and merged to our internal repo and removed WIP We are working on this feature internally labels Jul 19, 2024
eaufavor pushed a commit that referenced this pull request Jul 26, 2024
Includes-commit: 3faed99
Replicated-from: #312
eaufavor pushed a commit that referenced this pull request Jul 26, 2024
Includes-commit: 3faed99
Replicated-from: #312
eaufavor pushed a commit that referenced this pull request Jul 26, 2024
Includes-commit: 3faed99
Replicated-from: #312
@palant
Copy link
Contributor Author

palant commented Jul 27, 2024

This landed apparently: 18db42c

@palant palant closed this Jul 27, 2024
@palant palant deleted the sync-timeouts branch July 27, 2024 19:48
escoffier pushed a commit to escoffier/pingora that referenced this pull request Sep 6, 2024
xxargonaut added a commit to xxargonaut/pingora that referenced this pull request Feb 19, 2025
Includes-commit: 3faed99e264cda6ba3986912d06c6e67bf5df13b
Replicated-from: cloudflare/pingora#312
xxargonaut added a commit to xxargonaut/pingora that referenced this pull request Feb 19, 2025
Includes-commit: 3faed99e264cda6ba3986912d06c6e67bf5df13b
Replicated-from: cloudflare/pingora#312
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted This change is accepted by us and merged to our internal repo enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants