The standard bulk builder for B-tree indexes and the column store bulk builder share a lot of logic but have completely separate commit() functions that duplicate a lot of functionality. It may be worthwhile to refactor this code so that there is a single commit() function that can use dynamic dispatch to alter its behavior for the type of index it is loading.