graphprotocol / graphprotocol/graph-node
Priority of toml config vs env vars for connection pool size
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Rust
- Estrellas
- 3.2k
- Forks
- 1.1k
- Merge medio
- 4 d 1 h
- PR fusionados (30 d)
- 1
Descripción
Do you want to request a feature or report a bug?
Neither
What is the current behavior?
It seems that setting the size of a connection pool with the config file takes precedence over using STORE_CONNECTION_POOL_SIZE.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Start a graph node which uses
[store]
[store.primary]
connection = "<%= pgString %>"
pool_size = [
{ node = ".*", size = 30 },
]
and start the node with STORE_CONNECTION_POOL_SIZE=50. Then check the pool size using graphman and see that the config takes precedence.
What is the expected behavior?
I would expect the env var to take precedence over the config. Though I understand the env var isn't as fine grained.
Maybe I'm misunderstanding what STORE_CONNECTION_POOL_SIZE is meant for relative to pool_size in [store]?
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.
Línea de trabajo
Reproduce la configuración usando el ajuste pool_size de TOML en [store.primary] y STORE_CONNECTION_POOL_SIZE=50; después, inspecciona el tamaño resultante del pool con graphman. Determina la precedencia prevista entre el ajuste detallado pool_size y la variable de entorno, y verifica el comportamiento elegido con una prueba específica o un resultado documentado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rust
- Área
- backend, databases
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100