You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Purshotam Shah <pu...@yahoo-inc.com> on 2014/01/29 03:11:27 UTC

Review Request 17477: [OOZIE-1626]pig action pop-up is not working properly in UI

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17477/
-----------------------------------------------------------

Review request for oozie.


Bugs: OOZIE-1626
    https://issues.apache.org/jira/browse/OOZIE-1626


Repository: oozie-git


Description
-------

Sever side fix of OOZIE-1626(patch v2), which make sure that null IDs are never written to DB.

But for older jobs, with multiple null ids, UI fails.

If all IDs are null, then we should added 

var note = new Ext.form.TextField({
                    fieldLabel : 'Child Job',
                    value : 'n/a'
                });

to FormPanel.


Diffs
-----

  webapp/src/main/webapp/oozie-console.js 7999d35 

Diff: https://reviews.apache.org/r/17477/diff/


Testing
-------


Thanks,

Purshotam Shah

Re: Review Request 17477: [OOZIE-1626]pig action pop-up is not working properly in UI

Posted by Rohini Palaniswamy <ro...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17477/#review33237
-----------------------------------------------------------

Ship it!


Ship It!

- Rohini Palaniswamy


On Jan. 29, 2014, 2:11 a.m., Purshotam Shah wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17477/
> -----------------------------------------------------------
> 
> (Updated Jan. 29, 2014, 2:11 a.m.)
> 
> 
> Review request for oozie.
> 
> 
> Bugs: OOZIE-1626
>     https://issues.apache.org/jira/browse/OOZIE-1626
> 
> 
> Repository: oozie-git
> 
> 
> Description
> -------
> 
> Sever side fix of OOZIE-1626(patch v2), which make sure that null IDs are never written to DB.
> 
> But for older jobs, with multiple null ids, UI fails.
> 
> If all IDs are null, then we should added 
> 
> var note = new Ext.form.TextField({
>                     fieldLabel : 'Child Job',
>                     value : 'n/a'
>                 });
> 
> to FormPanel.
> 
> 
> Diffs
> -----
> 
>   webapp/src/main/webapp/oozie-console.js 7999d35 
> 
> Diff: https://reviews.apache.org/r/17477/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Purshotam Shah
> 
>

Re: Review Request 17477: [OOZIE-1626]pig action pop-up is not working properly in UI

Posted by Purshotam Shah <pu...@yahoo-inc.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17477/
-----------------------------------------------------------

(Updated Jan. 29, 2014, 2:11 a.m.)


Review request for oozie.


Bugs: OOZIE-1626
    https://issues.apache.org/jira/browse/OOZIE-1626


Repository: oozie-git


Description
-------

Sever side fix of OOZIE-1626(patch v2), which make sure that null IDs are never written to DB.

But for older jobs, with multiple null ids, UI fails.

If all IDs are null, then we should added 

var note = new Ext.form.TextField({
                    fieldLabel : 'Child Job',
                    value : 'n/a'
                });

to FormPanel.


Diffs
-----

  webapp/src/main/webapp/oozie-console.js 7999d35 

Diff: https://reviews.apache.org/r/17477/diff/


Testing
-------


Thanks,

Purshotam Shah