Skip to content

Commit

Permalink
Change op to make clear that we wrap data fetching methods
Browse files Browse the repository at this point in the history
  • Loading branch information
lforst committed Aug 19, 2022
1 parent 150843f commit 6ed57f6
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions packages/nextjs/src/config/wrappers/wrapperUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ export function callTracedServerSideDataFetcher<F extends (...args: any[]) => Pr
// TODO: Extract trace data from `req` object (trace and baggage headers) and attach it to transaction

const newTransaction = startTransaction({
op: 'nextjs.data',
op: 'nextjs.data.server',
name: options.requestedRouteName,
metadata: {
source: 'route',
Expand All @@ -98,7 +98,7 @@ export function callTracedServerSideDataFetcher<F extends (...args: any[]) => Pr
}

const dataFetcherSpan = requestTransaction.startChild({
op: 'nextjs.data',
op: 'nextjs.data.server',
description: `${options.dataFetchingMethodName} (${options.dataFetcherRouteName})`,
});

Expand Down Expand Up @@ -159,7 +159,7 @@ export async function callDataFetcherTraced<F extends (...args: any[]) => Promis
// Capture the route, since pre-loading, revalidation, etc might mean that this span may happen during another
// route's transaction
const span = transaction.startChild({
op: 'nextjs.data',
op: 'nextjs.data.server',
description: `${dataFetchingMethodName} (${parameterizedRoute})`,
});

Expand Down

0 comments on commit 6ed57f6

Please sign in to comment.