kelindar / column

High-performance, columnar, in-memory store with bitmap indexing in Go

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

ForKind should return an error instead of panicing

lukasmalkmus opened this issue · comments

The ForKind function panics for an unknown column type:

panic(fmt.Errorf("column: unsupported column kind (%v)", kind))

But it would be nicer if it would error and be picked up by:

c.CreateColumn(k, ForKind(reflect.TypeOf(v).Kind()))

Added in the last PR #39