Added more tests, removed unnecessary code

kelindar / column

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

MIT License

1.44k stars 57 forks source link

Closed kelindar closed 3 years ago

kelindar commented 3 years ago

This PR includes some fixes and a better error handling for the indexes:

CreateIndex now requires column to be created before the index. Prior to this you could create an index on a nil column, which was a weird behaviour.
DropIndex now has more validation.
Replaced fnv1 with crc32 for enum column, to simplify the code a bit.