Petals ESB Container

The NIO Transporter does not send back exception if an error happens while delivering the message on the receiving side

Details

  • Type: Improvement Request Improvement Request
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 5.0.0
  • Fix Version/s: 5.0.1
  • Component/s: Transporter
  • Security Level: Public
  • Description:
    Hide

    To improve reliability, it would be better if the NIO transporter would send back to the sender the error that happened on the receiving side (for example a component queue without space) so that the sender can try other endpoints if there is some.

    This is part of the effort to improve reliability in Petals: PETALSDISTRIB-146

    Show
    To improve reliability, it would be better if the NIO transporter would send back to the sender the error that happened on the receiving side (for example a component queue without space) so that the sender can try other endpoints if there is some. This is part of the effort to improve reliability in Petals: PETALSDISTRIB-146
  • Environment:
    -

Issue Links

Activity

Victor NOËL made changes - Fri, 27 Nov 2015 - 09:53:17 +0100
Field Original Value New Value
Status New [ 10000 ] Open [ 10002 ]
Priority Major [ 3 ]
Assignee Christophe DENEUX [ cdeneux ] Victor NOËL [ vnoel ]
Victor NOËL made changes - Fri, 27 Nov 2015 - 09:53:27 +0100
Link This issue blocks PETALSDISTRIB-146 [ PETALSDISTRIB-146 ]
Victor NOËL made changes - Fri, 8 Jan 2016 - 15:57:23 +0100
Status Open [ 10002 ] In Progress [ 10003 ]
Hide
Victor NOËL added a comment - Mon, 11 Jan 2016 - 15:02:43 +0100

Fixed in trunk, the error is sent back as an exchange through the router and not as a thrown exception because the NIO transporter is not built to send back an exception.

It is not really a problem, but it means the behaviour is a bit different than in local mode.
The main idea is that catching an exception from a send or receiving back an error in the exchange from a send should be considered equivalent for the sender.

Show
Victor NOËL added a comment - Mon, 11 Jan 2016 - 15:02:43 +0100 Fixed in trunk, the error is sent back as an exchange through the router and not as a thrown exception because the NIO transporter is not built to send back an exception. It is not really a problem, but it means the behaviour is a bit different than in local mode. The main idea is that catching an exception from a send or receiving back an error in the exchange from a send should be considered equivalent for the sender.
Victor NOËL made changes - Mon, 11 Jan 2016 - 15:02:43 +0100
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 5.0.1 [ 10579 ]
Resolution Fixed [ 1 ]
Hide
Victor NOËL added a comment - Tue, 12 Jan 2016 - 10:40:39 +0100

Reopen: it is not totally acceptable to send back an error, it would be best that an exception is sent back to the sender node so that its router can re-route the exchange like it does when a local delivery fails.

Show
Victor NOËL added a comment - Tue, 12 Jan 2016 - 10:40:39 +0100 Reopen: it is not totally acceptable to send back an error, it would be best that an exception is sent back to the sender node so that its router can re-route the exchange like it does when a local delivery fails.
Victor NOËL made changes - Tue, 12 Jan 2016 - 10:40:39 +0100
Status Resolved [ 10004 ] Open [ 10002 ]
Resolution Fixed [ 1 ]
Christophe DENEUX made changes - Mon, 23 May 2016 - 15:12:30 +0200
Fix Version/s 5.0.2 [ 10661 ]
Fix Version/s 5.0.1 [ 10579 ]
Christophe DENEUX made changes - Mon, 23 May 2016 - 15:12:55 +0200
Fix Version/s 5.0.1 [ 10579 ]
Victor NOËL made changes - Wed, 14 Sep 2016 - 10:14:16 +0200
Link This issue blocks PETALSESBCONT-438 [ PETALSESBCONT-438 ]
Victor NOËL made changes - Wed, 14 Sep 2016 - 10:14:43 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Hide
Victor NOËL added a comment - Wed, 14 Sep 2016 - 10:15:30 +0200

The extra requirements that the remote transporter should allow for retry to happen on the sender side has been moved to PETALSESBCONT-438.

Show
Victor NOËL added a comment - Wed, 14 Sep 2016 - 10:15:30 +0200 The extra requirements that the remote transporter should allow for retry to happen on the sender side has been moved to PETALSESBCONT-438.
Victor NOËL made changes - Wed, 14 Sep 2016 - 10:15:30 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 5.0.2 [ 10661 ]
Resolution Fixed [ 1 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
11s
1
Victor NOËL
Fri, 27 Nov 2015 - 09:53:17 +0100
Open Open In Progress In Progress
42d 6h 4m
1
Victor NOËL
Fri, 8 Jan 2016 - 15:57:23 +0100
In Progress In Progress Resolved Resolved
2d 23h 5m
1
Victor NOËL
Mon, 11 Jan 2016 - 15:02:43 +0100
Resolved Resolved Open Open
19h 37m
1
Victor NOËL
Tue, 12 Jan 2016 - 10:40:39 +0100
Open Open In Progress In Progress
245d 22h 34m
1
Victor NOËL
Wed, 14 Sep 2016 - 10:14:43 +0200
In Progress In Progress Resolved Resolved
47s
1
Victor NOËL
Wed, 14 Sep 2016 - 10:15:30 +0200



People

Dates

  • Created:
    Fri, 27 Nov 2015 - 09:53:06 +0100
    Updated:
    Wed, 14 Sep 2016 - 10:15:30 +0200
    Resolved:
    Wed, 14 Sep 2016 - 10:15:30 +0200