machristie commented on a change in pull request #255:
URL: https://github.com/apache/airavata/pull/255#discussion_r442489604



##########
File path: 
modules/registry/release-migration-scripts/next/DeltaScripts/appCatalog_schema_delta.sql
##########
@@ -24,3 +24,18 @@ use app_catalog;
 alter table GROUP_COMPUTE_RESOURCE_PREFERENCE drop column IF EXISTS 
RESERVATION;
 alter table GROUP_COMPUTE_RESOURCE_PREFERENCE drop column IF EXISTS 
RESERVATION_START_TIME;
 alter table GROUP_COMPUTE_RESOURCE_PREFERENCE drop column IF EXISTS 
RESERVATION_END_TIME;
+
+-- AIRAVATA-3343: Add UserStorageQuota entry to StoragePreferences table
+SET @AddUserStorageQuota = (SELECT IF(

Review comment:
       So we have migration scripts for production MariaDB databases, thats 
what release-migration-scripts is. Then there are migration scripts in 
ide-integration, the 0*-migrations.sql scripts are.  So yes, in the 
0*-migrations.sql script, IF NOT EXISTS won't work as you point out, but we can 
use IF NOT EXISTS in release-migration-scripts.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to