HHH-19005 Memory leak modify String to StringBuilder

Author: dev3way

hibernate-core/src/main/java/org/hibernate/engine/jdbc/internal/BasicFormatterImpl.java

@@ -70,38 +70,44 @@ public String perform() {
 			while ( tokens.hasMoreTokens() ) {
 				token = tokens.nextToken();
 
-				if ( "-".equals(token) && result.toString().endsWith("-") ) {
+				if ( "-".equals( token ) && result.toString().endsWith( "-" ) ) {
 					do {
 						result.append( token );
 						token = tokens.nextToken();
 					}
 					while ( !"\n".equals( token ) && tokens.hasMoreTokens() );
 				}
 
-				lcToken = token.toLowerCase(Locale.ROOT);
-				switch (lcToken) {
+				lcToken = token.toLowerCase( Locale.ROOT );
+				switch ( lcToken ) {
 
 					case "'":
 					case "`":
 					case "\"":
 						String t;
+						StringBuilder sb = new StringBuilder();
+						sb.append( this.token );
 						do {
 							t = tokens.nextToken();
-							token += t;
+							sb.append( t );
 						}
 						while ( !lcToken.equals( t ) && tokens.hasMoreTokens() );
+						this.token = sb.toString();
 						lcToken = token;
 						misc();
 						break;
 					// SQL Server uses "[" and "]" to escape reserved words
 					// see SQLServerDialect.openQuote and SQLServerDialect.closeQuote
 					case "[":
 						String tt;
+						StringBuilder sb2 = new StringBuilder();
+						sb2.append( this.token );
 						do {
 							tt = tokens.nextToken();
-							token += tt;
+							sb2.append( tt );
 						}
 						while ( !"]".equals( tt ) && tokens.hasMoreTokens() );
+						this.token = sb2.toString();
 						lcToken = token;
 						misc();
 						break;
@@ -238,7 +244,7 @@ private void from() {
 		}
 
 		private void comma() {
-			if ( afterByOrSetOrFromOrSelect && inFunction==0 ) {
+			if ( afterByOrSetOrFromOrSelect && inFunction == 0 ) {
 				commaAfterByOrFromOrSelect();
 			}
 //			else if ( afterOn && inFunction==0 ) {
@@ -313,7 +319,7 @@ private void beginCase() {
 
 		private void misc() {
 			out();
-			if ( afterInsert && inFunction==0 ) {
+			if ( afterInsert && inFunction == 0 ) {
 				newline();
 				afterInsert = false;
 			}
@@ -329,7 +335,7 @@ private void white() {
 		}
 
 		private void updateOrInsertOrDelete() {
-			if ( indent>1  ) {
+			if ( indent > 1 ) {
 				//probably just the insert SQL function
 				out();
 			}
@@ -367,7 +373,7 @@ private void select() {
 
 		private void out() {
 			if ( result.charAt( result.length() - 1 ) == ',' ) {
-				result.append(" ");
+				result.append( " " );
 			}
 			result.append( token );
 		}
@@ -507,7 +513,7 @@ private static boolean isWhitespace(String token) {
 
 		private void newline() {
 			result.append( System.lineSeparator() )
-					.append( INDENT_STRING.repeat(indent) );
+					.append( INDENT_STRING.repeat( indent ) );
 			beginLine = true;
 		}
 	}

hibernate-core/src/test/java/org/hibernate/orm/test/jdbc/util/BasicFormatterTest.java

@@ -21,7 +21,6 @@
  * @author Steve Ebersole
  */
 public class BasicFormatterTest extends BaseUnitTestCase {
-
 	@Test
 	public void testNoLoss() {
 		assertNoLoss( "insert into Address (city, state, zip, \"from\") values (?, ?, ?, 'insert value')" );
@@ -47,6 +46,30 @@ public void testNoLoss() {
 				"(select p.pid from Address where city = 'Boston') union (select p.pid from Address where city = 'Taipei')"
 		);
 		assertNoLoss( "select group0.[order] as order0 from [Group] group0 where group0.[order]=?1" );
+		assertNoLoss( """
+						INSERT INTO TEST_TABLE (JSON) VALUES
+						('{
+						"apiVersion": "2.0",
+						"data": {
+							"updated": "2010-01-07T19:58:42.949Z",
+							"totalItems": 800,
+							"startIndex": 1,
+							"itemsPerPage": 1,
+							"items": [
+							{
+								"id": "hYB0mn5zh2c",
+								"uploaded": "2007-06-05T22:07:03.000Z",
+								"updated": "2010-01-07T13:26:50.000Z",
+								"uploader": "GoogleDeveloperDay",
+								"category": "News",
+								"title": "Google Developers Day US - Maps API Introduction",
+								"description": "Google Maps API Introduction ..."
+							}
+							]
+						}
+						}')
+					"""
+		);
 	}
 
 	@Test