apache/pinot

Adding validation check to prevent creating two columns with different cases in schema

Chiusa

#6644 aperta il 4 mar 2021

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Java (1234 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (4937 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Although pinot columns can be case-sensitive, I think we should still prevent the scenarios like this. This easily breaks the external case-insensitive integrations like Presto.

E.g. in the same schema, we shouldn't allow creating two columns like: memberId and MemberID.

Guida contributor

Direzione di ricerca
Guarda il codice di validazione dello schema nel codebase di Pinot, specificamente dove le definizioni delle colonne vengono analizzate e memorizzate, per aggiungere un controllo che prevenga nomi di colonne duplicati senza distinzione tra maiuscole e minuscole.
Tech stack
java
Dominio
backenddatabase
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaSQL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60