fix: remove prefetching on spans table (#2555)

hypertrace · Dec 4, 2023 · cb8cc71 · cb8cc71
1 parent e743904
commit cb8cc71
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/projects/observability/src/pages/explorer/explorer.component.test.ts b/projects/observability/src/pages/explorer/explorer.component.test.ts
@@ -260,7 +260,7 @@ describe('Explorer component', () => {
       expect.objectContaining({
         requestType: SPANS_GQL_REQUEST,
         filters: [],
-        limit: 100,
+        limit: 50,
       }),
       undefined,
     );
@@ -310,7 +310,7 @@ describe('Explorer component', () => {
       2,
       expect.objectContaining({
         requestType: SPANS_GQL_REQUEST,
-        limit: 100,
+        limit: 50,
         filters: [new GraphQlFieldFilter({ key: 'first' }, GraphQlOperatorType.Equals, 'foo')],
       }),
       undefined,

diff --git a/...ervability/src/shared/dashboard/data/graphql/table/spans/spans-table-data-source.model.ts b/...ervability/src/shared/dashboard/data/graphql/table/spans/spans-table-data-source.model.ts
@@ -31,7 +31,7 @@ export class SpansTableDataSourceModel extends TableDataSourceModel {
       properties: request.columns
         .filter(column => column.visible || this.mandatoryColumns.includes(column.id))
         .map(column => column.specification),
-      limit: request.position.limit * 2, // Prefetch 2 pages
+      limit: request.position.limit,
       offset: request.position.startIndex,
       sort: request.sort && {
         direction: request.sort.direction,