can this be varchar instead of text?  It might not matter in terms of performance, but there's no reason to have text where a varchar will do, especially as we use varchar everywhere else