fix: Fixed Registry Cache Refresh Issues by ntkathole · Pull Request #5604 · feast-dev/feast

ntkathole

What this PR does / why we need it:

This PR fix the issues with the Feast registry cache refresh mechanism:

Fixed Circular Dependency in sql.py. The proto() method was calling cached methods (list_*) which would trigger cache refresh, creating a circular dependency.
Improved the readability of logic to handle cache expiry. Added is_cache_expired() function with ttl handling.
Added test coverage for cache ttl.

While testing the fix, identified issue with metric api, that's also fixed.