{"body":"1. If not in the API though, I'd prefer to see a wrapper method that can decorate certain \n   DB API create methods so we don't have to copy/paste this code all over.\n\nThis is ok, but decorating will call this method every time a call is made...\nI have proposed to check for 4 byte unicode only in case of failure just to save some cpu cycles.\n\n2. Why not re-raise the DataError?\n   If we re-raise DataError, I think some weird db error will be displayed to user..\n   So, moved this to invalid() specifically..\n\n3. 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\n  is not the minimum required version that we need in global-requirements:\n\nYou mean to update global-requirements with 4.4.0 in this patch, right?\nJust want to clear for better understanding.","name":"","extension":"txt","url":"https://www.irccloud.com/pastebin/oAbNytAl","modified":1456423064,"id":"oAbNytAl","size":840,"lines":15,"own_paste":false,"theme":"","date":1456423064}