Skip to content

Commit b089426

Browse files
committed
HHH-13547 - remove logging of BatchImpl rethrown exceptions
Signed-off-by: Jan Schatteman <[email protected]>
1 parent cf626df commit b089426

File tree

3 files changed

+5
-10
lines changed

3 files changed

+5
-10
lines changed

hibernate-core/src/main/java/org/hibernate/engine/jdbc/batch/JdbcBatchLogging.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515

1616
import java.lang.invoke.MethodHandles;
1717

18-
import static org.jboss.logging.Logger.Level.ERROR;
1918
import static org.jboss.logging.Logger.Level.INFO;
19+
import static org.jboss.logging.Logger.Level.WARN;
2020

2121
/**
2222
* Sub-system logging related to JDBC batch execution
@@ -35,11 +35,7 @@ public interface JdbcBatchLogging extends BasicLogger {
3535
Logger BATCH_LOGGER = Logger.getLogger( NAME );
3636
JdbcBatchLogging BATCH_MESSAGE_LOGGER = Logger.getMessageLogger( MethodHandles.lookup(), JdbcBatchLogging.class, NAME );
3737

38-
@LogMessage(level = ERROR)
39-
@Message(id = 100501, value = "Exception executing batch [%s], SQL: %s")
40-
void unableToExecuteBatch(Exception e, String sql );
41-
42-
@LogMessage(level = ERROR)
38+
@LogMessage(level = WARN)
4339
@Message(id = 100502, value = "Unable to release batch statement...")
4440
void unableToReleaseBatchStatement();
4541

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import org.hibernate.engine.jdbc.spi.SqlStatementLogger;
2424
import org.hibernate.event.spi.EventManager;
2525
import org.hibernate.event.spi.HibernateMonitoringEvent;
26+
import org.hibernate.jdbc.BatchFailedException;
2627
import org.hibernate.resource.jdbc.spi.JdbcEventHandler;
2728
import org.hibernate.resource.jdbc.spi.JdbcSessionOwner;
2829

@@ -302,13 +303,11 @@ protected void performExecution() {
302303
}
303304
catch (SQLException e) {
304305
abortBatch( e );
305-
BATCH_MESSAGE_LOGGER.unableToExecuteBatch( e, sql );
306306
throw sqlExceptionHelper.convert( e, "could not execute batch", sql );
307307
}
308308
catch (RuntimeException re) {
309309
abortBatch( re );
310-
BATCH_MESSAGE_LOGGER.unableToExecuteBatch( re, sql );
311-
throw re;
310+
throw new BatchFailedException( "An exception occurred while executing: " + sql, re);
312311
}
313312
} );
314313
}

hibernate-core/src/main/java/org/hibernate/internal/CoreMessageLogger.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -483,7 +483,7 @@ void synchronizationFailed(
483483
void unableToDeserializeCache(String path, SerializationException error);
484484

485485
/**
486-
* @deprecated Use {@link org.hibernate.engine.jdbc.batch.JdbcBatchLogging#unableToExecuteBatch} instead
486+
* @deprecated
487487
*/
488488
@LogMessage(level = ERROR)
489489
@Message(value = "Exception executing batch [%s], SQL: %s", id = 315)

0 commit comments

Comments
 (0)