From c7a709039af71f11f079b76f810fda693101ef4b Mon Sep 17 00:00:00 2001 From: David Kagan Date: Fri, 7 Apr 2023 14:21:17 -0400 Subject: [PATCH] fixed name size to 230 due to previous commit acknowledgement --- pilosa.go | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/pilosa.go b/pilosa.go index 2e4ab330a..e416775fb 100644 --- a/pilosa.go +++ b/pilosa.go @@ -47,7 +47,7 @@ var ( ErrInvalidView = errors.New("invalid view") ErrInvalidCacheType = errors.New("invalid cache type") - ErrName = errors.New("invalid index or field name, must match [a-z][a-z0-9Θ_-]* and contain at most 300 characters") + ErrName = errors.New("invalid index or field name, must match [a-z][a-z0-9Θ_-]* and contain at most 230 characters") // ErrFragmentNotFound is returned when a fragment does not exist. ErrFragmentNotFound = errors.New("fragment not found") @@ -135,7 +135,11 @@ func newPreconditionFailedError(err error) PreconditionFailedError { } // Regular expression to validate index and field names. -var nameRegexp = regexp.MustCompile(`^[a-z][a-z0-9Θ_-]{0,299}$`) +// The lowest limitation I've seen on any filesystem we care about is 255 +// characters. 230 leaves enough space that an index or field could be +// backed up and have a timestamp and file extension appended while +// still allowing for much longer index and field names. --Jaffee +var nameRegexp = regexp.MustCompile(`^[a-z][a-z0-9Θ_-]{0,229}$`) // TimeFormat is the go-style time format used to parse string dates. const TimeFormat = "2006-01-02T15:04"