[UI][EDP] Current validation is too loose for MapReduce and Java jobs

Bug #1272506 reported by Trevor McKay
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Sahara
Fix Released
Medium
Chad Roberts

Bug Description

For MapReduce and Java jobs, validation allows "mains" or "libs" to be set to pass validation. The truth is, for MapReduce and Java jobs "libs" must be set -- Oozie will only distribute files in "libs" for those job types. Putting something only in "mains" will result in a class not found error.

In fact, putting anything in "mains" for those job types is useless.

The UI should communicate this to the user, too.

Trevor McKay (tmckay)
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to savanna (master)

Reviewed: https://review.openstack.org/69019
Committed: https://git.openstack.org/cgit/openstack/savanna/commit/?id=98035a4f53260f6f496a0c5c441c6cdb78c5b5b3
Submitter: Jenkins
Branch: master

commit 98035a4f53260f6f496a0c5c441c6cdb78c5b5b3
Author: Trevor McKay <email address hidden>
Date: Fri Jan 24 16:20:19 2014 -0500

    Require "libs" for MapReduce and Java jobs and disallow "mains"

    The validation for MapReduce and Java job types was too loose.
    It required that mains or libs be set, but in reality libs is
    required and mains is useless in Oozie workflows for these types.

    Partial-Bug: #1272506

    Change-Id: I94a86f9581346741f935e2b79b3ba582b06f1f76

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to savanna-dashboard (master)

Reviewed: https://review.openstack.org/69432
Committed: https://git.openstack.org/cgit/openstack/savanna-dashboard/commit/?id=32c1b2b25b902ee4d378e5e71e896ec52221a816
Submitter: Jenkins
Branch: master

commit 32c1b2b25b902ee4d378e5e71e896ec52221a816
Author: Chad Roberts <email address hidden>
Date: Mon Jan 27 13:22:02 2014 -0500

    Dashboard now hides "mains" for Java/MapReduce jobs

    Mains are now disallowed by the api for Java and MapReduce jobs.
    Given that, the dashboard now hides mains when those job types
    are selected. Also adding some help text on the job create page
    to clarify what is going on.

    Partial-Bug: #1272506

    Change-Id: I780ff0e6bf002fb7bc626da6a35619bb4e9c2aa8

Chad Roberts (croberts)
Changed in savanna:
status: New → Fix Committed
assignee: nobody → Chad Roberts (croberts)
importance: Undecided → Medium
Changed in savanna:
milestone: none → icehouse-3
Thierry Carrez (ttx)
Changed in savanna:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in sahara:
milestone: icehouse-3 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.