I think I am missing something. If you are saying that we have no way to notify CQ listeners if the notification message failed, how do you plan to notify them about the failure? What if the failure message also fails?
On Sun, Nov 6, 2016 at 11:45 PM, Anton Vinogradov <a...@apache.org> wrote: > Dmitriy, > > In cases where ContinuousQuery Event unmarshalling failed we have to inform > node's owner somehow. > Failed CQ event means you'll just get no event, no exception, and you'll > have no need to read logs. > Currently, we have no possibility to inform user except writing error > message to logs. > > So, we have two ways to inform user: 1) to extend CQ API with new method > listentFailedEvents() 2) or to let user listen for UnhandledExceptionEvent > First case will break old sources using Ignite CQ in case method will be > required, and gives no profit in case it will be optional. > Second case will allow to listen such events or/and to see them at some > management tool (eg. Visor). > > So, current solution is to log such failures, write them to > ExceptionRegistry and inform user using UnhandledExceptionEvent (via > listener or management console). > > On Fri, Nov 4, 2016 at 6:34 AM, Dmitriy Setrakyan <dsetrak...@apache.org> > wrote: > > > Dmitriy, I am not sure how a public even would help fixing internal error > > handling. Also, the ticket has too many comments which makes it difficult > > to understand. Any chance you could provide the final proposal here? > > > > On Thu, Nov 3, 2016 at 8:01 AM, Dmitriy Govorukhin < > > dmitriy.govoruk...@gmail.com> wrote: > > > > > HI all, I think what we need add new event for handling > > > unhandled exception. We have some exception which not handled, new > > > event (UnhandledExceptionEvent) can > > > help know if was unhandled exception. Let's discuss. More details about > > > reason IGNITE-2079 <https://issues.apache.org/jira/browse/IGNITE-2079> > > > > > >
