Use visitor pattern for `expr_has_agg`

[kevinzwang]

The function expr_has_agg in src/daft-dsl/src/expr.rs currently traverses the expression by matching on all the expression types. We could use a tree visitor pattern instead to simplify it.