Purshotam, I have addressed your comments. https://reviews.apache.org/r/38088/
-Akshay On Mon, Sep 21, 2015 at 5:30 PM, Akshay Rai <[email protected]> wrote: > Thanks Purushotam for the review. Will address the changes soon. > > -Akshay > > On Mon, Sep 21, 2015 at 10:55 AM, Akshay Rai <[email protected]> wrote: > >> Eagerly waiting for a review. >> >> -Akshay >> >> On Tue, Sep 15, 2015 at 9:07 AM, Akshay Rai <[email protected]> wrote: >> >>> A gentle reminder. >>> >>> On Thu, Sep 10, 2015 at 10:35 PM, Purshotam Shah < >>> [email protected]> wrote: >>> >>>> Thanks Akshay for putting patch.. I will review it. >>>> >>>> From: Akshay Rai <[email protected]> >>>> To: [email protected] >>>> Sent: Thursday, September 10, 2015 9:07 AM >>>> Subject: Request for a review - OOZIE-2346 >>>> >>>> Hi Oozie Committers, >>>> >>>> I have uploaded a patch to OOZIE-2346 and the test-build has run with >>>> one >>>> error. I don't think the error is related to the patch. >>>> >>>> The patch adds information of sub-workflows and the job console url >>>> along >>>> with other information in *oozie.job.info <http://oozie.job.info> *to >>>> the >>>> job conf*. *This will be injected only when >>>> *oozie.action.jobinfo.enable* >>>> is enabled in oozie-site.xml. >>>> >>>> Could I get a review for it? >>>> >>>> JIRA: https://issues.apache.org/jira/browse/OOZIE-2346 >>>> RB: https://reviews.apache.org/r/38088/ >>>> >>>> Regards, >>>> Akshay >>>> >>>> >>>> >>>> >>> >>> >> >
