delta-io/delta

[Kernel] Refactor ParquetBatchReader converter APIs

Open

#1.859 aberto em 23 de jun. de 2023

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)Scala (2.100 forks)batch import
enhancementgood first issuekernel

Métricas do repositório

Stars
 (8.807 stars)
Métricas de merge de PR
 (Mesclagem média 7d 1h) (142 fundiu PRs em 30d)

Description

As noted in the comments here https://github.com/delta-io/delta/pull/1856#discussion_r1239171329 https://github.com/delta-io/delta/pull/1856#discussion_r1239172398 refactor the changes in that PR.

  • Change moveToNextRow() --> moveToNextRow(long index) everywhere
  • Update FileRowIndexColumnConverter to not extend LongColumnConverter

Guia do colaborador

Direção de pesquisa
Siga a discussão do PR vinculado e implemente a refatoração especificada: renomear moveToNextRow() para moveToNextRow(long index) em toda a base de código e modificar FileRowIndexColumnConverter para não estender LongColumnConverter.
Pilha de tecnologia
scala
Domain
backendapi
Tipo Issue
Refatoração
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
ScalaGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60