Skip to content

Commit 02aa8f9

Browse files
authored
fix: reduce enrichment payload size to prevent temporal failures (CM-1065) (#4001)
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
1 parent 697b076 commit 02aa8f9

File tree

3 files changed

+25
-7
lines changed

3 files changed

+25
-7
lines changed

services/apps/members_enrichment_worker/src/activities/enrichment.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -549,9 +549,10 @@ export function doesIncomingOrgExistInExistingOrgs(
549549
.filter((i) => i.type === OrganizationIdentityType.PRIMARY_DOMAIN && i.verified)
550550
.map((i) => i.value)
551551

552-
const existingVerifiedPrimaryDomainIdentityValues = existingOrg.identities
553-
.filter((i) => i.type === OrganizationIdentityType.PRIMARY_DOMAIN && i.verified)
554-
.map((i) => i.value)
552+
// existingOrg.identities is already filtered to primary-domain + verified at the query level
553+
const existingVerifiedPrimaryDomainIdentityValues = (existingOrg.identities || []).map(
554+
(i) => i.value,
555+
)
555556

556557
const incomingOrgStartDate = incomingOrg.startDate ? new Date(incomingOrg.startDate) : null
557558
const incomingOrgEndDate = incomingOrg.endDate ? new Date(incomingOrg.endDate) : null

services/libs/data-access-layer/src/old/apps/members_enrichment_worker/index.ts

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,13 @@ export async function fetchMemberDataForLLMSquashing(
2929
mo."dateStart",
3030
mo."dateEnd",
3131
mo.source,
32-
jsonb_agg(oi) as identities
32+
jsonb_agg(jsonb_build_object(
33+
'organizationId', oi."organizationId",
34+
'platform', oi.platform,
35+
'value', oi.value,
36+
'type', oi.type,
37+
'verified', oi.verified
38+
)) filter (where oi.type = 'primary-domain' and oi.verified = true) as identities
3339
from "memberOrganizations" mo
3440
inner join organizations o on mo."organizationId" = o.id
3541
inner join "organizationIdentities" oi on oi."organizationId" = o.id
@@ -63,7 +69,7 @@ export async function fetchMemberDataForLLMSquashing(
6369
mo."dateStart",
6470
mo."dateEnd",
6571
mo.source,
66-
mo.identities) r)
72+
coalesce(mo.identities, '[]'::jsonb) as identities) r)
6773
)
6874
from member_orgs mo
6975
where mo."memberId" = m.id
@@ -174,7 +180,13 @@ export async function fetchMembersForLFIDEnrichment(db: DbStore, limit: number,
174180
members."contributions",
175181
members."score",
176182
members."reach",
177-
jsonb_agg(mi.*) as identities
183+
jsonb_agg(jsonb_build_object(
184+
'platform', mi.platform,
185+
'value', mi.value,
186+
'type', mi.type,
187+
'verified', mi.verified,
188+
'sourceId', mi."sourceId"
189+
)) as identities
178190
FROM members
179191
INNER JOIN "memberIdentities" mi ON mi."memberId" = members.id
180192
AND mi."deletedAt" is null

services/libs/data-access-layer/src/old/apps/merge_suggestions_worker/memberMergeSuggestions.repo.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,12 @@ class MemberMergeSuggestionsRepository {
241241
mem.attributes,
242242
mem."displayName",
243243
mem."joinedAt",
244-
jsonb_agg(distinct mI) as identities,
244+
jsonb_agg(distinct jsonb_build_object(
245+
'platform', mI.platform,
246+
'value', mI.value,
247+
'type', mI.type,
248+
'verified', mI.verified
249+
)) as identities,
245250
coalesce(
246251
(
247252
select jsonb_agg(

0 commit comments

Comments
 (0)