AlenkaF commented on code in PR #45998: URL: https://github.com/apache/arrow/pull/45998#discussion_r2073379635
########## cpp/src/arrow/filesystem/hdfs_io.h: ########## @@ -38,18 +40,6 @@ namespace io { class HdfsReadableFile; class HdfsOutputStream; -/// DEPRECATED. Use the FileSystem API in arrow::fs instead. -struct ObjectType { - enum type { FILE, DIRECTORY }; -}; - -/// DEPRECATED. Use the FileSystem API in arrow::fs instead. -struct ARROW_EXPORT FileStatistics { - /// Size of file, -1 if finding length is unsupported - int64_t size; - ObjectType::type kind; -}; - class ARROW_EXPORT FileSystem { Review Comment: Ok, will go with the disappearing =) IIUC `hdfs_io.h` will be removed altogether: - `FileSystem` and `HadoopFileSystem` will go into `hdfs.cc`, folded into the public `HadoopFileSystem` - `HdfsConnectionConfig` will also go into `hdfs.cc` - declarations that are left will go into `hdfs_internal.cc` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org