aws / aws/aws-lambda-rust-runtime
Remove tokio `rt-multi-thread` feature from `lambda_runtime` and `lambda_extension`, and possibly `rt` as well
- Lenguaje dominante
- Rust
- Estrellas
- 3.6k
- Forks
- 396
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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?
Guía de contribución
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.
Evaluación
- Stack tecnológico
- rust
- Área
- backend
- Tipo de issue
- Refactorización
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100