rapidsai/cudf

[BUG] Loading a missing column from a Parquet file results in ArrayIndexOutOfBoundsException

Open

#11 278 ouverte le 15 juil. 2022

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C++ (735 forks)batch import
JavaPythonSparkbuggood first issue

Métriques du dépôt

Stars
 (6 000 stars)
Métriques de merge PR
 (Merge moyen 17j 21h) (230 PRs mergées en 30 j)

Description

Describe the bug Attempting to load a non-existent column from a Parquet file throws ArrayIndexOutOfBoundsException instead of a more specific error.

Steps/Code to reproduce bug Try to load a single column name that does not exist from a Parquet file. For example:

Table.readParquet(ParquetOptions.builder().includeColumn("doesnotexist").build(), new java.io.File("data.parquet"))

throws the following exception:

java.lang.ArrayIndexOutOfBoundsException: 0
  at ai.rapids.cudf.Table.<init>(Table.java:96)
  at ai.rapids.cudf.Table.readParquet(Table.java:974)
  ... 49 elided

Expected behavior A more useful error exception/message should be used than an array index error.

Guide contributeur

Direction de recherche
Inspectez le chemin du code de lecture Parquet dans les liaisons Java de cudf pour localiser l'endroit où l'index de colonne est accedé sans vérification de null ou hors limites, puis ajoutez une validation pour lancer une erreur descriptive.
Stack technique
java
Domaine
databackend
Type d'issue
Bug
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
JavacuDF library
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50