apache / apache/iceberg-rust

Refactor file io tests to support different file io storage

Open
#484 3 comments 0 reactions 0 assignees View on GitHub
rust
Dominant language
Rust
Stars
1.4k
Forks
567
Avg merge
2d 2h
Merged PRs (30d)
93

Description

> * Could we add some tests to verify it? I think it should be easy since we have it as default feature.

Current test is not extsiable for different storage service, please allow me to complement them in a new PR. For example, I can refactor `test_local_input_file`, `test_delete_local_file`, ... to make them work on different file io instead.

> Could we have a simple doc to explain how to use it? At least for me, it's not straight forward without refering to opendal's doc.

Sure, I will add this.

_Originally posted by @Xuanwo in https://github.com/apache/iceberg-rust/pull/481#discussion_r1693014389_

Contributor guide

Open the contributing guide

Research direction

Start by locating the existing test_local_input_file and test_delete_local_file tests and review how the current default file I/O storage is configured. Refactor the relevant tests so they work with different storage services, then add a simple usage document explaining the configuration without requiring readers to consult OpenDAL's documentation.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend, testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.