Skip to content

Commit 40da9e6

Browse files
[APM] Increase xpack.apm.ui.transactionGroupBucketSize (#71661) (#72431)
Co-authored-by: Søren Louv-Jansen <sorenlouv@gmail.com>
1 parent ad99cbd commit 40da9e6

File tree

7 files changed

+9
-7
lines changed

7 files changed

+9
-7
lines changed

docs/settings/apm-settings.asciidoc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ Changing these settings may disable features of the APM App.
5252
| Set to `false` to hide the APM app from the menu. Defaults to `true`.
5353

5454
| `xpack.apm.ui.transactionGroupBucketSize`
55-
| Number of top transaction groups displayed in the APM app. Defaults to `100`.
55+
| Number of top transaction groups displayed in the APM app. Defaults to `1000`.
5656

5757
| `xpack.apm.ui.maxTraceItems` {ess-icon}
5858
| Maximum number of child items displayed when viewing trace details. Defaults to `1000`.

x-pack/plugins/apm/server/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export const config = {
2727
autocreateApmIndexPattern: schema.boolean({ defaultValue: true }),
2828
ui: schema.object({
2929
enabled: schema.boolean({ defaultValue: true }),
30-
transactionGroupBucketSize: schema.number({ defaultValue: 100 }),
30+
transactionGroupBucketSize: schema.number({ defaultValue: 1000 }),
3131
maxTraceItems: schema.number({ defaultValue: 1000 }),
3232
}),
3333
telemetryCollectionEnabled: schema.boolean({ defaultValue: true }),

x-pack/plugins/apm/server/lib/transaction_groups/__snapshots__/fetcher.test.ts.snap

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/plugins/apm/server/lib/transaction_groups/__snapshots__/queries.test.ts.snap

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/plugins/apm/server/lib/transaction_groups/fetcher.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,9 @@ export async function transactionGroupsFetcher(
7272
aggs: {
7373
transaction_groups: {
7474
composite: {
75-
size: bucketSize + 1, // 1 extra bucket is added to check whether the total number of buckets exceed the specified bucket size.
75+
// traces overview is hardcoded to 10000
76+
// transactions overview: 1 extra bucket is added to check whether the total number of buckets exceed the specified bucket size.
77+
size: isTopTraces ? 10000 : bucketSize + 1,
7678
sources: [
7779
...(isTopTraces
7880
? [{ service: { terms: { field: SERVICE_NAME } } }]

x-pack/test/apm_api_integration/basic/tests/traces/top_traces.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ export default function ApiTest({ getService }: FtrProviderContext) {
2424
);
2525

2626
expect(response.status).to.be(200);
27-
expect(response.body).to.eql({ items: [], isAggregationAccurate: true, bucketSize: 100 });
27+
expect(response.body).to.eql({ items: [], isAggregationAccurate: true, bucketSize: 1000 });
2828
});
2929
});
3030

x-pack/test/apm_api_integration/basic/tests/transaction_groups/top_transaction_groups.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ export default function ApiTest({ getService }: FtrProviderContext) {
2525
);
2626

2727
expect(response.status).to.be(200);
28-
expect(response.body).to.eql({ items: [], isAggregationAccurate: true, bucketSize: 100 });
28+
expect(response.body).to.eql({ items: [], isAggregationAccurate: true, bucketSize: 1000 });
2929
});
3030
});
3131

0 commit comments

Comments
 (0)