let's not abuse Objects.toString to handle null

Author: gavinking

hibernate-core/src/main/java/org/hibernate/dialect/Dialect.java

@@ -228,6 +228,7 @@
 import static org.hibernate.internal.util.MathHelper.ceilingPowerOfTwo;
 import static org.hibernate.internal.util.StringHelper.isBlank;
 import static org.hibernate.internal.util.StringHelper.isEmpty;
+import static org.hibernate.internal.util.StringHelper.isNotEmpty;
 import static org.hibernate.internal.util.StringHelper.splitAtCommas;
 import static org.hibernate.internal.util.collections.ArrayHelper.EMPTY_STRING_ARRAY;
 import static org.hibernate.sql.ast.internal.NonLockingClauseStrategy.NON_CLAUSE_STRATEGY;
@@ -5222,9 +5223,12 @@ public static String addUseIndexQueryHint(String query, String hints) {
 		if ( matcher.matches() && matcher.groupCount() > 1 ) {
 			final String startToken = matcher.group(1);
 			// Null if there is no join in the query
-			final String joinToken = Objects.toString( matcher.group(2), "" );
+			final String joinToken = matcher.group(2);
 			final String endToken = matcher.group(3);
-			return startToken + " use index (" + hints + ") " + joinToken + endToken;
+			return startToken
+					+ " use index (" + hints + ") "
+					+ ( joinToken == null ? "" : joinToken )
+					+ endToken;
 		}
 		else {
 			return query;
@@ -5242,7 +5246,7 @@ protected String prependComment(String sql, String comment) {
 	 * Perform necessary character escaping on the text of the comment.
 	 */
 	public static String escapeComment(String comment) {
-		if ( StringHelper.isNotEmpty( comment ) ) {
+		if ( isNotEmpty( comment ) ) {
 			final String escaped = ESCAPE_CLOSING_COMMENT_PATTERN.matcher( comment ).replaceAll( "*\\\\/" );
 			return ESCAPE_OPENING_COMMENT_PATTERN.matcher( escaped ).replaceAll( "/\\\\*" );
 		}