feat/Refactor reimplement referenced columns & has table ref column & has agg call using visitor

[Howard2725]

What problem does this PR solve?

solving the issue#273
Refactor: reimplement referenced_columns & has_table_ref_column & has_agg_call using Visitor #273

Add corresponding issue link with summary if exists -->

Issue link:

What is changed and how it works?

Code changes

• [ v] Has Rust code change
• Has CI related scripts change

Check List

Tests

• [v ] Unit test
• Integration test
• Manual test (add detailed scripts or steps below)
• No code

Side effects

• Performance regression: Consumes more CPU
• Performance regression: Consumes more Memory
• Breaking backward compatibility

Note for reviewer

[KKould]

fn visit(&mut self, expr: &'a ScalarExpression) -> Result<(), DatabaseError> {
    if self.found {
        return Ok(());
    }
    walk_expr(self, expr)
}