Did anyone else thing that the fact that he needed to do
cache.set_user(user, id=user.id, username=user.username)
twice in his improved solution a bit of a giant red flag?