delta-io/delta

[Kernel] Refactor ParquetBatchReader converter APIs

Chiusa

#1859 aperta il 23 giu 2023

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (1 assegnatario)Scala (2159 fork)batch import
enhancementgood first issuekernelstale

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Segui la discussione del PR collegato e implementa il refactoring specificato: rinominare moveToNextRow() in moveToNextRow(long index) in tutto il codice e modificare FileRowIndexColumnConverter in modo che non estenda LongColumnConverter.
Tech stack
scala
Dominio
backendapi
Tipo issue
Refactoring
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
ScalaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60