Skip to content
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

[HOTFIX][BE] Fix race condition on scheduler when CRUD #269

Merged
merged 5 commits into from
Nov 6, 2024

Conversation

Nicklas373
Copy link
Owner

Current condition, schedule hana:clean-session and hana:clear-storage have time gap
15 minutes and 1 hours. In this condition, it could possibility that 2 scheduler run in the same
time. It can make same UUID that has been generated in one time to be used for 2 scheduler.

This condition can create error failed to insert into appLogs due "processId" already been used,
Latest condition are create separate new UUID for hana:clear-storage that is different and new generated
to avoid same UUID that already used by hana:clean-session. Because hana:clean-session are actually run
first before hana:clear-storage

Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
Current condition, schedule hana:clean-session and hana:clear-storage have time gap
15 minutes and 1 hours. In this condition, it could possibility that 2 scheduler run in the same
time. It can make same UUID that has been generated in one time to be used for 2 scheduler.

This condition can create error failed to insert into appLogs due "processId" already been used,
Latest condition are create separate new UUID for hana:clear-storage that is different and new generated
to avoid same UUID that already used by hana:clean-session. Because hana:clean-session are actually run
first before hana:clear-storage

Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
Signed-off-by: Dicky Herlambang (花) <herlambangdicky5@gmail.com>
@Nicklas373 Nicklas373 added bug Something isn't working php Pull requests that update Php code labels Nov 6, 2024
@Nicklas373 Nicklas373 self-assigned this Nov 6, 2024
@Nicklas373 Nicklas373 merged commit 87c8c7a into be/master Nov 6, 2024
2 checks passed
@Nicklas373 Nicklas373 deleted the dev/be/master branch November 6, 2024 00:45
@Nicklas373 Nicklas373 restored the dev/be/master branch November 6, 2024 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant