Github user pallavi-rao commented on a diff in the pull request:

    https://github.com/apache/falcon/pull/17#discussion_r51244424
  
    --- Diff: 
rerun/src/main/java/org/apache/falcon/rerun/handler/RetryHandler.java ---
    @@ -75,6 +76,9 @@ public void handleRerun(String clusterName, String 
entityType, String entityName
                             "All retry attempt failed out of configured: "
                                     + attempts + " attempt for entity 
instance::");
                 }
    +        } catch (EntityNotRegisteredException ee) {
    --- End diff --
    
    Most other listeners other than the ones under rerun seem to just read off 
the context and not really query the config store for the entity for which 
notification is generated. So, nothing needs to be fixed immediately.
    
    Agree that we have to have cleaner solution.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to