FileIO parameter should be arrow::fs::FileSystem ?
- 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