-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46269/
-----------------------------------------------------------
Review request for Ambari, Jonathan Robie, Nate Cole, and Nahappan Somasundaram.
Bugs: AMBARI-15915
https://issues.apache.org/jira/browse/AMBARI-15915
Repository: ambari
Description
-------
1. Inlined alter table statements (except one to overcome a circular FK
dependency)
2. Reorganized create table statemens to avoid forward references
3. Named all unnamed constraints, shortened names to fit Oracle's 30 character
limit
4. Unified constraint names across database flavors
5. Wrote a JUnit test that checks DDL's for consistency (all constraints are
named, the same tables and constraints are defined for all database flavors)
Diffs
-----
ambari-server/src/main/resources/Ambari-DDL-Derby-CREATE.sql f90ac96
ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 2b214c4
ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql fc93372
ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 870a8e8
ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql
71d4813
ambari-server/src/main/resources/Ambari-DDL-SQLAnywhere-CREATE.sql 6e600c7
ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql a3ea10d
ambari-server/src/test/java/org/apache/ambari/server/orm/db/DDLTestUtils.java
PRE-CREATION
ambari-server/src/test/java/org/apache/ambari/server/orm/db/DDLTests.java
PRE-CREATION
Diff: https://reviews.apache.org/r/46269/diff/
Testing
-------
1. Run all java unit tests in ambari-server. All passed, except
ViewDirectoryWatcherTest.testDirectoryExtractionOnFileAdd(). Failure occured to
a hardcoded timeout value in the test being to low for my environment (I raised
the timeout from 7 secs to 20 and the test passed)
2. Run DDL's with their respective DB environment. In cases where the DDL
contained additional things other than table/index cration and insertion (e.g.
creating databases and users) I only run the table/index creation part.
Thanks,
Balázs Bence Sári
