refactor: always use query limits (#1496)

* refactor: always use query limits

* test: update test limits

Author: aaron-steinfeld

projects/observability/src/shared/dashboard/widgets/trace-detail/data/api-trace-detail-data-source.model.test.ts

@@ -79,7 +79,7 @@ describe('API Trace detail data source model', () => {
       expect.objectContaining({
         requestType: TRACE_GQL_REQUEST,
         traceId: 'test',
-        spanLimit: 0,
+        spanLimit: 1000,
         timestamp: undefined,
         traceProperties: expect.arrayContaining([
           expect.objectContaining({ name: 'tags' }),
@@ -99,7 +99,7 @@ describe('API Trace detail data source model', () => {
       expect.objectContaining({
         requestType: TRACE_GQL_REQUEST,
         traceId: 'test',
-        spanLimit: 0,
+        spanLimit: 1000,
         timestamp: undefined,
         traceProperties: expect.arrayContaining([
           expect.objectContaining({ name: 'tags' }),
@@ -119,7 +119,7 @@ describe('API Trace detail data source model', () => {
       expect.objectContaining({
         requestType: TRACE_GQL_REQUEST,
         traceId: 'test',
-        spanLimit: 0,
+        spanLimit: 1000,
         timestamp: new Date(1568907645141),
         traceProperties: expect.arrayContaining([
           expect.objectContaining({ name: 'tags' }),

projects/observability/src/shared/dashboard/widgets/trace-detail/data/trace-detail-data-source.model.test.ts

@@ -74,7 +74,7 @@ describe('API Trace detail data source model', () => {
       expect.objectContaining({
         requestType: TRACE_GQL_REQUEST,
         traceId: 'test',
-        spanLimit: 0,
+        spanLimit: 1000,
         timestamp: undefined,
         traceProperties: expect.arrayContaining([
           expect.objectContaining({ name: 'tags' }),
@@ -94,7 +94,7 @@ describe('API Trace detail data source model', () => {
       expect.objectContaining({
         requestType: TRACE_GQL_REQUEST,
         traceId: 'test',
-        spanLimit: 0,
+        spanLimit: 1000,
         timestamp: new Date(1576364117792),
         traceProperties: expect.arrayContaining([
           expect.objectContaining({ name: 'tags' }),

projects/observability/src/shared/dashboard/widgets/trace-detail/data/trace-detail-data-source.model.ts

@@ -42,7 +42,7 @@ export class TraceDetailDataSourceModel extends GraphQlDataSourceModel<TraceDeta
       requestType: TRACE_GQL_REQUEST,
       traceType: this.trace[traceTypeKey],
       traceId: this.trace[traceIdKey],
-      spanLimit: 0,
+      spanLimit: 1000,
       timestamp: this.dateCoercer.coerce(this.startTime ?? this.trace.startTime),
       traceProperties: this.getTraceAttributes().map(attribute =>
         this.attributeSpecBuilder.attributeSpecificationForKey(attribute)