Change in ovirt-engine[master]: engine-config : add Double type support

masayag at redhat.com masayag at redhat.com
Tue Jul 7 16:25:52 UTC 2015


Moti Asayag has submitted this change and it was merged.

Change subject: engine-config : add Double type support
......................................................................


engine-config : add Double type support

Added support for Double type

Example: Adding ElimDoubleTest

engine-config.properties
---------------------------

ElimDoubleTest.type=Double
ElimDoubleTest.description=Test Double values

ConfigValues:
---------------------------

@TypeConverterAttribute(Double.class)
@DefaultValueAttribute("1.1")
ElimDoubleTest,

I also added the code for installing this value in 0000_config.sql and
called

Double d = Config.getValue("ElimDoubleTest");

But all the above was only for testing, the new helper class is the only
code added since all code uses either Generics or dynamic load of the
helper.

Change-Id: I0608da138b57c9b571e082eba2e5991b95ede01f
Signed-off-by: emesika <emesika at redhat.com>
---
A backend/manager/tools/src/main/java/org/ovirt/engine/core/config/entity/helper/DoubleValueHelper.java
1 file changed, 35 insertions(+), 0 deletions(-)

Approvals:
  Eli Mesika: Verified
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/43210
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0608da138b57c9b571e082eba2e5991b95ede01f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list