envoyproxy/envoy

CacheFilter: Ensure that cache key hash function is long-term stable

Open

#10,135 opened on Feb 21, 2020

View on GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (5,373 forks)batch import
area/cachehelp wanted

Repository metrics

Stars
 (27,997 stars)
PR merge metrics
 (Avg merge 8d) (378 merged PRs in 30d)

Description

CacheFilter uses MessageUtil::hash to compute key hashes, but it isn't clear whether MessageUtil::hash is guaranteed stable across releases, operating systems, etc. This stability is needed for persistent caches, to avoid cache flushes.

Not a blocker for most deployments.

Contributor guide

Research direction
Investigate the MessageUtil::hash implementation in the codebase to determine if it is guaranteed stable across releases, operating systems, and architectures. Check if it relies on any platform specific or non deterministic behavior. Consider proposing a custom hash function if needed.
Tech stack
cpp
Domain
backendnetworking
Issue type
Research
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Needs investigation
Prerequisites
C++hash functions
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40