-
Notifications
You must be signed in to change notification settings - Fork 89
SBM tool - migrate-stateless-ejb isssue #837
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
Comments
Thanks for reporting and giving SBM a try @brebDevv 🚀 What I learn from your issue is that SBM should allow using JDK versions supported by OpenRewrite and not force you to an outdated version. I will create a new issue for this. |
HI @fabapp2 . Unfortunately it's not easy to upgrade to JDK17. That's why I've tried with SBM versions: 0.11 and 0.11.1 , since I've seen that from 0.12 -> 0.14 it requires only the JDK 17. |
Hi @fabapp2 . I've managed to upgrade the project to JDK-17. I've used the SBM-0.14, and applied the following command: apply migrate-jax-rs . After around 5 minutes hanging, I've received the same error again: But even though that error appeared, the JAX-RS annotation were replaced by spring ones and spring-boot-starter was added. So I guess it did the job. Please give a sign, if you manage to fix these errors. Really appreciate it, |
Hi @brebDevv |
Describe the bug
Running the migrate-stateless-ejb from the terminal in Windows/Linux, is making SBM to throw this error:
ERROR o.s.sbm.java.impl.OpenRewriteMember - Could not get Type for annotation: 'EJB' while comparing with 'javax.inject.Inject'.
SBM Version
0.11 and 0.11.1 , since the application is using Java 11 and not Java 17
SBM Application
Shell
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I've would expect for all the @ejb to be migrated to @Autowired
Screenshots

Stacktrace
There isn't any stacktrace, except the error that appears in the terminal:
ERROR o.s.sbm.java.impl.OpenRewriteMember - Could not get Type for annotation: 'EJB' while comparing with 'javax.inject.Inject'.
The text was updated successfully, but these errors were encountered: