Petals SE Notification

Performance issues on XML Notification processing (read and write)

Details

  • Type: Improvement Request Improvement Request
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 1.0.1
  • Fix Version/s: 1.0.3
  • Component/s: None
  • Security Level: Public
  • Description:
    Hide

    This part of the noticiation processing is done with WS Star notification.
    A profiling of the notification process shows that some improvements can be done on the read and write processes (Dom, JDom, Jaxb conversion for exemple, factory creation and so on).

    Show
    This part of the noticiation processing is done with WS Star notification. A profiling of the notification process shows that some improvements can be done on the read and write processes (Dom, JDom, Jaxb conversion for exemple, factory creation and so on).
  • Environment:
    petals-se-notification 1.0.1, ws-notification-* 1.0
  1. unmarshall-source.patch
    (2 kB)
    Christophe DENEUX
    Mon, 6 Sep 2010 - 19:42:52 +0200

Issue Links

Activity

Olivier Fabre made changes - Mon, 6 Sep 2010 - 17:40:21 +0200
Field Original Value New Value
Assignee Roland Naudin [ rnaudin ] Olivier Fabre [ ofabre ]
Olivier Fabre made changes - Mon, 6 Sep 2010 - 17:40:40 +0200
Status New [ 10000 ] Open [ 10002 ]
Priority Critical [ 2 ]
Olivier Fabre made changes - Mon, 6 Sep 2010 - 17:41:08 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Olivier Fabre made changes - Mon, 6 Sep 2010 - 17:45:10 +0200
Link This issue blocks PETALSSEKPI-14 [ PETALSSEKPI-14 ]
Hide
Christophe DENEUX added a comment - Mon, 6 Sep 2010 - 19:31:15 +0200

A first optimization is to used only one unmarshaller instead of created one when unmarshalling is needed. Caution to the concurrency on unmarshalling.

Show
Christophe DENEUX added a comment - Mon, 6 Sep 2010 - 19:31:15 +0200 A first optimization is to used only one unmarshaller instead of created one when unmarshalling is needed. Caution to the concurrency on unmarshalling.
Hide
Christophe DENEUX added a comment - Mon, 6 Sep 2010 - 19:42:52 +0200 - edited

An other optimization could be to deprecate the API using DOM Document and replace it by an API using a Source. See the attachment 'unmarshall-source.patch'

Show
Christophe DENEUX added a comment - Mon, 6 Sep 2010 - 19:42:52 +0200 - edited An other optimization could be to deprecate the API using DOM Document and replace it by an API using a Source. See the attachment 'unmarshall-source.patch'
Christophe DENEUX made changes - Mon, 6 Sep 2010 - 19:42:52 +0200
Attachment unmarshall-source.patch [ 10274 ]
Hide
Christophe DENEUX added a comment - Mon, 6 Sep 2010 - 20:37:01 +0200

Caution don't deprecate the DOM API because using a API based on Source generates a XML parsing. So if on the client side the XML is already parsed as DOM document, use the DOM API. Otherwise, if the XML is not parsed, use the Source API. Please add a note on the usage of each API.

Show
Christophe DENEUX added a comment - Mon, 6 Sep 2010 - 20:37:01 +0200 Caution don't deprecate the DOM API because using a API based on Source generates a XML parsing. So if on the client side the XML is already parsed as DOM document, use the DOM API. Otherwise, if the XML is not parsed, use the Source API. Please add a note on the usage of each API.
Olivier Fabre made changes - Fri, 10 Sep 2010 - 16:23:39 +0200
Link This issue depends on WSNOTIF-3 [ WSNOTIF-3 ]
Olivier Fabre made changes - Fri, 10 Sep 2010 - 16:23:58 +0200
Link This issue depends on WSNOTIF-2 [ WSNOTIF-2 ]
Olivier Fabre made changes - Fri, 10 Sep 2010 - 16:24:15 +0200
Link This issue depends on WSNOTIF-1 [ WSNOTIF-1 ]
Olivier Fabre made changes - Mon, 20 Sep 2010 - 15:54:10 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 1.0.3 [ 10119 ]
Resolution Fixed [ 1 ]
Olivier Fabre made changes - Tue, 2 Nov 2010 - 17:25:04 +0100
Status Resolved [ 10004 ] Closed [ 10005 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
29s
1
Olivier Fabre
Mon, 6 Sep 2010 - 17:40:40 +0200
Open Open In Progress In Progress
28s
1
Olivier Fabre
Mon, 6 Sep 2010 - 17:41:08 +0200
In Progress In Progress Resolved Resolved
13d 22h 13m
1
Olivier Fabre
Mon, 20 Sep 2010 - 15:54:10 +0200
Resolved Resolved Closed Closed
43d 2h 30m
1
Olivier Fabre
Tue, 2 Nov 2010 - 17:25:04 +0100



People

Dates

  • Created:
    Mon, 6 Sep 2010 - 17:40:11 +0200
    Updated:
    Tue, 2 Nov 2010 - 17:25:04 +0100
    Resolved:
    Mon, 20 Sep 2010 - 15:54:10 +0200