@johnsimons I think you mixed 2 different concepts here

We want to phase out the use of attachments completely; this will save on operations (one write per msg instead of 2), and also will not be another thing to track when expiring messages.

What we decided was to store message bodies for failed messages (always, no matter what), and for non-binary audit messages which are not too big (up to 100 or 500kb).

This thing here sort of does both half way. Or am I missing something?

/cc @andreasohlund