Skip to content
This repository has been archived by the owner on Apr 5, 2022. It is now read-only.

[Sprint: 47] XD-2911 - Improved performance of Tuple Creation #1560

Closed
wants to merge 1 commit into from

Conversation

mminella
Copy link
Contributor

NOTE - This is a breaking change.

This commit refactors how the conversion service used within a Tuple is
created to a more performant option. Prior to this commit, for each
instance of TupleBuilder created (and under typical usage, each Tuple)
an instance of DefaultTupleConversionService was created. This updated
version changes the default to be a static instance and changes the
recommended approach for customization to be a
ConfigurableConversionService be configured as a singleton within your
module's context and then used across all Tuple instances. This
provides better performance in the creation of Tuples.

This commit refactors how the conversion service used within a Tuple is
created to a more performant option.  Prior to this commit, for each
instance of TupleBuilder created (and under typical usage, each Tuple)
an instance of DefaultTupleConversionService was created.  This updated
version changes the default to be a static instance and changes the
recommended approach for customization to be a
ConfigurableConversionService be configured as a singleton within your
module's context and then used across all Tuple instances.  This
provides better performance in the creation of Tuples.
@dturanski
Copy link
Contributor

I will merge as is with the performance tests but I would like to move them into spring-xd-benchmark as soon as #1558 which introduces that project is merged. This is intended to provide similar tests and separate them from the normal build.

@dturanski
Copy link
Contributor

rebased and merged

@dturanski dturanski closed this Apr 20, 2015
@sabbyanandan sabbyanandan changed the title XD-2911 - Improved performance of Tuple Creation [Sprint: 47] XD-2911 - Improved performance of Tuple Creation Apr 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants