Skip to content

Commit

Permalink
fix: Prevent scanning the whole class path when packages are configur…
Browse files Browse the repository at this point in the history
…ed but no entities present.

Thanks @meistermeier.
  • Loading branch information
michael-simons committed Apr 19, 2023
1 parent 5153581 commit 342248b
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -161,10 +161,17 @@ Neo4jOgmSessionFactoryBuildItem createSessionFactory(Neo4jOgmRecorder recorder,
ShutdownContextBuildItem shutdownContext,
BuildProducer<SyntheticBeanBuildItem> syntheticBeans,
Neo4jOgmProperties ogmProperties,
EntitiesBuildItem allClasses) {
Neo4jOgmBuiltTimeProperties buildTimeProperties,
EntitiesBuildItem allClasses
) {

var allPackages = allClasses.getValue().stream().map(Class::getPackageName)
.distinct().toArray(String[]::new);

if (allPackages.length == 0 && buildTimeProperties.basePackages.isPresent()) {
allPackages = buildTimeProperties.basePackages.get().toArray(new String[0]);
}

var sessionFactoryRuntimeValue = recorder
.initializeSessionFactory(driverBuildItem.getValue(), shutdownContext, ogmProperties, allPackages);

Expand Down
4 changes: 2 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@
<docker-maven-plugin.version>0.42.1</docker-maven-plugin.version>
<exec-maven-plugin.version>3.1.0</exec-maven-plugin.version>
<graalvm.version>22.3.1</graalvm.version>
<java-module-name />
<java-module-name/>
<java.version>17</java.version>
<junit.jupiter.version>5.8.2</junit.jupiter.version>
<maven-checkstyle-plugin.version>3.2.1</maven-checkstyle-plugin.version>
Expand Down Expand Up @@ -299,7 +299,7 @@
<requireJavaVersion>
<version>${java.version}</version>
</requireJavaVersion>
<DependencyConvergence />
<DependencyConvergence/>
<requireMavenVersion>
<version>${maven.version}</version>
</requireMavenVersion>
Expand Down

0 comments on commit 342248b

Please sign in to comment.