corepointer
commented
1 year ago I have left a few inline comments for improvements. In addition to that:
- Compiling with
`--target allfails somewhere in the theta_join_test iirc. - The test cases need to run successfully
- I know I'm a bit lazy on this one myself but I have to mention it: please write documentation (for users and developers alike)
- After creating this diff commit, I appear as the author of this. We'll fix this in the final merge.
- You could put the changes into sub-topic commits
- Please write a detailed commit message if this should go in all at once
This is a work in progress to bring the columnar ops developments to the main branch.
I have created a diff to main branch based on #640.
More details on what still needs fixing will follow.