---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28624/#review63641
---
Ship it!
Looks good; a minor nit about @Column vs @Basic. I dislike @Basic :)
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ViewParameterEntity.java
https://reviews.apache.org/r/28624/#comment105915
In this case, you don't need both @Basic and @Column. @Basic is really only
used to override the fetch type; we use @Column usually with an explicit name.
I like using an explicit name since you change the field without repurcussions.
- Jonathan Hurley
On Dec. 2, 2014, 9:33 p.m., Tom Beerbower wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28624/
---
(Updated Dec. 2, 2014, 9:33 p.m.)
Review request for Ambari, Alejandro Fernandez and Jonathan Hurley.
Bugs: AMBARI-8519
https://issues.apache.org/jira/browse/AMBARI-8519
Repository: ambari
Description
---
Add the following view parameter fields :
# label (a user friendly display in the UI). This element is optional.
Admin View: If label is not specified, use the name.
# placeholder text (used as placeholder text in the UI inputfield). This
element is optional. Admin View: If not specified, blank.
# default value. Admin View: if set, use as the default value and shown in
the inputfield. This element is optional. This will show instead of
placeholder text (if placeholder is specified).
For example ...
{code}
parameter
nameambari.server.username/name
descriptionAmbari administrator username (for example:
admin)/description
labelUserName/label
placeholdersome placeholder text.../placeholder
defaultadmin/default
requiredtrue/required
/parameter
{code}
Diffs
-
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ViewInstanceResourceProvider.java
a944e95
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ViewInstanceEntity.java
1dd1b5a
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/ViewParameterEntity.java
0c4c70f
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog200.java
79e0daa
ambari-server/src/main/java/org/apache/ambari/server/view/ViewRegistry.java
77c7c60
ambari-server/src/main/java/org/apache/ambari/server/view/configuration/ParameterConfig.java
827b5e7
ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 4bd374d
ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 9eb4e3a
ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql e8f20f6
ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql
059662f
ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 232b283
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ViewInstanceResourceProviderTest.java
023cba5
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog200Test.java
c920905
ambari-server/src/test/java/org/apache/ambari/server/view/ViewContextImplTest.java
519d010
ambari-server/src/test/java/org/apache/ambari/server/view/configuration/ParameterConfigTest.java
e3350bc
ambari-server/src/test/java/org/apache/ambari/server/view/configuration/ViewConfigTest.java
04cbe2b
ambari-server/src/test/resources/test_view.xml 5742bfe
ambari-views/src/main/resources/view.xsd a9e5b12
Diff: https://reviews.apache.org/r/28624/diff/
Testing
---
New unit tests added. Manual testing for default value.
All existing tests pass ...
Results :
Tests run: 2316, Failures: 0, Errors: 0, Skipped: 22
Thanks,
Tom Beerbower