From d27d1dfcbe2c90ab330ca4de220284496c5c7565 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=81lex=20Gonz=C3=A1lez?= Date: Mon, 3 Nov 2014 00:25:44 +0000 Subject: [PATCH] Log when truncindex.Get returns >1 container When the user is not using the full has to retrieve a container it's possible that we find conflicts with the ids of other containers. At the moment it's just failing saying that it can not find a container, but it doesn't say why. Adding a small log saying that duplicates where found is going to help the user. Closes #8098 Signed-off-by: Alex Gonzalez --- truncindex/truncindex.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/truncindex/truncindex.go b/truncindex/truncindex.go index c5b7175..eec5597 100644 --- a/truncindex/truncindex.go +++ b/truncindex/truncindex.go @@ -11,8 +11,9 @@ import ( var ( // ErrNoID is thrown when attempting to use empty prefixes - ErrNoID = errors.New("prefix can't be empty") - errDuplicateID = errors.New("multiple IDs were found") + ErrNoID = errors.New("prefix can't be empty") + // ErrDuplicateID is thrown when a duplicated id was found + ErrDuplicateID = errors.New("multiple IDs were found") ) func init() { @@ -98,7 +99,7 @@ func (idx *TruncIndex) Get(s string) (string, error) { if id != "" { // we haven't found the ID if there are two or more IDs id = "" - return errDuplicateID + return ErrDuplicateID } id = string(prefix) return nil