[hibernate-commits] [hibernate/hibernate-orm] d6cb72: HHH-13409 Add integration tests for running Hibern...

[Yoann Rodière]

  Branch: refs/heads/master
  Home:   https://github.com/hibernate/hibernate-orm
  Commit: d6cb72ee0dc0b95445267e9cc20d5c3d5bec9e76
      https://github.com/hibernate/hibernate-orm/commit/d6cb72ee0dc0b95445267e9cc20d5c3d5bec9e76
  Author: Yoann Rodière <yoann at hibernate.org>
  Date:   2019-06-20 (Thu, 20 Jun 2019)

  Changed paths:
    M documentation/documentation.gradle
    A hibernate-integrationtest-java-modules/hibernate-integrationtest-java-modules.gradle
    A hibernate-integrationtest-java-modules/src/main/java/module-info.java
    A hibernate-integrationtest-java-modules/src/main/java/org/hibernate/orm/integrationtest/java/module/entity/Author.java
    A hibernate-integrationtest-java-modules/src/main/java/org/hibernate/orm/integrationtest/java/module/service/AuthorService.java
    A hibernate-integrationtest-java-modules/src/main/resources/META-INF/persistence.xml
    A hibernate-integrationtest-java-modules/src/test/java/org/hibernate/orm/integrationtest/java/module/test/JavaModulePathIT.java
    A hibernate-integrationtest-java-modules/src/test/resources/logging.properties
    M settings.gradle

  Log Message:
  -----------
  HHH-13409 Add integration tests for running Hibernate ORM in the module path in Java 11+


  Commit: b7e5106358650b86be31aadde28acf44ba74b907
      https://github.com/hibernate/hibernate-orm/commit/b7e5106358650b86be31aadde28acf44ba74b907
  Author: Yoann Rodière <yoann at hibernate.org>
  Date:   2019-06-20 (Thu, 20 Jun 2019)

  Changed paths:
    M hibernate-core/src/main/java/org/hibernate/boot/registry/classloading/spi/ClassLoaderService.java

  Log Message:
  -----------
  HHH-13409 Remove unnecessary "public" keywords in ClassLoaderService


  Commit: fc52db5cb49c0ec3581414debd96bd747e80a92d
      https://github.com/hibernate/hibernate-orm/commit/fc52db5cb49c0ec3581414debd96bd747e80a92d
  Author: Yoann Rodière <yoann at hibernate.org>
  Date:   2019-06-20 (Thu, 20 Jun 2019)

  Changed paths:
    A hibernate-core/src/main/java/org/hibernate/boot/registry/classloading/internal/AggregatedServiceLoader.java
    M hibernate-core/src/main/java/org/hibernate/boot/registry/classloading/internal/ClassLoaderServiceImpl.java

  Log Message:
  -----------
  HHH-13409 Move aggregated service loading to a dedicated component

So that we can provide a different implementation for Java 9+.


  Commit: 9b34fc69a35fc21233160004f32ab771bc50b048
      https://github.com/hibernate/hibernate-orm/commit/9b34fc69a35fc21233160004f32ab771bc50b048
  Author: Yoann Rodière <yoann at hibernate.org>
  Date:   2019-06-20 (Thu, 20 Jun 2019)

  Changed paths:
    M hibernate-core/src/main/java/org/hibernate/boot/registry/classloading/internal/AggregatedClassLoader.java
    M hibernate-core/src/main/java/org/hibernate/boot/registry/classloading/internal/AggregatedServiceLoader.java

  Log Message:
  -----------
  HHH-13409 Make AggregatedServiceLoader compatible with the module path (JDK9+)


  Commit: e7cefd8ddd00eff6a4614508d5bb0ee15bfd9aeb
      https://github.com/hibernate/hibernate-orm/commit/e7cefd8ddd00eff6a4614508d5bb0ee15bfd9aeb
  Author: Yoann Rodière <yoann at hibernate.org>
  Date:   2019-06-20 (Thu, 20 Jun 2019)

  Changed paths:
    M hibernate-core/src/main/java/org/hibernate/boot/registry/classloading/internal/AggregatedServiceLoader.java
    M hibernate-core/src/main/java/org/hibernate/boot/registry/classloading/internal/ClassLoaderServiceImpl.java

  Log Message:
  -----------
  HHH-13409 Rework AggregatedServiceLoader to minimize the risk of regression

In particular:

* Keep the old behavior when retrieving services on JDK8
* On JDK9+, query the AggregatedClassLoader first (before individual
class loaders) when retrieving services.
* On JDK9+, use ServiceLoader.Provider to avoid instantiating services
a second time if we know we already instantiated them with another class
loader.


Compare: https://github.com/hibernate/hibernate-orm/compare/6b199d2e5285...e7cefd8ddd00