fix: Check for snowflake functions when setting up materialization engine by benchoncy · Pull Request #4456 · feast-dev/feast

Why remove the existing stage check? Does checking for functions make checking for stage redundant?

Thanks for the review, @tokoko. My understanding is the stage is a stepping stone to creating the functions as a place to upload and store files required by the functions. The main objective is to create the functions, as part of that feast creates a stage, but outside of these functions, this stage goes unused.

The functions cannot exist without the stage, and the stage is not reused elsewhere, so a stage check is redundant when checking for functions. Instead, I've added IF NOT EXISTS to the stage creation to make the operation idempotent and avoid failure if the stage already exists. I hope this better explains my reasoning.