aws / aws/aws-lambda-rust-runtime
Remove tokio `rt-multi-thread` feature from `lambda_runtime` and `lambda_extension`, and possibly `rt` as well
- Langage dominant
- Rust
- Étoiles
- 3.6k
- Forks
- 396
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
Currently the `lambda_runtime` crate has an unconditional dependency on tokio withe `rt-multi-thread` feature enabled:
- https://github.com/awslabs/aws-lambda-rust-runtime/blob/main/lambda-runtime/Cargo.toml#L53
- https://github.com/awslabs/aws-lambda-rust-runtime/blob/main/lambda-extension/Cargo.toml#L35
I'm struggling to see anywhere this dependency is actually used. Meanwhile, given that lambdas only receive one request at a time, it will frequently make sense for callers to only want to use the current thread runtime anyway (ref: #985 ).
I don't think we even need the `rt` (not multithreaded) feature for `lambda-runtime`, not seeing anywhere we are spawning or interacting with feature gated APIs. Though, for #983 we would need it, so probably ok to leave in?
Anyway, is there a reason we need to depend on the multithreaded runtime feature? Am I missing something?
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Examinez les déclarations de dépendances de tokio dans lambda-runtime/Cargo.toml et lambda-extension/Cargo.toml, puis recherchez dans ces crates les APIs activées par les features rt et rt-multi-thread. Confirmez si l’une ou l’autre de ces features est requise et vérifiez que la configuration des dépendances se compile toujours pour les cas d’utilisation du runtime pris en charge ; l’issue est résolue lorsque les features inutiles sont supprimées ou que leur nécessité est documentée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- rust
- Domaine
- backend
- Type d'issue
- Refactorisation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100