# Pastebin oAbNytAl
1. If not in the API though, I'd prefer to see a wrapper method that can decorate certain 
   DB API create methods so we don't have to copy/paste this code all over.

This is ok, but decorating will call this method every time a call is made...
I have proposed to check for 4 byte unicode only in case of failure just to save some cpu cycles.

2. Why not re-raise the DataError?
   If we re-raise DataError, I think some weird db error will be displayed to user..
   So, moved this to invalid() specifically..

3. Doesn't this require https://review.openstack.org/#/c/265921/4/oslo_db/sqlalchemy/exc_filters.py which went into oslo.db 4.4.0 which
  is not the minimum required version that we need in global-requirements:

You mean to update global-requirements with 4.4.0 in this patch, right?
Just want to clear for better understanding.