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: performance issue while submitting the Journal Entry #28425

Conversation

rohitwaghchaure
Copy link
Collaborator

While submitting the Journal Entry user were getting the Timeout Error. During profiling we came to know that the check_credit_limit method taking time, even though user has not setup the credit limit in the customer master or customer group or in the company master.

Found below query in the processlist which was taking time. System run this query before to check whether the credit limit has set in the customer master or not

Screenshot 2021-11-16 at 11 51 45 PM

After Fix

System has submitted journal entry within 20 seconds

@rohitwaghchaure
Copy link
Collaborator Author

rohitwaghchaure commented Nov 17, 2021

@Mergifyio backport version-13-hotfix @Mergifyio backport version-13-pre-release

@mergify
Copy link
Contributor

mergify bot commented Nov 17, 2021

backport version-13-hotfix. @Mergifyio backport version-13-pre-release

❌ No backport have been created

@rohitwaghchaure
Copy link
Collaborator Author

@Mergifyio backport version-13-hotfix

@mergify
Copy link
Contributor

mergify bot commented Nov 17, 2021

backport version-13-hotfix

✅ Backports have been created

@mergify
Copy link
Contributor

mergify bot commented Nov 17, 2021

backport version-13-pre-release

✅ Backports have been created

rohitwaghchaure added a commit that referenced this pull request Nov 17, 2021
…-28425

fix: performance issue while submitting the Journal Entry (backport #28425)
rohitwaghchaure added a commit that referenced this pull request Nov 17, 2021
…se/pr-28425

fix: performance issue while submitting the Journal Entry (backport #28425)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant