chore: add workarounds for testing with Hibernate

pom.xml

@@ -4,7 +4,7 @@
  xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
  <modelVersion>4.0.0</modelVersion>
  <artifactId>google-cloud-spanner-jdbc</artifactId>
- <version>2.17.1-SNAPSHOT</version><!-- {x-version-update:google-cloud-spanner-jdbc:current} -->
+ <version>2.17.2-SNAPSHOT</version><!-- {x-version-update:google-cloud-spanner-jdbc:current} -->
  <packaging>jar</packaging>
  <name>Google Cloud Spanner JDBC</name>
  <url>https://github.com/googleapis/java-spanner-jdbc</url>

src/main/java/com/google/cloud/spanner/jdbc/AbstractJdbcStatement.java

@@ -57,6 +57,25 @@ public JdbcConnection getConnection() {
  return connection;
  }
 
+ String maybeAddWhereClause(String sql) {
+ String lowerCaseSql = sql.toLowerCase();
+ if (lowerCaseSql.startsWith("delete") || lowerCaseSql.startsWith("update")) {
+ if (!lowerCaseSql.contains(" where ")) {
+ return sql + " where true";
+ }
+ }
+ return sql;
+ }
+
+ boolean isAllowedDdlStatementInTransaction(String sql) throws SQLException {
+ if (!getConnection().getAutoCommit() && getConnection().getParser().isDdlStatement(sql)) {
+ if (!getConnection().isTransactionStarted()) {
+ return true;
+ }
+ }
+ return false;
+ }
+
  private Options.QueryOption[] getQueryOptions(QueryOption... options) throws SQLException {
  QueryOption[] res = options == null ? new QueryOption[0] : options;
  if (getFetchSize() > 0) {

src/main/java/com/google/cloud/spanner/jdbc/JdbcPreparedStatement.java

@@ -47,7 +47,7 @@ class JdbcPreparedStatement extends AbstractJdbcPreparedStatement
  JdbcConnection connection, String sql, ImmutableList<String> generatedKeysColumns)
  throws SQLException {
  super(connection);
- this.sql = sql;
+ this.sql = maybeAddWhereClause(sql);
  try {
  // The PostgreSQL parser allows comments to be present in the SQL string that is used to parse
  // the query parameters.

src/main/java/com/google/cloud/spanner/jdbc/JdbcStatement.java

@@ -103,7 +103,7 @@ public long executeLargeUpdate(String sql) throws SQLException {
 
  private long executeLargeUpdate(String sql, ImmutableList<String> generatedKeysColumns)
  throws SQLException {
- return executeLargeUpdate(Statement.of(sql), generatedKeysColumns);
+ return executeLargeUpdate(Statement.of(maybeAddWhereClause(sql)), generatedKeysColumns);
  }
 
  protected long executeLargeUpdate(Statement statement, ImmutableList<String> generatedKeysColumns)
@@ -112,7 +112,19 @@ protected long executeLargeUpdate(Statement statement, ImmutableList<String> gen
  checkClosed();
  Statement statementWithReturningClause =
  addReturningToStatement(statement, generatedKeysColumns);
- StatementResult result = execute(statementWithReturningClause);
+ boolean allowedDdlStatementInTransaction =
+ isAllowedDdlStatementInTransaction(statement.getSql());
+ StatementResult result;
+ if (allowedDdlStatementInTransaction) {
+ getConnection().setAutoCommit(true);
+ }
+ try {
+ result = execute(statementWithReturningClause);
+ } finally {
+ if (allowedDdlStatementInTransaction) {
+ getConnection().setAutoCommit(false);
+ }
+ }
  switch (result.getResultType()) {
  case RESULT_SET:
  if (generatedKeysColumns.isEmpty()) {
@@ -256,7 +268,7 @@ String quoteColumn(String column) {
 
  @Override
  public boolean execute(String sql) throws SQLException {
- return executeStatement(Statement.of(sql), NO_GENERATED_KEY_COLUMNS);
+ return executeStatement(Statement.of(maybeAddWhereClause(sql)), NO_GENERATED_KEY_COLUMNS);
  }
 
  boolean executeStatement(Statement statement, ImmutableList<String> generatedKeysColumns)