refactor: Remove support of manifest list format as a list of file paths.

[liurenjie1024]

          Do you think it's a good idea to check this first so we don't need to write in enum pattern matching?

Originally posted by @Xuanwo in #129 (comment)

As we don't plan to support the manifest list as manifest list file, we will remove support for it.

[hiirrxnn]

Could I work on this?

[liurenjie1024]

Could I work on this?

Sure, thanks for contributing! @hiirrxnn

[hiirrxnn]

Which file/files should I make this change in ?

[liurenjie1024]

See this:

iceberg-rust/crates/iceberg/src/spec/snapshot.rs

Line 98 in c91aeae

pub enum ManifestListLocation {

You can remove the enum, but it may requires modifying some places to get it compiled.

[Dysprosium0626]

Dysprosium0626@e25adbb

Hi @liurenjie1024, here is my version of refactor. I just remove the enum and all the ManifestListLocation::ManifestFiles and change ManifestListLocation::ManifestListFile to String.

[liurenjie1024]

Hi, @Dysprosium0626 That looks great to me! Welcome to contribute.

[Dysprosium0626]

Hi, you may close this issue now!
(BTW It looks like some of the issues like #192 could be close too)