note down an important TODO for later

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

@@ -66,6 +66,10 @@ public class H2SqlAstTranslator<T extends JdbcOperation> extends AbstractSqlAstT
 		assert tableInsert.getReturningColumns() != null
 				&& !tableInsert.getReturningColumns().isEmpty();
 
+		//TODO: This is a terrible way to solve this problem, please fix it!
+		//      Not every "returning insert" statement has something to do
+		//      with identity columns! (Nor is it an elegant implementation.)
+
 		final H2IdentityColumnSupport  identitySupport = (H2IdentityColumnSupport) getSessionFactory()
 				.getJdbcServices()
 				.getDialect()

hibernate-core/src/main/java/org/hibernate/dialect/identity/H2FinalTableIdentityColumnSupport.java

@@ -34,6 +34,7 @@ public class H2FinalTableIdentityColumnSupport extends H2IdentityColumnSupport {
 		return "select " + identityColumnName + " from final table ( " + insertString + " )";
 	}
 
+	@Deprecated //TODO: move this logic back to H2SqlAstTranslator
 	public void render(
 			TableInsert tableInsert,
 			Consumer<String> sqlAppender,

hibernate-core/src/main/java/org/hibernate/dialect/identity/H2IdentityColumnSupport.java

@@ -48,6 +48,7 @@ public class H2IdentityColumnSupport extends IdentityColumnSupportImpl {
 		void renderInsertValues();
 	}
 
+	@Deprecated //TODO: get rid of this layer-breaking method!
 	public void render(
 			TableInsert tableInsert,
 			Consumer<String> sqlAppender,