Skip to content

WIP: Remove an object allocation from instrumentation decorators which interact with fields. - #2742

Closed
treverj wants to merge 2 commits into
graphql-java:masterfrom
treverj:reduce-instrumentation-allocations
Closed

treverj wants to merge 2 commits into
graphql-java:masterfrom
treverj:reduce-instrumentation-allocations

Conversation

@treverj

@treverj treverj commented Feb 25, 2022 •

Copy link
Copy Markdown

See discussion here for more info, this is intended as a POC

Comment thread src/main/java/graphql/execution/ExecutionStrategy.java Outdated
@bbakerman bbakerman added the Not to be merged spikes or other stuff that should never or not yet to be merged label Feb 28, 2022
@treverj

treverj commented Mar 1, 2022

Copy link
Copy Markdown
Author

Apologies, this PR has now been fixed to just include the correct changes. Sorry for the confusion.

@bbakerman

Copy link
Copy Markdown
Member

This has been addressed in another PR

@bbakerman bbakerman closed this May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Not to be merged spikes or other stuff that should never or not yet to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants