HUE-1857 [oozie] Allow import workflows with subworkflow identified at runtime

Review Request #4052 — Created Jan. 22, 2014 and submitted

abec
old-hue-rw
HUE-1857
hue
enricoberti, romain
commit f29343a649e6396ad56e51a35004c1489047b8ae
Author: Abraham Elmahrek <abraham@elmahrek.com>
Date:   Wed Jan 22 14:19:12 2014 -0800

    HUE-1857 [oozie] Allow import workflows with subworkflow identified at runtime
    
    Fix subworkflow ID handling so that we can see what subworkflow is selected.
    Make subworkflow optional in the editor as well.

:100644 100644 786cd9a... 52594df... M	apps/oozie/src/oozie/forms.py
:100644 100644 dace0dd... a13f881... M	apps/oozie/src/oozie/importlib/workflows.py
:000000 100644 0000000... 2ce84d5... A	apps/oozie/src/oozie/migrations/0024_auto__chg_field_subworkflow_sub_workflow.py
:100644 100644 1f5e99e... 8675655... M	apps/oozie/src/oozie/models.py
:100644 100644 99a9e16... fb6880d... M	apps/oozie/src/oozie/utils.py
:100644 100644 774402a... bf8f44e... M	apps/oozie/src/oozie/views/api.py
:100644 100644 f29968c... 8a0f365... M	apps/oozie/static/js/workflow.js
:100644 100644 8f207f0... 0c929d2... M	apps/oozie/static/js/workflow.node.js
Tested import path.
Made sure subworkflow selection is editable.
  • 2
  • 0
  • 0
  • 2
  • 4
Description From Last Updated
I wonder if this could be added to request.warn as messages? romain romain
same romain romain
romain
  1. Cool, will create a bug for adding a test.
    
    At some point we will have to improve the UI of the action + provide a text field for variables in the sub workflow action.
  2. I wonder if this could be added to request.warn as messages?
    1. Checked exception to bubble this up? Also, keep logs I feel?
    2. Doing this in another jira.
  3. apps/oozie/static/js/workflow.js (Diff revision 1)
     
     
    why not int or null?
    1. knockout "checked" doesn't understand ints apparently. It wasn't working :(.
  4. apps/oozie/static/js/workflow.node.js (Diff revision 1)
     
     
    don't we send ints or null anyway?
    1. Need to convert for KO "checked" to work.
  5. 
      
abec
Review request changed

Status: Closed (submitted)

Loading...