When getting cache values, dereference pointers.

This allows using pointer fields (e.g. *string) as client indices.

Signed-off-by: Nadia Pinaeva <[email protected]>

cache/cache.go

@@ -1251,7 +1251,7 @@ func valueFromIndex(info *mapper.Info, columnKeys []model.ColumnKey) (interface{
 }
 
 func valueFromColumnKey(info *mapper.Info, columnKey model.ColumnKey) (interface{}, error) {
-	val, err := info.FieldByColumn(columnKey.Column)
+	val, err := info.DerefFieldByColumn(columnKey.Column)
 	if err != nil {
 		return nil, err
 	}

mapper/info.go

@@ -48,6 +48,20 @@ func (i *Info) FieldByColumn(column string) (interface{}, error) {
 	return reflect.ValueOf(i.Obj).Elem().FieldByName(fieldName).Interface(), nil
 }
 
+// DerefFieldByColumn returns the field value that corresponds to a column.
+// If value is a pointer, it will be dereferenced.
+func (i *Info) DerefFieldByColumn(column string) (interface{}, error) {
+	fieldName, ok := i.Metadata.Fields[column]
+	if !ok {
+		return nil, NewErrColumnNotFound(column, i.Metadata.TableName)
+	}
+	v := reflect.ValueOf(i.Obj).Elem().FieldByName(fieldName)
+	if v.Kind() == reflect.Ptr && !v.IsNil() {
+		v = v.Elem()
+	}
+	return v.Interface(), nil
+}
+
 // FieldByColumn returns the field value that corresponds to a column
 func (i *Info) hasColumn(column string) bool {
 	_, ok := i.Metadata.Fields[column]