Skip to content

Add Statistics hashes_with_expiry_fields to INFO#13275

Merged
moticless merged 2 commits intoredis:hash-field-expiry-integfrom
moticless:hfe-integ-statistics
May 23, 2024
Merged

Add Statistics hashes_with_expiry_fields to INFO#13275
moticless merged 2 commits intoredis:hash-field-expiry-integfrom
moticless:hfe-integ-statistics

Conversation

@moticless
Copy link
Copy Markdown
Collaborator

@moticless moticless commented May 16, 2024

Added hashes_with_expiry_fields.
Optimially it would better to have statistic of that counts all fields with expiry. But it requires careful logic and computation to follow and deep dive listpacks and hashes. This statistics is trivial to achieve and reflected by global HFE DS that has builtin enumeration of all the hashes that are registered in it.

@moticless moticless changed the title Add Statisticst to INFO of hashes_with_expiry_fields Add Statistics to INFO of hashes_with_expiry_fields May 16, 2024
@moticless moticless changed the title Add Statistics to INFO of hashes_with_expiry_fields Add Statistics hashes_with_expiry_fields to INFO May 17, 2024
@ronen-kalish
Copy link
Copy Markdown
Collaborator

LGTM

@moticless moticless merged commit f34f2ad into redis:hash-field-expiry-integ May 23, 2024
@moticless moticless deleted the hfe-integ-statistics branch May 23, 2024 14:29
@sundb
Copy link
Copy Markdown
Collaborator

sundb commented May 26, 2024

@sundb sundb added the state:needs-doc-pr requires a PR to redis-doc repository label May 26, 2024
funny-dog pushed a commit to funny-dog/redis that referenced this pull request Sep 17, 2025
Added hashes_with_expiry_fields.
Optimially it would better to have statistic of that counts all fields
with expiry. But it requires careful logic and computation to follow and
deep dive listpacks and hashes. This statistics is trivial to achieve
and reflected by global HFE DS that has builtin enumeration of all the
hashes that are registered in it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:needs-doc-pr requires a PR to redis-doc repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants