memory.py -> in_memory.py

Author: kevinjqliu

pyiceberg/catalog/__init__.py

@@ -145,7 +145,7 @@ def load_sql(name: str, conf: Properties) -> Catalog:
 
 
 def load_memory(name: str, conf: Properties) -> Catalog:
-    from pyiceberg.catalog.memory import InMemoryCatalog
+    from pyiceberg.catalog.in_memory import InMemoryCatalog
 
     return InMemoryCatalog(name, **conf)
 

pyiceberg/catalog/memory.py pyiceberg/catalog/in_memory.pypyiceberg/catalog/memory.py renamed to pyiceberg/catalog/in_memory.py

@@ -27,7 +27,7 @@
     Catalog,
     PropertiesUpdateSummary,
 )
-from pyiceberg.catalog.memory import InMemoryCatalog
+from pyiceberg.catalog.in_memory import InMemoryCatalog
 from pyiceberg.exceptions import (
     NamespaceAlreadyExistsError,
     NamespaceNotEmptyError,
@@ -52,7 +52,7 @@
 
 @pytest.fixture
 def catalog(tmp_path: PosixPath) -> InMemoryCatalog:
-    return InMemoryCatalog("test.in.memory.catalog", **{WAREHOUSE: tmp_path.absolute().as_posix(), "test.key": "test.value"})
+    return InMemoryCatalog("test.in_memory.catalog", **{WAREHOUSE: tmp_path.absolute().as_posix(), "test.key": "test.value"})
 
 
 TEST_TABLE_IDENTIFIER = ("com", "organization", "department", "my_table")
@@ -486,4 +486,4 @@ def test_add_column_with_statement(catalog: InMemoryCatalog) -> None:
 
 def test_catalog_repr(catalog: InMemoryCatalog) -> None:
     s = repr(catalog)
-    assert s == "test.in.memory.catalog (<class 'pyiceberg.catalog.memory.InMemoryCatalog'>)"
+    assert s == "test.in_memory.catalog (<class 'pyiceberg.catalog.in_memory.InMemoryCatalog'>)"

tests/catalog/test_base.py

@@ -23,7 +23,7 @@
 from click.testing import CliRunner
 from pytest_mock import MockFixture
 
-from pyiceberg.catalog.memory import DEFAULT_WAREHOUSE_LOCATION
+from pyiceberg.catalog.in_memory import DEFAULT_WAREHOUSE_LOCATION
 from pyiceberg.cli.console import run
 from pyiceberg.partitioning import PartitionField, PartitionSpec
 from pyiceberg.schema import Schema
@@ -53,7 +53,7 @@ def env_vars(mocker: MockFixture) -> None:
 
 @pytest.fixture(name="catalog")
 def fixture_catalog(mocker: MockFixture) -> InMemoryCatalog:
-    in_memory_catalog = InMemoryCatalog("test.in.memory.catalog", **{"test.key": "test.value"})
+    in_memory_catalog = InMemoryCatalog("test.in_memory.catalog", **{"test.key": "test.value"})
     mocker.patch("pyiceberg.cli.console.load_catalog", return_value=in_memory_catalog)
     return in_memory_catalog