Martin Peřina has posted comments on this change.

Change subject: core: Fix CDI in dal module to work with EAP/WildFly
......................................................................


Patch Set 3:

@Liran:

EAP 6 will be minimal supported platform, so we will stick to CDI 1.0, but EAP 
contains much more newer and fixed Weld then JBoss 7.1. Upstream oVirt will be 
executed on WildFly, but we won't be able to use its new features until 
downstream will be moved to EAP 7.


@Yevgeny:

I didn't want to loose ability to load db dialect dynamically using 
configuration file, although you're right that another supported db is in the 
distant future ...
I will try to use @New and post results if it helps or not

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I122b02f846cfbb9b861d13792790cf93ca7ee0d6
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Liran Zelkha <[email protected]>
Gerrit-Reviewer: Martin Peřina <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yevgeny Zaspitsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to