Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WFLY-15041 MDB not working when the bean class does not implement Mes… #14493

Merged
merged 1 commit into from Aug 6, 2021

Conversation

chengfang
Copy link
Contributor

@chengfang chengfang commented Jul 20, 2021

…sageListener interface

Change test bean MDBWithLookupActivationConfigProperties to not implement MessageListener interface to expose the problem

https://issues.redhat.com/browse/WFLY-15041

This pull request will be ready to merge after wildfly-core is upgraded to 17.0.0.Beta3 (WFLY-15042 Upgrade WildFly Core to 17.0.0.Beta3). Once the wildfly-core upgrade is in place, we should re-run all checks. Right now (2021-07-21) the test MDBActivationConfigTestCase is failing, as expected.

Update 2021-07-29: all dependency of this pull request has been satisfied and this PR is ready to be reviewed.

…sageListener interface

Change test bean MDBWithLookupActivationConfigProperties to not implement MessageListener interface to expose the problem
@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Jul 20, 2021
@fl4via
Copy link
Contributor

fl4via commented Jul 26, 2021

@chengfang The PR for including the JBoss Invocation fix (jbossas/jboss-invocation#24) in WildFly Core is here: wildfly/wildfly-core#4683

@chengfang chengfang changed the title (DRAFT) WFLY-15041 MDB not working when the bean class does not implement Mes… WFLY-15041 MDB not working when the bean class does not implement Mes… Jul 29, 2021
@chengfang chengfang marked this pull request as ready for review July 29, 2021 17:19
@chengfang
Copy link
Contributor Author

/retest

@chengfang
Copy link
Contributor Author

I think this PR is ready to be merged.

@bstansberry bstansberry merged commit 2b20c5e into wildfly:main Aug 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
3 participants