Hi Tom,> is quite useless: we dealt with expr->plan == NULL a few lines> earlier, so this if-test can never succeed. Pushed with that> change.
Thanks for pushing, and for the quick look. Agreed on the plan check.
Thanks,Shihao