feathr-ai/feathr

Add guardrails for materialize different keys

Aberta

#529 aberto em 29 de jul. de 2022

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)Scala (244 forks)batch import
good first issuepython

Métricas do repositório

Stars
 (1.929 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
O problema aponta para a linha 558 em feathr project/feathr/client.py. A tarefa é adicionar uma salvaguarda que levante uma exceção quando os usuários tentarem materializar recursos com chaves diferentes na mesma tabela online. Examine o método materialize atual e adicione uma verificação de consistência de chaves. Considere escrever um teste de unidade para verificar o comportamento.
Pilha de tecnologia
python
Domain
databackend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Python
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65