Eduardo Martins

Results 101 comments of Eduardo Martins

temp-concurrency-tck-emartins.yml removed, so this is it, ready for merge, please review

@bstansberry all done except the ee jboss module breakup, which I am still working on. Meanwhile if you can please resolve the other issues fixed by the introduced commit

@bstansberry ee jboss module breakup added, so it's now ready for another round of review

@tadamski I am seeing the following warning when using the DataSourceDefinition instead of the -ds.xml 20:12:46,089 WARN [org.jboss.jca.core.connectionmanager.pool.strategy.OnePool] (ServerService Thread Pool -- 46) IJ000407: No lazy enlistment available for batch-processing_batch-processing_batch-processing_java:jboss/datasources/batch-processingDS...

@jfdenise FYI it seems Glow is not aware of DataSourceDefinition annotation and reports the following error in this project, if you remove the failOnError `[ERROR] * unbound datasources error: java:jboss/datasources/batch-processingDS`

@liweinan IMHO this is still a work in progress, after all it currently replaces one warn with another, and then there is the provisioning error... I suspect we will need...

@tadamski can you please have a look at https://github.com/wildfly/quickstart/pull/973#issuecomment-2448281052 above once you have a chance?

@kstekovi for the record I don't think we should merge this till we have that new warning suppressed or documented, we should not replace one warning with another.

thanks @jfdenise , I will keep this open till that happens.