Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Metadata integration tests #57

Merged
merged 13 commits into from
Sep 15, 2023
Merged

Metadata integration tests #57

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
877fcb6
add test for valid metadata v2
JanKaul Aug 29, 2023
cd6e9d0
add test for minimal valid table metadata v2
JanKaul Aug 29, 2023
93ec8b3
test valied v1 table metadata
JanKaul Aug 29, 2023
b563e27
Test for invalid schema id
JanKaul Aug 29, 2023
08f11ed
improve failing test
JanKaul Sep 1, 2023
078e0dd
missing sort oder test
JanKaul Sep 8, 2023
6d86746
test for missing partition spec
JanKaul Sep 8, 2023
b628d0c
test missing partition id
JanKaul Sep 8, 2023
45c892e
test missing schemas
JanKaul Sep 8, 2023
2321b64
test unsupported version
JanKaul Sep 8, 2023
a33934e
fix changes
JanKaul Sep 8, 2023
f57c0ed
improve error message
JanKaul Sep 8, 2023
c02e882
fix clippy warnings
JanKaul Sep 8, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion crates/iceberg/src/spec/snapshot.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ pub struct Snapshot {
/// A string map that summarizes the snapshot changes, including operation.
summary: Summary,
/// ID of the table’s current schema when the snapshot was created.
#[builder(setter(strip_option))]
#[builder(setter(strip_option), default = "None")]
schema_id: Option<i64>,
}

Expand Down
Loading
Loading