apache / apache/iceberg-rust

Add integration tests for transaction APIs

Open
#1,322 2 comments 2 reactions 0 assignees View on GitHub
enhancement not-stale
Dominant language
Rust
Stars
1.4k
Forks
567
Avg merge
2d 2h
Merged PRs (30d)
93

Description

### Is your feature request related to a problem or challenge?

Now many APIs in `transaction/mod.rs` don't have an integration tests to test if they actually work after committing the transaction. We can probably create a new test to test these APIs using a memory catalog

### Describe the solution you'd like

_No response_

### Willingness to contribute

I would be willing to contribute to this feature with guidance from the Iceberg Rust community

Contributor guide

Open the contributing guide

Research direction

Start in transaction/mod.rs and identify the transaction APIs that lack integration coverage. Use a memory catalog to exercise those APIs after committing the transaction; done means the relevant APIs have integration tests demonstrating that their committed behavior works.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend-api-design, testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.