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

[Failed-Request-Alert-Tuning] Enhanced retry mechanism for MN contract results to poll until records are fully mature #3366

Closed
Tracked by #3344
quiet-node opened this issue Jan 9, 2025 · 0 comments · Fixed by #3368
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@quiet-node
Copy link
Member

quiet-node commented Jan 9, 2025

Description

Currently, when MN returns immature records (with multiple null or empty fields), the getContractResultWithRetry() and getContractResultsLogsWithRetry() function only waits 500ms before resubmitting the request once again. While this retry mechanism successfully retrieves complete records in some cases, it is not consistently effective. Therefore, implementing a polling mechanism is recommended.

Solution

Replace the current retry mechanism with a polling mechanism that continues polling until the records are fully mature. Introduce a maximum retry limit to safeguard the process, and return a 500 error if the records remain immature after exhausting the polling attempts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
1 participant