Skip to content

Commit

Permalink
chore: remove redundant ObjectStore
Browse files Browse the repository at this point in the history
  • Loading branch information
mesejo committed Sep 27, 2024
1 parent acab244 commit d3953b0
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 14 deletions.
4 changes: 0 additions & 4 deletions python/datafusion/object_store.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,3 @@
Http = object_store.Http

__all__ = ["AmazonS3", "GoogleCloud", "LocalFileSystem", "MicrosoftAzure", "Http"]


def __getattr__(name):
return getattr(object_store, name)
3 changes: 1 addition & 2 deletions python/datafusion/tests/test_sql.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
from pyarrow.csv import write_csv
import pyarrow.dataset as ds
import pytest
from datafusion.object_store import LocalFileSystem, Http
from datafusion.object_store import Http

from datafusion import udf, col

Expand Down Expand Up @@ -462,7 +462,6 @@ def test_register_listing_table(

dir_root = f"file://{dir_root}/" if path_to_str else dir_root

ctx.register_object_store("file://local", LocalFileSystem())
ctx.register_listing_table(
"my_table",
dir_root,
Expand Down
10 changes: 2 additions & 8 deletions python/datafusion/tests/test_store.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,21 +16,15 @@
# under the License.

import os

import pytest

from datafusion import SessionContext
from datafusion.object_store import LocalFileSystem


@pytest.fixture
def local():
return LocalFileSystem()


@pytest.fixture
def ctx(local):
def ctx():
ctx = SessionContext()
ctx.register_object_store("file://local", local)
return ctx


Expand Down

0 comments on commit d3953b0

Please sign in to comment.