Collaborator

Description

• Reset all counter in metadata with purge for RequestQueue. This will be more consistent with JS. Also, saving the total_request_count and handled_request_count counters causes confusion among users, especially when this happens with the default queue.

Issues

• Closes: Logs: incorrect number of pages to crawl #1682

Collaborator Author

@vdusek The current behavior of purge was implemented during the storage refactoring. Perhaps there was a reason for keeping the handled_request_count' and total_request_count' counters?

Pijukatel approved these changes Jan 26, 2026

View reviewed changes

Collaborator

Yeah, I initially thought total_request_count should represent the actual total number of requests this instance has processed (even if purge was executed), with only a drop resetting it. But if that's confusing or inconsistent with the JS version, then let's change it.

View reviewed changes

Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.