Change in ovirt-engine[master]: core: Explicitly require JEE concurrency API

Code Review gerrit at ovirt.org
Thu Jul 20 06:52:30 UTC 2017


>From Juan Hernandez <juan.hernandez at redhat.com>:

Juan Hernandez has submitted this change and it was merged. ( https://gerrit.ovirt.org/79615 )

Change subject: core: Explicitly require JEE concurrency API
......................................................................


core: Explicitly require JEE concurrency API

A recent patch changed the 'utils' module to use the JEE concurrency
API, but it didn't explicitly include the required artifact in the POM
file. In some environemts this means that the project won't build
correctly, in particular it doesn't build correctly in the latest
version of IntelliJ IDEA. This patch adds the dependency, as it is good
practice and solves the issue in this environment.

Change-Id: Ib449fcd2d72c8ed3d7d3f6964264a94931e2d990
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/modules/utils/pom.xml
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Ravi Nori: Looks good to me, but someone else must approve
  Juan Hernandez: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib449fcd2d72c8ed3d7d3f6964264a94931e2d990
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list