delta-io/delta

[Kernel] Refactor ParquetBatchReader converter APIs

Open

#1859 aperta il 23 giu 2023

Vedi su GitHub
 (2 commenti) (0 reazioni) (1 assegnatario)Scala (2100 fork)batch import
enhancementgood first issuekernel

Metriche repository

Star
 (8807 star)
Metriche merge PR
 (Merge medio 7g 1h) (142 PR mergiate in 30 g)

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