Sure, if those tables are not being used otherwise.
Edit:
He should also drop the table: dbo.schema_version. This is the flyway table that keeps track of which tables have been deployed. The reason why he should drop this is that the flyway DDL for MSSQL tests to see if the table already exists, and if so, considers it done. Example is here:
So, if you don’t delete the dbo.schema_version, then it will try to continue where it failed last time, which is after the cohort_definition creation (which was skipped because it detected the table already existed), but the table that was there is the wrong table definition (it came from the CDM Schema). So, here’s what he needs to do:
DROP TABLE dbo.schema_version;
DROP TABLE dbo.cohort;
DROP TABLE dbo.chort_definition;
Then, next time he runs Olympus, it will run all the flyway migration scripts from step 0, and that will create the correct tables, and set up the proper database relationships.
-Chris