Skip to content

Commit

Permalink
Revert "Register java.sql.*isClosed methods for reflection"
Browse files Browse the repository at this point in the history
This reverts commit 8b90462.

The need for these additional registrations has been handled in upstream
GraalVM with oracle/graal#9184 so they are no
longer needed.

Closes quarkusio#41283
  • Loading branch information
zakkak authored and holly-cummins committed Jul 31, 2024
1 parent 5585de8 commit d770f3d
Showing 1 changed file with 0 additions and 15 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,7 @@

import static io.quarkus.deployment.Capability.OPENTELEMETRY_TRACER;

import java.sql.Connection;
import java.sql.Driver;
import java.sql.ResultSet;
import java.sql.Statement;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
Expand Down Expand Up @@ -63,7 +60,6 @@
import io.quarkus.deployment.builditem.nativeimage.NativeImageResourceBuildItem;
import io.quarkus.deployment.builditem.nativeimage.NativeImageResourceBundleBuildItem;
import io.quarkus.deployment.builditem.nativeimage.ReflectiveClassBuildItem;
import io.quarkus.deployment.builditem.nativeimage.ReflectiveMethodBuildItem;
import io.quarkus.deployment.pkg.builditem.CurateOutcomeBuildItem;
import io.quarkus.maven.dependency.ArtifactKey;
import io.quarkus.narayana.jta.deployment.NarayanaInitBuildItem;
Expand Down Expand Up @@ -425,15 +421,4 @@ void registerRowSetSupport(
"com.sun.rowset.providers.RIOptimisticProvider",
"com.sun.rowset.providers.RIXMLProvider").build());
}

/**
* Required with GraalVM for JDK 23 and up
*/
@BuildStep()
List<ReflectiveMethodBuildItem> registerIsClosedMethod() {
return List.of(
new ReflectiveMethodBuildItem(Statement.class.getName(), "isClosed", new Class[0]),
new ReflectiveMethodBuildItem(Connection.class.getName(), "isClosed", new Class[0]),
new ReflectiveMethodBuildItem(ResultSet.class.getName(), "isClosed", new Class[0]));
}
}

0 comments on commit d770f3d

Please sign in to comment.