feathr-ai/feathr

Add guardrails for materialize different keys

Open

#529 aperta il 29 lug 2022

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Scala (244 fork)batch import
good first issuepython

Metriche repository

Star
 (1929 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

In theory, we can't materialize features of different keys into same online table as tables are always keyed by same key.

In the documentation, we do warn users not to materialize different keys to same table.

Right now, if users materialize features to the same table, a random group of the same key will materialize to the table. The other different groups will be discarded. The documentation is not fool-proof enough, so we should throw exception when users do this.

The code to modify is: https://github.com/linkedin/feathr/blob/main/feathr_project/feathr/client.py#L558

Guida contributor

Direzione di ricerca
Il problema si riferisce alla riga 558 in feathr project/feathr/client.py. Il compito è aggiungere una salvaguardia che sollevi un'eccezione quando gli utenti tentano di materializzare feature con chiavi diverse nella stessa tabella online. Esamina il metodo materialize corrente e aggiungi un controllo per la coerenza delle chiavi. Considera di scrivere un test unitario per verificare il comportamento.
Tech stack
python
Dominio
databackend
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
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65