apache / apache/iceberg-cpp

FileIO parameter should be arrow::fs::FileSystem ?

Ouverte
#307 14 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
C++
Étoiles
221
Forks
124
Merge moyen
1 j 16 h
PR mergées (30 j)
21

Description

The ManifestWriter/ManifestListWriter API uses the` std::shared_ptr` parameter for the IO interface,
which actually should be an internal `arrow::ArrowFileSystemFileIO` (not publicly accessible).
It appears that only the `fs()` is actually utilized.
```c++
auto io = internal::checked_pointer_cast(options.io);
ICEBERG_ARROW_ASSIGN_OR_RETURN(auto output, io->fs()->OpenOutputStream(options.path));
```

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par les API de ManifestWriter et ManifestListWriter, puis suivez options.io à travers le checked_pointer_cast montré et l’appel à fs()->OpenOutputStream. Déterminez le paramètre approprié orienté système de fichiers et mettez à jour les utilisations d’API associées afin que la création du flux de sortie reste valide ; le travail est terminé lorsque les tests C++ concernés ou le build réussissent.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
backend-api-design
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
42/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.