[ http://opensource.atlassian.com/projects/hibernate/browse/HHH-766?page=co... ] Jacob Poder Kristensen commented on HHH-766: -------------------------------------------- Even though it might be a bit controversial to use a huge number of constants in the WHERE IN clause, the complexity of the used utility, StringHelper. firstIndexOfChar(String sqlString, String string, int startindex), is unnecessarily high. With the current implementation, 'sqlString' is scanned for all of the characters in 'string', even if a match is found early: public static int firstIndexOfChar(String sqlString, String string, int startindex) { int matchAt = -1; for ( int i = 0; i < string.length(); i++ ) { int curMatch = sqlString.indexOf( string.charAt( i ), startindex ); if ( curMatch >= 0 ) { if ( matchAt == -1 ) { // first time we find match! matchAt = curMatch; } else { matchAt = Math.min( matchAt, curMatch ); } } } return matchAt; } Matching one character at a time from 'sqlString' against all those in 'string', has a much better chance for early exit: /** * Starting from <code>startindex</code>, find the first index in <code>sqlString</code> * where any one of the characters in <code>string</code> matches. * @param sqlString the String to look in. * @param string the characters to find one of. * @param startindex the zero based string index to start looking from. * @return the index of the first matching character or -1 if not found. */ public static int firstIndexOfChar(final String sqlString, final String string, final int startindex) { final int length = string.length(); final char[] chars = new char[length]; string.getChars(0, length, chars, 0); for ( int i = startindex; i < sqlString.length(); ++i ) { char c = sqlString.charAt(i); for ( int j = 0; j < length; ++j ) { if(c == chars[j]) { return i; } } } return -1; } So, even if you are happy with the performance of setParameterList, the above is an improvement to firstIndexOfChar in general. -- 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.... - For more information on JIRA, see: http://www.atlassian.com/software/jira