pub fn expr_to_columns(expr: &Expr, accum: &mut HashSet<Column>) -> Result<()>
Recursively walk an expression tree, collecting the unique set of columns referenced in the expression