Change reading spans from GET to POST

Arize-ai · Apr 12, 2024 · 5d8bd30 · 5d8bd30
1 parent 427d130
commit 5d8bd30
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/phoenix/server/api/routers/v1/__init__.py b/src/phoenix/server/api/routers/v1/__init__.py
@@ -6,5 +6,5 @@
     Route("/v1/evaluations", evaluations.post_evaluation, methods=["POST"]),
     Route("/v1/evaluations", evaluations.get_evaluations, methods=["GET"]),
     Route("/v1/traces", traces.post_traces, methods=["POST"]),
-    Route("/v1/spans", spans.get_spans, methods=["GET"]),
+    Route("/v1/spans", spans.read_spans, methods=["POST"]),
 ]
diff --git a/src/phoenix/server/api/routers/v1/spans.py b/src/phoenix/server/api/routers/v1/spans.py
@@ -14,10 +14,10 @@
 
 
 # TODO: Add property details to SpanQuery schema
-async def get_spans(request: Request) -> Response:
+async def read_spans(request: Request) -> Response:
     """
     summary: Get gets from Phoenix
-    operationId: getSpans
+    operationId: readSpans
     tags:
       - spans
     parameters:

diff --git a/src/phoenix/session/client.py b/src/phoenix/session/client.py
@@ -88,7 +88,7 @@ def query_spans(
                 root_spans_only=root_spans_only,
                 project_name=project_name,
             )
-        response = self._session.get(
+        response = self._session.post(
             url=urljoin(self._base_url, "/v1/spans"),
             params={"project_name": project_name},
             json={