Hi All Patch [1] is about to be merged soon and it changes the behaviour of dbscripts/common_sp.sql functions validation.
In dbscripts/common_sp.sql we have several helper functions that execute dynamic SQL using the EXECUTE command In order to validate parameters as table name , column name etc, we used to check those parameters with the information stored in the information_schema data Due to that we had some side effects: 1. fn_db_change_column_type did not accept valid types as DECIMAL since in the information_schema data in is named NUMERIC 2. fn_db_change_column_type did not accept precision in the origin column type as NUMERIC(24,4) for example 3. fn_db_change_column_type swallowed any error causing scripts not to perform the action but to consider as successful Patch [1] fixes all those issues and now the comon_sp.sql functions will trust errors thrown by using the EXECUTE command without validating the SQL syntax before. This will resolve issues 1 and 2 listed above In addition, the patch fixes issue 3 by adding the needed upgrade scripts to enforce the changes that were not performed in former upgrade scripts updates. Please feel free to contact me on any issues/questions regarding those changes Regards Eli Mesika [1] https://gerrit.ovirt.org/#/c/ovirt-engine/+/117448/
_______________________________________________ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-le...@ovirt.org Privacy Statement: https://www.ovirt.org/privacy-policy.html oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/devel@ovirt.org/message/QT533FHOOFYZFXKTEKYWULYCGTV7C2JT/
