feat: new experimental gc friendly flatten cache

[rueian]

No description provided.

[Copilot]

Pull Request Overview

Introduces a new experimental GC-friendly flatten cache implementation by adding a time.Time parameter to the Update method signature across the caching system. This change enables better temporal tracking and supports the implementation of a new chained cache structure with improved memory management.

Key changes:

• Updated Update method signature to include time.Time parameter
• Added new cache data structures in internal/cache package with optimized memory layout
• Implemented NewChainedCache as an experimental alternative to the default cache

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
pipe_test.go	Updated test calls to include time parameter in cache Update method
pipe.go	Modified cache Update calls to pass current time
lru_test.go	Updated all test cases to use new Update method signature
lru.go	Added time parameter to Update method (parameter is unused)
internal/cache/lru_test.go	Added comprehensive tests for new LRU double map implementation
internal/cache/lru.go	Implemented new LRU double map with GC-friendly design
internal/cache/double_test.go	Added tests for new double map data structure
internal/cache/double.go	Implemented core double map functionality
internal/cache/chain_test.go	Added tests for chain data structure
internal/cache/chain.go	Implemented linked chain for hash collision handling
cache_test.go	Updated test calls and added test for new flatten cache
cache.go	Updated Update method signature and implemented new chained cache

Comments suppressed due to low confidence (1)

lru.go:221

• [nitpick] The parameter name _ (blank identifier) indicates the time parameter is intentionally unused. Consider using a more descriptive name like now or timestamp for better code readability, even if the parameter is currently unused.

func (c *lru) Update(key, cmd string, value RedisMessage, _ time.Time) (pxat int64) {

[Copilot]

The condition len(e.s) >= 0 is always true since slice length is never negative. This should likely be len(e.s) > 0 to only delete when there are actually entries to process.

Suggested change

	if len(e.s) >= 0 {
	if len(e.s) > 0 {