prestodb/presto

[native] Make "memory-pool-init-capacity" and "memory-pool-transfer-capacity" registered SystemConfig properties

Open

#21.345 aperta il 8 nov 2023

Vedi su GitHub
 (4 commenti) (1 reazione) (1 assegnatario)Java (5240 fork)batch import
buggood first issue

Metriche repository

Star
 (15.558 star)
Metriche merge PR
 (Merge medio 34g 14h) (120 PR mergiate in 30 g)

Descrizione

memory-pool-init-capacity and memory-pool-transfer-capacity are not registered SystemConfig properties in https://github.com/prestodb/presto/blob/master/presto-native-execution/presto_cpp/main/common/Configs.cpp#L131

This leads to warnings ("Unregistered properties from ...") during server initialization. This is misleading.

These properties should be registered SystemConfig properties.

Steps to Reproduce

Add entries for memory-pool-init-capacity and memory-pool-transfer-capacity in config.properties and start Prestissimo

"memory-arbitrator-kind": "SHARED"
"memory-pool-init-capacity": "536870912"
"memory-pool-transfer-capacity": "536870912"

Guida contributor

Direzione di ricerca
Aggiungi le voci per memory pool init capacity e memory pool transfer capacity alle proprietà SystemConfig in presto native execution/presto cpp/main/common/Configs.cpp seguendo il pattern di registrazione esistente.
Tech stack
cppjava
Dominio
backenddatabaseinfrastructure
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Presto internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80