delta-io/delta

[Kernel] Refactor ParquetBatchReader converter APIs

Open

#1 859 ouverte le 23 juin 2023

Voir sur GitHub
 (2 commentaires) (0 réactions) (1 assigné)Scala (2 100 forks)batch import
enhancementgood first issuekernel

Métriques du dépôt

Stars
 (8 807 stars)
Métriques de merge PR
 (Merge moyen 7j 1h) (142 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Suivez la discussion du PR lié et implémentez le refactoring spécifié : renommer moveToNextRow() en moveToNextRow(long index) dans toute la base de code et modifier FileRowIndexColumnConverter pour qu'il n'étende plus LongColumnConverter.
Stack technique
scala
Domaine
backendapi
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ScalaGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60