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

fix: Exception handled for multiple entries found for task invalidati… #6444

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

prabinoid
Copy link
Contributor

…on history while validating a task

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation
  • 🧑‍💻 Refactor
  • ✅ Test
  • 🤖 Build or CI
  • ❓ Other (please specify)

Related Issue

Handles #6320

Describe this PR

Handles #6320 issue of not being able to validate a task because of the multiple open entries being created task invalidation history table by introducing exception handling for multiple open entries found closing the duplicate entries and recursively calling itself until one open entry is left.

Screenshots

Previously:
Screenshot 2024-05-27 at 11 32 29
Handled Exception:
Screenshot 2024-05-27 at 11 32 54

Review Guide

Need to create two or more open entries in the task invalidation history table (Entry in this table means that the task was previously invalidated)and try to validate the task.

Copy link

sonarcloud bot commented May 27, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ramyaragupathy
Copy link
Member

Per chat w/ @manjitapandey @prabinoid this has been tested with a Db copy close to production and the fix works as expected.

cc @kaditya97

@ramyaragupathy ramyaragupathy merged commit df85442 into develop Jun 3, 2024
8 of 9 checks passed
@prabinoid prabinoid deleted the fix-validate-task branch December 3, 2024 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants