Standardize extensions redis init

[tylerhutcherson]

Standardizes the way our extension modules connect to redis.

• First check for client
• Then check for URL
• Then raise a CLEAR error if neither are provided

[tylerhutcherson]

(I need to fix the failing test on the notebook)

[justin-cechmanek]

Looks good. Just curious why we don't want to have a default redis url anymore

[tylerhutcherson]

Looks good. Just curious why we don't want to have a default redis url anymore

Certainly open to discussing this with everyone more. Let me ping the rest of the team to see what the collective thoughts are. My preference is for these to be less magicy and focus on clarity. But if it'a a common of enough pattern, we can go for it.

[tylerhutcherson]

Looks good. Just curious why we don't want to have a default redis url anymore

Certainly open to discussing this with everyone more. Let me ping the rest of the team to see what the collective thoughts are. My preference is for these to be less magicy and focus on clarity. But if it'a a common of enough pattern, we can go for it.

After reviewing with team, we went back to defaulting to localhost Redis.

[justin-cechmanek]

Looks good!