Skip to content

Commit 6b6bcab

Browse files
authored
Merge pull request #5959 from secfree/250917-distinct-profile-meta
feat: use Distinct instead of Project for profile metadata queries
2 parents b8bd784 + 5c4979c commit 6b6bcab

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

pkg/parcacol/querier.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1566,7 +1566,7 @@ func (q *Querier) GetProfileMetadataMappings(
15661566
records := make(map[string]struct{})
15671567
err = q.engine.ScanTable(q.tableName).
15681568
Filter(filterExpr).
1569-
Project(logicalplan.Col("stacktrace")).
1569+
Distinct(logicalplan.Col("stacktrace")).
15701570
Execute(ctx, func(ctx context.Context, r arrow.Record) error {
15711571
r.Retain()
15721572

@@ -1631,7 +1631,7 @@ func (q *Querier) GetProfileMetadataLabels(
16311631

16321632
err = q.engine.ScanTable(q.tableName).
16331633
Filter(filterExpr).
1634-
Project(logicalplan.DynCol("labels")).
1634+
Distinct(logicalplan.DynCol("labels")).
16351635
Execute(ctx, func(ctx context.Context, ar arrow.Record) error {
16361636
for i, field := range ar.Schema().Fields() {
16371637
nulls := ar.Column(i).NullN()

0 commit comments

Comments
 (0)