Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes proposed ✍️
What
🤖 Generated by Copilot at 5359728
This pull request improves the database queries and data processing for segments and activity channels in the
SegmentRepository
. It also adds theapplication_name
option to the database connection configuration in both themodels
and theconnection
modules, using theSERVICE
environment variable to identify the source of the queries.
🤖 Generated by Copilot at 5359728
Why
How
🤖 Generated by Copilot at 5359728
application_name
option to database configuration and connection to identify query source (link, link)segmentActivityChannels
table to improve performance and readability (link, link)jsonb_merge_agg
function to aggregate activity channels by segment and platform in queries ofSegmentRepository
(link, link, link)segmentId
column andDISTINCT
modifier to join and aggregate activity channels correctly in queries ofSegmentRepository
(link, link, link)activityChannels
property before using spread operator inSegmentRepository
(link, link)activityChannels
property inSegmentRepository
(link)Checklist ✅
Feature
,Improvement
, orBug
.