Remove tokio `rt-multi-thread` feature from `lambda_runtime` and `lambda_extension`, and possibly `rt` as well
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Esamina le dichiarazioni delle dipendenze di tokio in lambda-runtime/Cargo.toml e lambda-extension/Cargo.toml, quindi cerca in questi crate le API abilitate dalle feature rt e rt-multi-thread. Conferma se una delle due feature è necessaria e verifica che la configurazione delle dipendenze continui a compilare per i casi d’uso del runtime supportati; l’issue è risolta quando le feature non necessarie vengono rimosse o viene documentata la loro necessità.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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?
- Lingua principale
- Rust
- Stelle
- 3.6k
- Fork
- 397
- Merge medio
- 55m
- PR unite (30g)
- 1
Preparare l'ambiente
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di aws/aws-lambda-rust-runtime
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
aws/aws-lambda-rust-runtime#1099 · 1 reazione ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 74/100
aws/aws-lambda-rust-runtime#1165 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
aws/aws-lambda-rust-runtime#1161 ·
-
Add Lambda-Runtime-Invocation-Id header support for cross-wiring protectionForse di nuovo libera @darklight3it l’ha presa 61 giorni fa e non c’è nessuna pull request aperta. Aperta
aws/aws-lambda-rust-runtime#1155 · 1 reazione · 1 assegnatario ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
aws/aws-lambda-rust-runtime#1131 · 1 commento ·
Tutte le issue di aws/aws-lambda-rust-runtime
Issue simili
-
area:casework bug criticality:p3 triage:needs-implementation
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
registrystack/registry-stack#1623 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
leptos-rs/leptos#4885 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
longbridge/gpui-kit#3276 ·
I maintainer di solito rispondono entro 1 giorno
-
A-Migration Guides D-Straightforward S-Ready-For-Implementation X-Uncontroversial
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
bevyengine/bevy-website#2607 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno