XStream

AttributeMapper should use ReflectionProvider for looking up fields

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.3.1
  • Fix Version/s: 1.4
  • Component/s: Core
  • Labels:
    None

Description

Currently AttributeMapper uses Fields.find to lookup Fields. This is problematic because Fields.find is not cached like ReflectionProvider/FieldDictionary. This should be changed to maximize performance.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Keith Kowalczykowski added a comment -

Patch to fix this issue. Seeing ~15% speedup.

Show
Keith Kowalczykowski added a comment - Patch to fix this issue. Seeing ~15% speedup.
Hide
Permalink
Jörg Schaible added a comment -

Thanks for this, Keith. Applied to HEAD.

Show
Jörg Schaible added a comment - Thanks for this, Keith. Applied to HEAD.

People

  • Assignee:
    Jörg Schaible
    Reporter:
    Keith Kowalczykowski
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: