----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32312/#review77660 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserWidgetDAO.java <https://reviews.apache.org/r/32312/#comment125816> Can use daoUtils.selectList, which returns an empty collection when there are no results. Same for the methods: findBySectionName findAll ambari-server/src/main/java/org/apache/ambari/server/orm/dao/WidgetLayoutDAO.java <https://reviews.apache.org/r/32312/#comment125822> Can use daoUtils.selectList, which returns an empty collection when there are no results. Same for the methods: findBySectionName findAll ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UserWidgetEntity.java <https://reviews.apache.org/r/32312/#comment125824> This is just my preference, but can we start using "id" for ID fields, instead of {table_name}_id? ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java <https://reviews.apache.org/r/32312/#comment125828> Can use dbAccessor.addFKConstraint so it works for all DB-types. ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql <https://reviews.apache.org/r/32312/#comment125829> Suggest NUMBER(19) since it's a Long, in all of the other IDs as well. - Alejandro Fernandez On March 24, 2015, 9:37 p.m., Ivan Kozlov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32312/ > ----------------------------------------------------------- > > (Updated March 24, 2015, 9:37 p.m.) > > > Review request for Ambari, Jaimin Jetly and Sid Wagle. > > > Repository: ambari > > > Description > ------- > > Apart from adding new widget related table definitions in the DDL script for > all the three database types, also update the upgrade catalog for > Amabri-2.1.0 to create these tables on ambari upgrade. > Tasks: > Create WidgetEntity and WidgetLayoutEntity. > Add relationship from both entities to cluster. This will not be used to load > widgets but only to find what StackVersion to use for lookup of stack widgets > when requesting WidgetInfo with layouts. > Create corresponding DAO objects with unit tests > Create findById and findBySectionName and findByCluster > Note: There is no direct relationship between the two. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UserWidgetDAO.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/dao/WidgetLayoutDAO.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UserWidgetEntity.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/WidgetLayoutEntity.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/WidgetLayoutUserWidgetEntity.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/orm/entities/WidgetLayoutUserWidgetEntityPK.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/SchemaUpgradeHelper.java > 5968b2f > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java > PRE-CREATION > ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 9ff62df > ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 7d62aee > ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 78a263f > ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql > a06f1d2 > ambari-server/src/main/resources/META-INF/persistence.xml 8c0d85f > > ambari-server/src/test/java/org/apache/ambari/server/orm/dao/UserWidgetDAOTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/orm/dao/WidgetLayoutDAOTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog210Test.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/32312/diff/ > > > Testing > ------- > > > Thanks, > > Ivan Kozlov > >
