[hibernate-issues] [Hibernate-JIRA] Updated: (HHH-5469) HHH-3659 is only half done, due to HHH-4989 (i.e. no HQL performance log when running Java 5)

[Steve Ebersole (JIRA)]

     [ http://opensource.atlassian.com/projects/hibernate/browse/HHH-5469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Steve Ebersole updated HHH-5469:
--------------------------------

             Assignee: Steve Ebersole
    Affects Version/s:     (was: 3.6.0.Beta2)
        Fix Version/s: 3.6.0.Beta3
                       3.5.5

Not going to get into magic string based logger names anymore.  

> HHH-3659 is only half done, due to HHH-4989 (i.e. no HQL performance log when running Java 5)
> ---------------------------------------------------------------------------------------------
>
>                 Key: HHH-5469
>                 URL: http://opensource.atlassian.com/projects/hibernate/browse/HHH-5469
>             Project: Hibernate Core
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 3.5.4, 3.6.0.Beta1
>            Reporter: Cédrik LIME
>            Assignee: Steve Ebersole
>            Priority: Trivial
>             Fix For: 3.5.5, 3.6.0.Beta3
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> HHH-3659 introduced a well-deserved HQL execution time log, which is good.
> Unfortunately, HHH-4989 (Java 5 concurrent for statistics) didn't pick up those changes, so no execution time logs when running Java 5.
> To fix, add the same log statement to {{ConcurrentStatisticsImpl#queryExecuted(String hql, int rows, long time)}}:
> {code:java}
> log.info("HQL: {}, time: {}ms, rows: {}", new Object[]{hql, new Long(time), Long.valueOf(rows)});
> {code}
> May I also suggest that we change this particular logger from whichever implementation class du jour to a {{org.hibernate.performance}} logger.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/hibernate/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira