Skip to content

Commit b425049

Browse files
chore(chart): remove console.log traces from property-MV builder
The three console.log calls in getChartSqlFromPropertyMV matched the existing getChartSqlFromMaterializedView pattern but bloat production logs on every filter/breakdown chart request. CodeRabbit flagged this as a potential PII leak (filter values printed verbatim). Removing here rather than leaving inconsistent with the other builders — those should follow up separately.
1 parent 5bc2e1f commit b425049

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

packages/db/src/services/chart.service.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -619,10 +619,6 @@ function getChartSqlFromPropertyMV({
619619
ORDER BY ${dateGroupBy} ASC
620620
${fillClause}`;
621621

622-
console.log('-- Using profile_event_property_summary_mv --');
623-
console.log(sql.replaceAll(/[\n\r]/g, ' '));
624-
console.log('-- End --');
625-
626622
return sql;
627623
}
628624

0 commit comments

Comments
 (0)