----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58840/ -----------------------------------------------------------
(Updated May 2, 2017, 12:09 p.m.) Review request for Ambari, Jonathan Hurley, Laszlo Puskas, Sandor Magyari, Sumit Mohanty, Sebastian Toader, and Sid Wagle. Changes ------- re-implemented in Python Bugs: AMBARI-20886 https://issues.apache.org/jira/browse/AMBARI-20886 Repository: ambari Description ------- Based on existing SQL Server script generate at compile-time for AzureDB a SQL script that: * runs CREATE TABLE, CREATE INDEX and ADD CONSTRAINT statements only if the objects do not exist yet, * DELETEs rows before running INSERT statements. Diffs (updated) ----- ambari-server/pom.xml da1f09eceb999650c33729120b6959462445a115 ambari-server/src/main/assemblies/server.xml 51ee14e0bc215ba2fdad24112c7c0ab3ff2d1189 ambari-server/src/main/python/azuredb_create_generator.py PRE-CREATION ambari-server/src/main/resources/.gitignore PRE-CREATION ambari-server/src/main/sh/azuredb_create_generator.sh PRE-CREATION Diff: https://reviews.apache.org/r/58840/diff/4/ Changes: https://reviews.apache.org/r/58840/diff/3-4/ Testing ------- Tested with Microsoft SQL Server on Linux using Docker: ``` $ mvn -q -pl ambari-server clean generate-resources $ export MSSQL_PASSWORD='hVn?1AY6b\l&' $ docker run -e 'ACCEPT_EULA=Y' -e "SA_PASSWORD=$MSSQL_PASSWORD" --name mssql -p 1433:1433 -v $(PWD):/tmp/ambari -d microsoft/mssql-server-linux ba7ad59f747eac75e696cbab8ed7f30af0c98dc81250ee393ebf0e01e71f2965 $ docker exec -it mssql /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P "$MSSQL_PASSWORD" 1> select count(*) from sys.objects; 2> go ----------- 107 (1 rows affected) $ docker exec -it mssql /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P "$MSSQL_PASSWORD" -i /tmp/ambari/ambari-server/src/main/resources/Ambari-DDL-AzureDB-CREATE.sql (0 rows affected) $ docker exec -it mssql /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P "$MSSQL_PASSWORD" 1> select count(*) from sys.objects; 2> go ----------- 525 (1 rows affected) 1> select count(*) from ambari_sequences; 2> go ----------- 54 (1 rows affected) $ docker exec -it mssql /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P "$MSSQL_PASSWORD" -i /tmp/ambari/ambari-server/src/main/resources/Ambari-DDL-AzureDB-CREATE.sql (1 rows affected) (1 rows affected) (184 rows affected) (53 rows affected) (7 rows affected) (1 rows affected) (8 rows affected) (3 rows affected) (1 rows affected) (3 rows affected) (54 rows affected) (54 rows affected) (3 rows affected) (1 rows affected) (3 rows affected) (8 rows affected) (1 rows affected) (7 rows affected) (53 rows affected) (1 rows affected) (14 rows affected) (19 rows affected) (24 rows affected) (31 rows affected) (42 rows affected) (53 rows affected) (1 rows affected) (1 rows affected) $ docker exec -it mssql /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P "$MSSQL_PASSWORD" 1> select count(*) from sys.objects; 2> go ----------- 525 (1 rows affected) 1> select count(*) from ambari_sequences; 2> go ----------- 54 (1 rows affected) $ docker rm -f mssql mssql $ docker run -e 'ACCEPT_EULA=Y' -e "SA_PASSWORD=$MSSQL_PASSWORD" --name mssql -p 1433:1433 -v $(PWD):/tmp/ambari -d microsoft/mssql-server-linux 5dfdd09a70b76f5cd9716fff94680b758c92f3850b227c0c274eea800bdf3402 $ docker exec -it mssql /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P "$MSSQL_PASSWORD" -i /tmp/ambari/ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql (54 rows affected) (3 rows affected) (1 rows affected) (3 rows affected) (8 rows affected) (1 rows affected) (7 rows affected) (53 rows affected) (1 rows affected) (14 rows affected) (19 rows affected) (24 rows affected) (31 rows affected) (42 rows affected) (53 rows affected) (1 rows affected) (1 rows affected) $ docker exec -it mssql /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P "$MSSQL_PASSWORD" 1> select count(*) from sys.objects; 2> go ----------- 525 (1 rows affected) 1> select count(*) from ambari_sequences; 2> go ----------- 54 (1 rows affected) ``` Thanks, Attila Doroszlai