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

report error when results exceed 100k nodes #2108

Merged
merged 2 commits into from
Apr 30, 2024
Merged

report error when results exceed 100k nodes #2108

merged 2 commits into from
Apr 30, 2024

Conversation

gnmahanth
Copy link
Contributor

@gnmahanth gnmahanth commented Apr 22, 2024

Changes proposed in this pull request:

  • report error when results exceed 100k nodes while generating reports
  • add status_message filed in reports data
  • change reports struck in inprogress to error state if in inprogress for greater than 30mins
  • remove html generation when generating pdf reports

@gnmahanth gnmahanth force-pushed the reports-limits branch 5 times, most recently from 889faa7 to 7412021 Compare April 24, 2024 13:56
@gnmahanth gnmahanth requested a review from noboruma April 24, 2024 14:06
@gnmahanth gnmahanth marked this pull request as ready for review April 24, 2024 14:06
@gnmahanth gnmahanth force-pushed the reports-limits branch 5 times, most recently from ae24b6a to 0dc4ef7 Compare April 25, 2024 08:11
@gnmahanth gnmahanth merged commit 960b717 into main Apr 30, 2024
3 of 4 checks passed
@gnmahanth gnmahanth deleted the reports-limits branch April 30, 2024 14:22
@gnmahanth gnmahanth restored the reports-limits branch April 30, 2024 14:29
gnmahanth added a commit that referenced this pull request Apr 30, 2024
gnmahanth added a commit that referenced this pull request Apr 30, 2024
Revert "report error when results exceed 100k nodes (#2108)"

This reverts commit 960b717.
@gnmahanth gnmahanth mentioned this pull request Apr 30, 2024
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.

3 participants