Skip to content

CacheValueError name is confusing #23

@rlinehan

Description

@rlinehan

Currently we raise a CacheValueError for any INVALID_ARGUMENT grpc status codes. For example, this is raised if create_cache is called with an empty string.

This name was derived from ValueError in Python, but the naming of it makes it seem like something is wrong with the value being cached.

It might be better to rename this to something like InvalidArgumentError so it's clear that it's not just for cache values.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions