Skip to content

Commit d99dfdd

Browse files
committed
chore: lint and update names
1 parent 2ae47e3 commit d99dfdd

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

pyiceberg/table/update/sorting.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,9 @@ def _add_sort_field(
7676
)
7777
return self
7878

79-
def asc(self, source_column_name: str, transform: Transform[Any, Any], null_order: NullOrder = NullOrder.NULLS_LAST) -> UpdateSortOrder:
79+
def asc(
80+
self, source_column_name: str, transform: Transform[Any, Any], null_order: NullOrder = NullOrder.NULLS_LAST
81+
) -> UpdateSortOrder:
8082
"""Add a sort field with ascending order."""
8183
return self._add_sort_field(
8284
source_id=self._column_name_to_id(source_column_name),

tests/integration/test_sort_order_update.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def test_map_column_name_to_id(catalog: Catalog, format_version: str, table_sche
6767
(pytest.lazy_fixture("session_catalog_hive"), "2"),
6868
],
6969
)
70-
def test_replace_sort_order(catalog: Catalog, format_version: str, table_schema_simple: Schema) -> None:
70+
def test_update_sort_order(catalog: Catalog, format_version: str, table_schema_simple: Schema) -> None:
7171
simple_table = _simple_table(catalog, table_schema_simple, format_version)
7272
simple_table.update_sort_order().asc("foo", IdentityTransform(), NullOrder.NULLS_FIRST).desc(
7373
"bar", IdentityTransform(), NullOrder.NULLS_LAST
@@ -89,7 +89,7 @@ def test_replace_sort_order(catalog: Catalog, format_version: str, table_schema_
8989
(pytest.lazy_fixture("session_catalog_hive"), "2"),
9090
],
9191
)
92-
def test_replace_existing_sort_order(catalog: Catalog, format_version: str, table_schema_simple: Schema) -> None:
92+
def test_update_existing_sort_order(catalog: Catalog, format_version: str, table_schema_simple: Schema) -> None:
9393
simple_table = _simple_table(catalog, table_schema_simple, format_version)
9494
simple_table.update_sort_order().asc("foo", IdentityTransform(), NullOrder.NULLS_FIRST).commit()
9595
assert simple_table.sort_order() == SortOrder(

0 commit comments

Comments
 (0)