You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@deltaspike.apache.org by GitBox <gi...@apache.org> on 2022/04/21 01:53:06 UTC

[GitHub] [deltaspike] hazendaz opened a new pull request, #124: Fix weld3 build by using nweer arquillian-weld and bump weld to 3.1.9.Final

hazendaz opened a new pull request, #124:
URL: https://github.com/apache/deltaspike/pull/124

   weld3 build is broken currently when running this profile - mvn clean install -PWeld3.  Fixed it by updating arquillian.  To error on side of caution, only fixed for weld 3.  Also bumped weld to 3.1.9.Final.  Build tested as passing using jdk 8.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@deltaspike.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [deltaspike] struberg commented on pull request #124: Fix weld3 build by using newer arquillian-weld, weld 3.1.9.Final, geronimo fix for HV 6, build from 8 to 17

Posted by GitBox <gi...@apache.org>.
struberg commented on PR #124:
URL: https://github.com/apache/deltaspike/pull/124#issuecomment-1179747268

   Hi @hazendaz ! Thanks a lot for these really cool fixes!
   I picked them separately, sometimes squashed em and added minor changes. I think I grabbed all except the one I commented on (jakarta-el api) where I did not find why it should be necessary. Gonna close this PR now, but please ping us if you think we missed something!
   
   Thanks again and looking forward to further contributions!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@deltaspike.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [deltaspike] hazendaz commented on a diff in pull request #124: Fix weld3 build by using newer arquillian-weld, weld 3.1.9.Final, geronimo fix for HV 6, build from 8 to 17

Posted by GitBox <gi...@apache.org>.
hazendaz commented on code in PR #124:
URL: https://github.com/apache/deltaspike/pull/124#discussion_r922761257


##########
deltaspike/parent/code/pom.xml:
##########
@@ -240,6 +240,13 @@
                 <validation.artifactId>geronimo-validation_2.0_spec</validation.artifactId>
                 <hibernate.validator.version>6.0.17.Final</hibernate.validator.version>
             </properties>
+            <dependencies>

Review Comment:
   jdk 17 for the el need.  hibernate needs it there.  Will send new PR for it again.
   
   INFO: HV000003: Using org.apache.deltaspike.beanvalidation.impl.CDIAwareConstraintValidatorFactory as constraint validator factory.
   [ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.49 s <<< FAILURE! - in org.apache.deltaspike.beanvalidation.test.InjectedValidationTest
   [ERROR] testValidate(org.apache.deltaspike.beanvalidation.test.InjectedValidationTest)  Time elapsed: 0.37 s  <<< ERROR!
   javax.validation.ValidationException: HV000183: Unable to initialize 'javax.el.ExpressionFactory'. Check that you have the EL dependencies on the classpath, or use ParameterMessageInterpolator instead
           at org.hibernate.validator.messageinterpolation.ResourceBundleMessageInterpolator.buildExpressionFactory(ResourceBundleMessageInterpolator.java:123)
           at org.hibernate.validator.messageinterpolation.ResourceBundleMessageInterpolator.<init>(ResourceBundleMessageInterpolator.java:47)
           at org.hibernate.validator.internal.engine.ConfigurationImpl.getDefaultMessageInterpolator(ConfigurationImpl.java:474)
           at org.hibernate.validator.internal.engine.ConfigurationImpl.getDefaultMessageInterpolatorConfiguredWithClassLoader(ConfigurationImpl.java:650)
           at org.hibernate.validator.internal.engine.ConfigurationImpl.getMessageInterpolator(ConfigurationImpl.java:397)
           at org.hibernate.validator.internal.engine.ValidatorFactoryImpl.<init>(ValidatorFactoryImpl.java:183)
           at org.hibernate.validator.HibernateValidator.buildValidatorFactory(HibernateValidator.java:38)
           at org.hibernate.validator.internal.engine.ConfigurationImpl.buildValidatorFactory(ConfigurationImpl.java:364)
           at org.apache.deltaspike.beanvalidation.test.InjectedValidationTest.initValidator(InjectedValidationTest.java:60)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)
           at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
           at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:56)
           at org.junit.internal.runners.statements.RunBefores.invokeMethod(RunBefores.java:33)
           at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
           at org.jboss.arquillian.junit.Arquillian$StatementLifecycleExecutor.invoke(Arquillian.java:459)
           at org.jboss.arquillian.container.test.impl.execution.ClientBeforeAfterLifecycleEventExecuter.execute(ClientBeforeAfterLifecycleEventExecuter.java:99)
           at org.jboss.arquillian.container.test.impl.execution.ClientBeforeAfterLifecycleEventExecuter.on(ClientBeforeAfterLifecycleEventExecuter.java:72)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.invokeObservers(EventContextImpl.java:99)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:81)
           at org.jboss.arquillian.container.test.impl.client.ContainerEventController.createContext(ContainerEventController.java:142)
           at org.jboss.arquillian.container.test.impl.client.ContainerEventController.createBeforeContext(ContainerEventController.java:124)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:88)
           at org.jboss.arquillian.test.impl.TestContextHandler.createClassContext(TestContextHandler.java:92)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:88)
           at org.jboss.arquillian.test.impl.TestContextHandler.createSuiteContext(TestContextHandler.java:73)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:88)
           at org.jboss.arquillian.test.impl.TestContextHandler.createTestContext(TestContextHandler.java:130)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:88)
           at org.jboss.arquillian.core.impl.ManagerImpl.fire(ManagerImpl.java:145)
           at org.jboss.arquillian.core.impl.ManagerImpl.fire(ManagerImpl.java:116)
           at org.jboss.arquillian.test.impl.EventTestRunnerAdaptor.before(EventTestRunnerAdaptor.java:108)
           at org.jboss.arquillian.junit.Arquillian$4.evaluate(Arquillian.java:241)
           at org.jboss.arquillian.junit.Arquillian.multiExecute(Arquillian.java:422)
           at org.jboss.arquillian.junit.Arquillian.access$200(Arquillian.java:54)
           at org.jboss.arquillian.junit.Arquillian$5.evaluate(Arquillian.java:259)
           at org.jboss.arquillian.junit.Arquillian$7$1.invoke(Arquillian.java:315)
           at org.jboss.arquillian.container.test.impl.execution.ClientBeforeAfterLifecycleEventExecuter.execute(ClientBeforeAfterLifecycleEventExecuter.java:99)
           at org.jboss.arquillian.container.test.impl.execution.ClientBeforeAfterLifecycleEventExecuter.on(ClientBeforeAfterLifecycleEventExecuter.java:72)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.invokeObservers(EventContextImpl.java:99)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:81)
           at org.jboss.arquillian.container.test.impl.client.ContainerEventController.createContext(ContainerEventController.java:142)
           at org.jboss.arquillian.container.test.impl.client.ContainerEventController.createBeforeContext(ContainerEventController.java:124)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:88)
           at org.jboss.arquillian.test.impl.TestContextHandler.createClassContext(TestContextHandler.java:92)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:88)
           at org.jboss.arquillian.test.impl.TestContextHandler.createSuiteContext(TestContextHandler.java:73)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:88)
           at org.jboss.arquillian.test.impl.TestContextHandler.createTestContext(TestContextHandler.java:130)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
           at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
           at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
           at java.base/java.lang.reflect.Method.invoke(Method.java:568)
           at org.jboss.arquillian.core.impl.ObserverImpl.invoke(ObserverImpl.java:94)
           at org.jboss.arquillian.core.impl.EventContextImpl.proceed(EventContextImpl.java:88)
           at org.jboss.arquillian.core.impl.ManagerImpl.fire(ManagerImpl.java:145)
           at org.jboss.arquillian.core.impl.ManagerImpl.fire(ManagerImpl.java:116)
           at org.jboss.arquillian.test.impl.EventTestRunnerAdaptor.fireCustomLifecycle(EventTestRunnerAdaptor.java:159)
           at org.jboss.arquillian.junit.Arquillian$7.evaluate(Arquillian.java:311)
           at org.junit.runners.BlockJUnit4ClassRunner$1.evaluate(BlockJUnit4ClassRunner.java:100)
           at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:366)
           at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:103)
           at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:63)
           at org.junit.runners.ParentRunner$4.run(ParentRunner.java:331)
           at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:79)
           at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:329)
           at org.junit.runners.ParentRunner.access$100(ParentRunner.java:66)
           at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:293)
           at org.jboss.arquillian.junit.Arquillian$2.evaluate(Arquillian.java:204)
           at org.jboss.arquillian.junit.Arquillian.multiExecute(Arquillian.java:422)
           at org.jboss.arquillian.junit.Arquillian.access$200(Arquillian.java:54)
           at org.jboss.arquillian.junit.Arquillian$3.evaluate(Arquillian.java:218)
           at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
           at org.junit.runners.ParentRunner.run(ParentRunner.java:413)
           at org.jboss.arquillian.junit.Arquillian.run(Arquillian.java:166)
           at org.junit.runners.Suite.runChild(Suite.java:128)
           at org.junit.runners.Suite.runChild(Suite.java:27)
           at org.junit.runners.ParentRunner$4.run(ParentRunner.java:331)
           at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:79)
           at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:329)
           at org.junit.runners.ParentRunner.access$100(ParentRunner.java:66)
           at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:293)
           at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
           at org.junit.runners.ParentRunner.run(ParentRunner.java:413)
           at org.apache.maven.surefire.junitcore.JUnitCore.run(JUnitCore.java:55)
           at org.apache.maven.surefire.junitcore.JUnitCoreWrapper.createRequestAndRun(JUnitCoreWrapper.java:137)
           at org.apache.maven.surefire.junitcore.JUnitCoreWrapper.executeEager(JUnitCoreWrapper.java:107)
           at org.apache.maven.surefire.junitcore.JUnitCoreWrapper.execute(JUnitCoreWrapper.java:83)
           at org.apache.maven.surefire.junitcore.JUnitCoreWrapper.execute(JUnitCoreWrapper.java:75)
           at org.apache.maven.surefire.junitcore.JUnitCoreProvider.invoke(JUnitCoreProvider.java:158)
           at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:384)
           at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:345)
           at org.apache.maven.surefire.booter.ForkedBooter.execute(ForkedBooter.java:126)
           at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:418)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@deltaspike.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [deltaspike] hazendaz commented on pull request #124: Fix weld3 build by using newer arquillian-weld, weld 3.1.9.Final, geronimo fix for HV 6, build from 8 to 17

Posted by GitBox <gi...@apache.org>.
hazendaz commented on PR #124:
URL: https://github.com/apache/deltaspike/pull/124#issuecomment-1174224340

   Adding onto this PR the following
   - Add missing relativePath so that parent looks to .m2 cache instead of build space
   - Correct geronimo 2.0 spec as required level for hibernate validator 6.0 which was breaking on newer jdks
   - Add opens to tests for jdk through 17 support
   - Add jakarta.el for jdk through 17 support
   - Correct memory usage on tests since jdk 8


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@deltaspike.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [deltaspike] hazendaz commented on a diff in pull request #124: Fix weld3 build by using newer arquillian-weld, weld 3.1.9.Final, geronimo fix for HV 6, build from 8 to 17

Posted by GitBox <gi...@apache.org>.
hazendaz commented on code in PR #124:
URL: https://github.com/apache/deltaspike/pull/124#discussion_r922761642


##########
deltaspike/parent/code/pom.xml:
##########
@@ -240,6 +240,13 @@
                 <validation.artifactId>geronimo-validation_2.0_spec</validation.artifactId>
                 <hibernate.validator.version>6.0.17.Final</hibernate.validator.version>
             </properties>
+            <dependencies>

Review Comment:
   happens in [INFO] Apache DeltaSpike BeanValidation-Module Impl ....... FAILURE [  6.293 s]



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@deltaspike.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [deltaspike] struberg commented on a diff in pull request #124: Fix weld3 build by using newer arquillian-weld, weld 3.1.9.Final, geronimo fix for HV 6, build from 8 to 17

Posted by GitBox <gi...@apache.org>.
struberg commented on code in PR #124:
URL: https://github.com/apache/deltaspike/pull/124#discussion_r917395365


##########
deltaspike/parent/code/pom.xml:
##########
@@ -240,6 +240,13 @@
                 <validation.artifactId>geronimo-validation_2.0_spec</validation.artifactId>
                 <hibernate.validator.version>6.0.17.Final</hibernate.validator.version>
             </properties>
+            <dependencies>

Review Comment:
   in which combination of JDK and server did you see this as a problem?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@deltaspike.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [deltaspike] struberg closed pull request #124: Fix weld3 build by using newer arquillian-weld, weld 3.1.9.Final, geronimo fix for HV 6, build from 8 to 17

Posted by GitBox <gi...@apache.org>.
struberg closed pull request #124: Fix weld3 build by using newer arquillian-weld, weld 3.1.9.Final, geronimo fix for HV 6, build from 8 to 17
URL: https://github.com/apache/deltaspike/pull/124


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@deltaspike.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org