Error correction tracking follow-up from "Upgrade redis gem to v5"
The following discussion from !141213 (merged) should be addressed:
-
@drew started a discussion: (+2 comments) when Integer, Float element.to_s when NilClass # Reporting this gracefully so we can go fix it SomeKindOfLoggerClass.track("unformatted nil passed into redis from: #{backtrace}") element.to_s
@schin1 What do you think of something like this? I think it would be preferable to introduce the CommandBuilder in a way that we plan to use it as a transitional piece and eventually remove it. I don't think we should be catching and fixing any new introductions of bad types being passed into Redis, just looking out for old ones that we don't know about yet.