Skip to content

Commit d6139e3

Browse files
committed
Remove unused arrow_client package
also format code
1 parent ed6e295 commit d6139e3

File tree

5 files changed

+12
-42
lines changed

5 files changed

+12
-42
lines changed

graphdatascience/__init__.py

-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from .arrow_client.job_id import JobId
21
from .graph.graph_create_result import GraphCreateResult
32
from .graph.graph_object import Graph
43
from .graph_data_science import GraphDataScience
@@ -34,5 +33,4 @@
3433
"NRModel",
3534
"GraphSageModel",
3635
"SimpleRelEmbeddingModel",
37-
"JobId"
3836
]

graphdatascience/arrow_client/job_id.py

-12
This file was deleted.

graphdatascience/arrow_client/job_status.py

-18
This file was deleted.

graphdatascience/query_runner/gds_arrow_client.py

+5-7
Original file line numberDiff line numberDiff line change
@@ -700,13 +700,11 @@ def _versioned_action_type(self, action_type: str) -> str:
700700
return self._arrow_endpoint_version.prefix() + action_type
701701

702702
def _versioned_flight_descriptor(self, flight_descriptor: dict[str, Any]) -> dict[str, Any]:
703-
return (
704-
{
705-
"name": "PUT_COMMAND",
706-
"version": ArrowEndpointVersion.V1.version(),
707-
"body": flight_descriptor,
708-
}
709-
)
703+
return {
704+
"name": "PUT_COMMAND",
705+
"version": ArrowEndpointVersion.V1.version(),
706+
"body": flight_descriptor,
707+
}
710708

711709
@staticmethod
712710
def _sanitize_arrow_table(arrow_table: Table) -> Table:

graphdatascience/tests/unit/test_arrow_runner.py

+7-3
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,15 @@
55
from graphdatascience.query_runner.arrow_query_runner import ArrowQueryRunner
66
from graphdatascience.server_version.server_version import ServerVersion
77

8-
from .conftest import CollectingQueryRunner
98
from ...query_runner.arrow_endpoint_version import ArrowEndpointVersion
9+
from .conftest import CollectingQueryRunner
1010

1111

1212
@pytest.mark.parametrize("server_version", [ServerVersion(2, 6, 0)])
1313
def test_create(runner: CollectingQueryRunner) -> None:
14-
arrow_info = ArrowInfo(listenAddress="localhost:1234", enabled=True, running=True, versions=[ArrowEndpointVersion.V1.version()])
14+
arrow_info = ArrowInfo(
15+
listenAddress="localhost:1234", enabled=True, running=True, versions=[ArrowEndpointVersion.V1.version()]
16+
)
1517
arrow_runner = ArrowQueryRunner.create(runner, arrow_info)
1618

1719
assert isinstance(arrow_runner, ArrowQueryRunner)
@@ -30,7 +32,9 @@ def test_return_fallback_when_arrow_is_not_enabled(runner: CollectingQueryRunner
3032

3133
@pytest.mark.parametrize("server_version", [ServerVersion(2, 6, 0)])
3234
def test_create_with_provided_connection(runner: CollectingQueryRunner) -> None:
33-
arrow_info = ArrowInfo(listenAddress="localhost:1234", enabled=True, running=True, versions=[ArrowEndpointVersion.V1.version()])
35+
arrow_info = ArrowInfo(
36+
listenAddress="localhost:1234", enabled=True, running=True, versions=[ArrowEndpointVersion.V1.version()]
37+
)
3438

3539
arrow_runner = ArrowQueryRunner.create(runner, arrow_info, connection_string_override="localhost:4321")
3640

0 commit comments

Comments
 (0)