Skip to content

Fix: Removing custom paginated and added iterator - #1800

Closed
Dedsec0098 wants to merge 5 commits into
aboutcode-org:mainfrom
Dedsec0098:removing-custom-paginated
Closed

Fix: Removing custom paginated and added iterator#1800
Dedsec0098 wants to merge 5 commits into
aboutcode-org:mainfrom
Dedsec0098:removing-custom-paginated

Conversation

@Dedsec0098

Copy link
Copy Markdown

Fix #1798

Changes Made:

  1. I have removed paginated ( ) and have added interator ( ) to improve the performance in terms of compute time and memory usage.
  2. Removed per_page parameter as Django's built-in .iterator() doesn't accept this parameter.

@keshav-space I have made the changes according to the issue and it is passing all the tests on my local system, however I am not sure how to exactly run the iterations for advisories to check the new response time can you please suggest me on how can I run those and also If you have any suggestions please let me know I will make the changes required.

Dedsec0098 and others added 5 commits March 4, 2025 21:26
….com

Signed-off-by: Shrish0098 <shrish409@gmail.com>
Signed-off-by: Shrish0098 <shrish409@gmail.com>
Signed-off-by: Shrish0098 <shrish409@gmail.com>
Signed-off-by: Shrish0098 <shrish409@gmail.com>
@Dedsec0098 Dedsec0098 closed this Mar 8, 2025
@Dedsec0098
Dedsec0098 deleted the removing-custom-paginated branch March 8, 2025 05:55
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.

Get rid of the custom paginated() and use the built-in iterator()

1 participant