Skip to content

Also check if the total hash count of a hashlist needs to be fixed#2033

Open
s3inlc wants to merge 1 commit intodevfrom
global-chunk-scan-improvement
Open

Also check if the total hash count of a hashlist needs to be fixed#2033
s3inlc wants to merge 1 commit intodevfrom
global-chunk-scan-improvement

Conversation

@s3inlc
Copy link
Copy Markdown
Member

@s3inlc s3inlc commented Apr 10, 2026

So far only the cracked hashes counts where check, but not the total hash counts of the hashlist.

@s3inlc s3inlc requested a review from jessevz April 10, 2026 12:51
@jessevz
Copy link
Copy Markdown
Contributor

jessevz commented Apr 11, 2026

We could reduce the amount of database queries here by instead of doing Factory::getHashlistFactory()->set(....) when a count doesnt match, is by adding the field that needs to be updated to an array, and then afterwards do mset if the array is not empty, then both hashcount and cracked can be updated in a single query. But this is a very minor performance improvement so I approved this pr

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.

2 participants