introduced a well-deserved HQL execution time log, which is good.
Unfortunately, (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):
May I also suggest that we change this particular logger from whichever implementation class du jour to a org.hibernate.performance logger.
Not going to get into magic string based logger names anymore.
I understand you want to avoid magic strings.
How about logging to the interface (org.hibernate.stat.Statistics) instead of the implementation for this particular log? This way, you will be able to swap the implementation (which the user shouldn't care about) and keep the public information with no change on the client side.