[jbossts-issues] [JBoss JIRA] (JBTM-2207) Make default name null in order for quick lookup in BeanPopulator

[Tom Jenkinson (JIRA)]

    [ https://issues.jboss.org/browse/JBTM-2207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983091#comment-12983091 ] 

Tom Jenkinson commented on JBTM-2207:
-------------------------------------

Looking at the diff I would have no objection to merging this, but please can you provide more details in the description of what the performance win is that you are seeing and also a rationale for the reason to make the change so we can audit the modification. For example, it looks from the diff as if the main reason is to avoid a string comparison per lookup of the bean name. I believe we only lookup the beans once during initialization. That being said if there is something else please do let me know. As always, seeing some performance numbers before and after would help too if you have them available.

Thanks,
Tom

> Make default name null in order for quick lookup in BeanPopulator
> -----------------------------------------------------------------
>
>                 Key: JBTM-2207
>                 URL: https://issues.jboss.org/browse/JBTM-2207
>             Project: JBoss Transaction Manager
>          Issue Type: Sub-task
>      Security Level: Public(Everyone can see) 
>          Components: Transaction Core
>            Reporter: Jesper Pedersen
>            Assignee: Michael Musgrove
>




--
This message was sent by Atlassian JIRA
(v6.2.6#6264)