-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[$40]Duplicate traits are created when altering the payment method #54
Comments
@sdgun , Cannot re-produce the issue, please see this video: https://monosnap.com/direct/SUVbF5eXiktXWgi3BR0jA9cPaDg2mp For testing, I setup the following apps locally (on another machine):
Can you provide more details about how to re-produce the issue ? specifically the initial traits for the users used for testing cc @veshu |
@Schpotsky can you please try to make two changes for the same user? I mean our job should receive one record for user in one run and then two records in the next run. |
@Schpotsky Attached the initial status of the given test users before we changed the payment method The step we followed was generating the payment method change event for these users twice during a period of like 30 minutes. |
Getting two records for the same user will not happen, we only get the one with latest modify_date GET_MODIFIED_PAYMENT_METHODS_QUERY Can you provide an extract of the user_payment_method table in the dev environment for the above users ? select * from user_payment_method where user_id in (88778061,40152238); Also the corresponding data in Elasticsearch for the profile traits for these users will be helpful:
|
It is an issue related to the member-api itself, it is not related to onboarding-processor. Duplicate traits are created when calling It looks like this is the branch which is deployed on the dev environment, can you double check ? Also let me know, whether I should fix it in the above mentioned branch or this will be done by the team working on the CQRS feature. Thanks |
Yes you are right @Schpotsky, that branch is deployed into dev right now. Let me ask @sachin-maheshwari how he want to fix the issue with that branch for duplicate traits. |
Duplicate traits are created when altering the payment method more than once for a particular user within polling duration (3 days). We noticed dynamodb has only one record, but somehow ES is receiving multiple records
Example users: taasintake1000, testsdgun
Example response:
[
{
"traitId": "onboarding_checklist",
"categoryName": "Onboarding Checklist",
"traits": {
"traitId": "onboarding_checklist",
"data": [
{
"user_payment_method": {
"date": 1633497003327,
"message": "success",
"payment_method": "wipro payroll",
"status": "completed"
}
}
]
}
},
{
"traitId": "onboarding_checklist",
"categoryName": "Onboarding Checklist",
"traits": {
"traitId": "onboarding_checklist",
"data": [
{
"user_payment_method": {
"status": "completed",
"message": "success",
"date": 1633497003327,
"payment_method": "wipro payroll"
}
}
]
}
}
]
The text was updated successfully, but these errors were encountered: