Skip to content

Commit

Permalink
fix: invert filter additional actions in create command
Browse files Browse the repository at this point in the history
  • Loading branch information
roeap authored and rtyler committed May 2, 2023
1 parent 4c7c226 commit edf1355
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions python/tests/test_writer.py
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ def test_roundtrip_metadata(tmp_path: pathlib.Path, sample_data: pa.Table):
sample_data,
name="test_name",
description="test_desc",
configuration={"configTest": "foobar"},
configuration={"delta.appendOnly": "false"},
)

delta_table = DeltaTable(tmp_path)
Expand All @@ -153,7 +153,7 @@ def test_roundtrip_metadata(tmp_path: pathlib.Path, sample_data: pa.Table):

assert metadata.name == "test_name"
assert metadata.description == "test_desc"
assert metadata.configuration == {"configTest": "foobar"}
assert metadata.configuration == {"delta.appendOnly": "false"}


@pytest.mark.parametrize(
Expand Down
2 changes: 1 addition & 1 deletion rust/src/operations/create.rs
Original file line number Diff line number Diff line change
Expand Up @@ -272,7 +272,7 @@ impl CreateBuilder {
actions.extend(
self.actions
.into_iter()
.filter(|a| matches!(a, Action::protocol(_))),
.filter(|a| !matches!(a, Action::protocol(_))),
);

Ok((table, actions, operation))
Expand Down

0 comments on commit edf1355

Please sign in to comment.