mirror of
https://github.com/hanzokv/go.git
synced 2026-09-24 00:31:51 +00:00
zeekay
6994cf05aa
We do not use Redis. The client was a go-redis fork and still said so
everywhere: `package redis`, `redis.NewClient`, `redis://`, `extra/redisotel`.
Consumers wrote `redis.Client` against a Valkey-backed store called KV.
package redis -> package kv (93 decls)
redis.X -> kv.X (2,794 call sites)
"redis: …" errors -> "kv: …" (206)
extra/redis{otel,cmd,prometheus,census} -> extra/kv{otel,cmd,prometheus,census}
redis:// , rediss:// -> kv:// , kvs:// (ParseURL)
Redis*/*Redis* idents -> KV*/*KV*
REDIS_VERSION, REDIS_ENDPOINTS_CONFIG_PATH -> KV_*
I previously argued this rename was unsafe because much of it was "wire
contract". That was wrong, and the categorisation shows why: RESP command names
are GET/SET/HSET — they never contain the word. Of 5,771 occurrences only two
kinds were server-visible at all: the ParseURL scheme (we choose what we pass;
our configs already use kv://) and the CLIENT SETINFO lib-name label, which the
server merely records. Nothing here changes a byte of protocol.
Two things deliberately keep the old spelling:
RediSearch — names a specific external module the FT.* commands map to.
StoreDist — contains the letters by accident (sto-reDis-t), not the word.
Breaking for consumers by design: every import site must move redis.X -> kv.X.
Minor bump, never a major — this module stays v9 forever.
55 lines
1.6 KiB
Go
55 lines
1.6 KiB
Go
// Package interfaces provides shared interfaces used by both the main kv package
|
|
// and the maintnotifications upgrade package to avoid circular dependencies.
|
|
package interfaces
|
|
|
|
import (
|
|
"context"
|
|
"net"
|
|
"time"
|
|
)
|
|
|
|
// NotificationProcessor is (most probably) a push.NotificationProcessor
|
|
// forward declaration to avoid circular imports
|
|
type NotificationProcessor interface {
|
|
RegisterHandler(pushNotificationName string, handler interface{}, protected bool) error
|
|
UnregisterHandler(pushNotificationName string) error
|
|
GetHandler(pushNotificationName string) interface{}
|
|
}
|
|
|
|
// ClientInterface defines the interface that clients must implement for maintnotifications upgrades.
|
|
type ClientInterface interface {
|
|
// GetOptions returns the client options.
|
|
GetOptions() OptionsInterface
|
|
|
|
// GetPushProcessor returns the client's push notification processor.
|
|
GetPushProcessor() NotificationProcessor
|
|
}
|
|
|
|
// OptionsInterface defines the interface for client options.
|
|
// Uses an adapter pattern to avoid circular dependencies.
|
|
type OptionsInterface interface {
|
|
// GetReadTimeout returns the read timeout.
|
|
GetReadTimeout() time.Duration
|
|
|
|
// GetWriteTimeout returns the write timeout.
|
|
GetWriteTimeout() time.Duration
|
|
|
|
// GetNetwork returns the network type.
|
|
GetNetwork() string
|
|
|
|
// GetAddr returns the connection address.
|
|
GetAddr() string
|
|
|
|
// IsTLSEnabled returns true if TLS is enabled.
|
|
IsTLSEnabled() bool
|
|
|
|
// GetProtocol returns the protocol version.
|
|
GetProtocol() int
|
|
|
|
// GetPoolSize returns the connection pool size.
|
|
GetPoolSize() int
|
|
|
|
// NewDialer returns a new dialer function for the connection.
|
|
NewDialer() func(context.Context) (net.Conn, error)
|
|
}
|