On Mon, Feb 4, 2013 at 10:30 PM, Reinhard Nägele <
[email protected]> wrote:

> Remove the @Inject annotations from the fields. You are combining field
> and constructor injection thus making Guice do the injection twice.


I find it hard to believe that this would make a significant difference
since Guice has to look up these fields anyway (if only to find out whether
they have an @Inject annotation).

Just speculating though, happy to be proven wrong and to learn something
about Guice along the way.

Satish: you didn't say exactly where YourKit was saying that the spike was
coming from, do you have more specific results to share?

-- 
Cédric

-- 
You received this message because you are subscribed to the Google Groups 
"google-guice" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/google-guice?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to