Petals Distribution

Remove dependencies to Saxon and to easycommons-utils from petals-log

Details

  • Type: Task Task
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 4.3.0-beta-1
  • Fix Version/s: 5.0.0-M1
  • Component/s: Tools
  • Security Level: Public
  • Description:
    Hide

    petals-log depends on easycommons-utils which have a static pool of Transformers used all across petals and the components.

    petals-log is present in the system classloader so the same pool is used everywhere even though components could desire to use another transformer implementation than the default one (Saxon).

    Hence, we should remove the dependency to easycommons-utils and Saxon-HE from petals-log.

    To replace the pool, we will use for now a threadlocal Transformer.
    This means one Transformer per thread to prevent concurrent access as well as permit each thread (and thus each component) to have its desired implementation.
    This means a small impact on performance during thread creation but in components threads are reused and the transformer implementation doesn't need to change so it's ok.

    Show
    petals-log depends on easycommons-utils which have a static pool of Transformers used all across petals and the components. petals-log is present in the system classloader so the same pool is used everywhere even though components could desire to use another transformer implementation than the default one (Saxon). Hence, we should remove the dependency to easycommons-utils and Saxon-HE from petals-log. To replace the pool, we will use for now a threadlocal Transformer. This means one Transformer per thread to prevent concurrent access as well as permit each thread (and thus each component) to have its desired implementation. This means a small impact on performance during thread creation but in components threads are reused and the transformer implementation doesn't need to change so it's ok.
  • Environment:
    -

Issue Links

Activity

Victor NOËL made changes - Fri, 3 Jul 2015 - 13:31:24 +0200
Field Original Value New Value
Link This issue blocks PETALSESBCONT-323 [ PETALSESBCONT-323 ]
Victor NOËL made changes - Fri, 3 Jul 2015 - 13:32:19 +0200
Link This issue blocks PETALSESBCONT-341 [ PETALSESBCONT-341 ]
Victor NOËL made changes - Fri, 3 Jul 2015 - 13:32:52 +0200
Status New [ 10000 ] Open [ 10002 ]
Priority Major [ 3 ]
Component/s Tools [ 10271 ]
Victor NOËL made changes - Fri, 3 Jul 2015 - 13:32:54 +0200
Assignee Christophe DENEUX [ cdeneux ] Victor NOËL [ vnoel ]
Victor NOËL made changes - Fri, 3 Jul 2015 - 13:32:58 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Victor NOËL made changes - Fri, 3 Jul 2015 - 16:40:19 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 5.0.0 [ 10412 ]
Resolution Fixed [ 1 ]
Victor NOËL made changes - Fri, 3 Jul 2015 - 16:45:43 +0200
Status Resolved [ 10004 ] Open [ 10002 ]
Resolution Fixed [ 1 ]
Victor NOËL made changes - Fri, 3 Jul 2015 - 16:54:22 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Victor NOËL made changes - Mon, 6 Jul 2015 - 16:44:38 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Resolution Fixed [ 1 ]
Victor NOËL made changes - Tue, 7 Jul 2015 - 12:09:26 +0200
Status Resolved [ 10004 ] Open [ 10002 ]
Resolution Fixed [ 1 ]
Victor NOËL made changes - Wed, 8 Jul 2015 - 14:01:24 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Victor NOËL made changes - Fri, 10 Jul 2015 - 13:39:52 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Resolution Fixed [ 1 ]
Christophe DENEUX made changes - Tue, 6 Oct 2015 - 11:12:39 +0200
Link This issue blocks PETALSSETAL-15 [ PETALSSETAL-15 ]

People

Dates

  • Created:
    Fri, 3 Jul 2015 - 13:30:56 +0200
    Updated:
    Tue, 6 Oct 2015 - 11:12:39 +0200
    Resolved:
    Fri, 10 Jul 2015 - 13:39:51 +0200