ClickHouse/ClickHouse

MaterializedPostgreSQL engine type mapping for uint256

Chiusa

#59.224 aperta il 25 gen 2024

Vedi su GitHubGuida contributor
 (5 commenti) (3 reazioni) (0 assegnatari)C++ (8400 fork)batch import
experimental featurehelp wantedquestion

Metriche repository

Star
 (47.419 stelle)
Metriche merge PR
 (Merge medio 2g 2h) (1000 PR mergiate in 30 g)

Descrizione

Currently, Clickhouse has support to uint256 type. In PostgreSQL, this can achieved by creating a field with type numeric(78,0). However, when using MaterializedPostgreSQL engine, the automatic table creation fails as the maximum precision value for numeric is 76. Is it possible to change this? I understand Clickhouse can handle numbers that big.

Guida contributor

Direzione di ricerca
Il problema richiede il supporto per il mapping del tipo uint256 nel motore MaterializedPostgreSQL. Attualmente, il motore PostgreSQL di ClickHouse mappa numeric(78,0) a uint256 ma fallisce a causa di una precisione massima hardcoded di 76. Per indagare, esamina i file sorgente che implementano il motore MaterializedPostgreSQL, probabilmente sotto `src/Storages/PostgreSQL/`. Cerca il limite hardcoded '76' o la gestione della precisione nella logica di conversione dei tipi. La correzione implicherebbe l'adeguamento della precisione massima consentita a 78 o la gestione dinamica di precisioni maggiori. Rivedi la discussione nei commenti dell'issue per eventuali contesti aggiuntivi o soluzioni proposte.
Tech stack
cppsqlpostgresql
Dominio
database
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
basic C++ knowledgefamiliarity with MaterializedPostgreSQL engine
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45