Remove tokio `rt-multi-thread` feature from `lambda_runtime` and `lambda_extension`, and possibly `rt` as well
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Revisa las declaraciones de dependencia de tokio en lambda-runtime/Cargo.toml y lambda-extension/Cargo.toml; después, busca en estos crates APIs habilitadas por las features rt y rt-multi-thread. Confirma si alguna de las dos features es necesaria y verifica que la configuración de dependencias siga compilando para los casos de uso de runtime compatibles; el issue se resuelve cuando se eliminan las features innecesarias o se documenta su necesidad.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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?
- Lenguaje dominante
- Rust
- Estrellas
- 3.6k
- Forks
- 397
- Merge medio
- 55 min
- PR fusionados (30 d)
- 1
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de aws/aws-lambda-rust-runtime
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
aws/aws-lambda-rust-runtime#1099 · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 74/100
aws/aws-lambda-rust-runtime#1165 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
aws/aws-lambda-rust-runtime#1161 ·
-
aws/aws-lambda-rust-runtime#1155 · 1 reacción · 1 asignado ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
aws/aws-lambda-rust-runtime#1131 · 1 comentario ·
Todos los issues de aws/aws-lambda-rust-runtime
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
bug good first issue package: quic
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
`dora trace view` sends a non-canonical full UUID as-is, so a valid trace ID shows "No spans found" Abiertocli coordinator rust
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
area: tasks enhancement good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Jason-jo17/Polybench#15 · 1 comentario ·