Fix situation when a view full-refresh may remove another view if that view has '_mv' as suffix

[koletzilla]

Summary

A situation that we didn't controlled in #516. Related to #507

1. Model with materialization: view is created with hackers name
2. Model with materialization: materializaed_view is created with hackers_mv name.

• Take into account that in this case 2 resources are created: a CH table named hackers_mv and a CH MV called hackers_mv_mv

3. Refresh view hackers -> Error is raised because the code is trying to drop the view hackers_mv but that's a table. if it were a view, we would have been removing it :(

The Error came on the new code we added to drop the MVs:

• The code that internally looks for MVs pointing to hackers works ok: it doesn't find MVs.
• The issue is that when we call clickhouse__drop_mvs macro, it just tries to remove any _mv thing available, even if it doesn't make sense

Changes:

• Created a new clickhouse__drop_mvs_with_names macro and send to it the filtered list of names to drop so now it doesn't include already filtered ones.

[windsurf-bot]

Looks good to me 🤙

💡 To request another review, post a new comment with "/windsurf-review".