apache / apache/iceberg-cpp

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

Aperta
#307 14 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
C++
Stelle
221
Fork
124
Merge medio
1g 16h
PR unite (30g)
21

Descrizione

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));
```

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Parti dalle API di ManifestWriter e ManifestListWriter, quindi segui options.io attraverso il checked_pointer_cast mostrato e la chiamata a fs()->OpenOutputStream. Determina il parametro appropriato orientato al filesystem e aggiorna gli utilizzi API correlati in modo che la creazione dello stream di output rimanga valida; il lavoro è completato quando i test C++ interessati o la build hanno esito positivo.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
backend-api-design
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
42/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.