Skip to content

2.0 Checklist #74

Open
Open
@yaauie

Description

@yaauie

The last 1.x version should support ALL of these changes when Redis::Namespace#initialize is called with deprecations: true OR the REDIS_NAMESPACE_DEPRECATIONS environment varible set, and should warn unless Redis::Namespace#initialize is called with warnings: false OR the REDIS_NAMESPACE_QUIET environment variable set.


  • deprecations pathway
    • deprecation path can be enabled with deprecations: true
    • deprecation path can be enabled with REDIS_NAMESPACE_DEPRECATIONS
    • deprecations: false is default
    • warnings pathway
    • warnings path can be disabled with warnings: false
    • warnings path can be disabled with REDIS_NAMESPACE_QUIET
    • warnings: true is default

  • Don't blindly proxy unknown commands. (Limit method_missing to only handle known commands #73)
  • Don't allow direct use of non-namespaced commands, but provide a straightforward way to run them when done intentionally, perhaps Redis::Namespace#without_namespace.flushdb
    • select
    • flushdb
    • flushall
    • shutdown ?
    • script ?
    • exec
  • Guarantee threadsafety for Redis::Namespace#pipelined and Redis::Namespace#multi (preferably without the use of mutexes, although we can use Redis#synchronize if we need to)
  • Avoid method_missing altogether, since we know the complete list of supported commands

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions