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

Quick starts example 'Using HTTP with Reactive Messaging' is not working; the import javax.ws cannot be resolved. #24948

Closed
ben-eddy74 opened this issue Apr 14, 2022 · 9 comments

Comments

@ben-eddy74
Copy link

Describe the bug

Following guide Using HTTP With Reactive Messaging using the creating the Maven Project method, generates error the import javax.ws cannot be resolved.

Expected behavior

Following a guide as a beginner, i expect the outcome is working as described.

Actual behavior

Code does not compile, the import javax.ws cannot be resolved.

How to Reproduce?

  1. Follow the guide Using HTTP With Reactive Messaging
  2. Use the creating the Maven Project method.
  3. Add the CostCollector class code

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

Extension io.quarkus:quarkus-resteasy-reactive is missing from the Create a new project with the following command example.

@quarkus-bot
Copy link

quarkus-bot bot commented Apr 14, 2022

@geoand
Copy link
Contributor

geoand commented Apr 15, 2022

Seems like RESTEasy Reactive is not added to the pom.xml. @ia3andy @gsmet is there a reason for that?

@ia3andy
Copy link
Contributor

ia3andy commented Apr 15, 2022

I guess the command should have the resteasy-reactive extension like this:

mvn io.quarkus.platform:quarkus-maven-plugin:2.8.0.Final:create \
    -DprojectGroupId=org.acme \
    -DprojectArtifactId=reactive-messaging-http-quickstart \
    -Dextensions="reactive-messaging-http,resteasy-reactive" \
    -DnoCode

-DnoExamples does not exist anymore it was replaced by -DnoCode, we should check all the guides.

@ia3andy
Copy link
Contributor

ia3andy commented Apr 15, 2022

I create a PR for the first-extension guide: #24957

@ia3andy
Copy link
Contributor

ia3andy commented Apr 15, 2022

Regarding the missing resteasy-reactive extension shouldn't this be a conditional dependency?

@geoand
Copy link
Contributor

geoand commented Apr 15, 2022

A conditional dependency of what?

@ia3andy
Copy link
Contributor

ia3andy commented Apr 15, 2022

Ok I thought it was needed by reactive-messaging, but it's just the guide example which needs it. So it's just a matter of adding it to the create command, I am updating the issue: quarkiverse/quarkus-reactive-messaging-http#59

@cescoffier
Copy link
Member

Closing as the downstream issue is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants