Binary Boolean VTable - Githubissues

spiraldb / vortex

An extensible, state-of-the-art columnar file format

https://vortex.dev

Apache License 2.0

996 stars 27 forks source link

Closed gatesn closed 5 days ago

gatesn commented 5 days ago

Also merged all binary boolean functions since they're trivial to implement if you implement one of them.
And switched over to similar semantics as CompareFn where an encoding gets to see the arguments before deciding if they're supported.

FLUP: we weren't short-circuiting boolean operations if one of the sides was constant.

gatesn commented 5 days ago

it is a bit weird to have a "vtable" that requires executing the function to figure out the mapping, but I don't have a better solution rn

I think #1409 might be a better fix?