Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: resolve interceptors parameter converters during build #212

Conversation

jlabedo
Copy link
Contributor

@jlabedo jlabedo commented Sep 27, 2023

No description provided.

@jlabedo jlabedo changed the title interceptors: resolve parameters converters during build refactor: resolve interceptors parameter converters during build Sep 27, 2023
continue;
} elseif ($parameter->getTypeDescriptor()->isNonCollectionArray()) {
$builtConverters[] = AllHeadersBuilder::createWith($parameter->getName())->build($referenceSearchService);
continue;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will simplify the code, yet the performance will stay the same, as build method is executed on run time.
Whatever has ReferenceSearchService $referenceSearchService, means it has to work booted container.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or maybe it will help, if given class is executed more than once. As we will then reuse constructed parameters

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you planning to create parameter converter builder during the object construction as follow up?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes I know :) the plan is to do all this work before something is serialized for caching, but I am not sure at that time of how to do it. My goal is to try something wich would work with incremental refactorings to demonstrate the performance gain.
And whatever you cache, the core messaging components should have low construction cost to show the performance boost.

@dgafka dgafka merged commit 986a9cc into ecotoneframework:main Sep 27, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants