falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ajay Yadava" <ajayn...@gmail.com>
Subject Re: Review Request 41885: EntityNotRegisteredException when process with no input/output feed is scheduled
Date Mon, 04 Jan 2016 17:40:58 GMT


> On Jan. 4, 2016, 3:46 p.m., Peeyush Bishnoi wrote:
> > prism/src/main/java/org/apache/falcon/handler/SLAMonitoringHandler.java, line 57
> > <https://reviews.apache.org/r/41885/diff/1/?file=1180907#file1180907line57>
> >
> >     Should not be here check for "IGNORE" as well along with check for "NONE".

We don't put IGNORE for output feed names, we put only "NONE". There are some places where
we put IGNORE and some places where we put "NONE" hence it is confusing. But for this particular
case it is "NONE". You can see the same in the stack trace of the error in the description
and also in the OozieProcessWorkflowBuilderTest#testProcessWithInputsNoOutputs.


- Ajay


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


On Jan. 4, 2016, 2:47 p.m., Ajay Yadava wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41885/
> -----------------------------------------------------------
> 
> (Updated Jan. 4, 2016, 2:47 p.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1714
>     https://issues.apache.org/jira/browse/FALCON-1714
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> EntityNotRegisteredException when process with no input/output feed is scheduled
> 
> 
> Diffs
> -----
> 
>   prism/src/main/java/org/apache/falcon/handler/SLAMonitoringHandler.java bb3b8e0 
> 
> Diff: https://reviews.apache.org/r/41885/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ajay Yadava
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message