What purpose does this test serve w.r.t this patch? Before checking the sync for different column orders, the patch has already changed binary to false, so it doesn't seem to test the functionality of this patch. Am, I missing something?
I missed that binary has changed to false before testing column orders. I moved that test case up before changing binary to false.