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

[Java][MicroProfile] Use jsonb imports for serializing and deserializing enum #6064

Merged

Conversation

Ravisankar-Challa
Copy link
Contributor

Fixes the issue no: 2 reported in the issue #6028

The pull request removes all jaxb (xml) imports when withXml=false also it uses JsonbTypeSerializer & JsonbTypeDeserializer to deal with enum data types.

@Ravisankar-Challa Ravisankar-Challa force-pushed the microprofile_jaxb_imports branch 3 times, most recently from cab7830 to 5b375bb Compare April 28, 2020 07:03
@Ravisankar-Challa Ravisankar-Challa changed the title Use jsonb imports for serializing and deserializing pojo Use jsonb imports for serializing and deserializing enum Apr 29, 2020
@wing328
Copy link
Member

wing328 commented Jun 15, 2020

@Ravisankar-Challa thanks for the PR. I did a test with petstore and withXml set to true but got the following error messages (partial):

[ERROR] /private/tmp/microprofile/src/main/java/org/openapitools/client/model/Order.java:[213,4] cannot find symbol
[ERROR]   symbol:   class JsonbProperty
[ERROR]   location: class org.openapitools.client.model.Order
[ERROR] /private/tmp/microprofile/src/main/java/org/openapitools/client/model/Pet.java:[108,4] cannot find symbol
[ERROR]   symbol:   class JsonbProperty
[ERROR]   location: class org.openapitools.client.model.Pet
[ERROR] /private/tmp/microprofile/src/main/java/org/openapitools/client/model/Pet.java:[129,4] cannot find symbol
[ERROR]   symbol:   class JsonbProperty
[ERROR]   location: class org.openapitools.client.model.Pet
[ERROR] /private/tmp/microprofile/src/main/java/org/openapitools/client/model/Pet.java:[150,4] cannot find symbol
[ERROR]   symbol:   class JsonbProperty
[ERROR]   location: class org.openapitools.client.model.Pet
[ERROR] /private/tmp/microprofile/src/main/java/org/openapitools/client/model/Pet.java:[171,4] cannot find symbol
[ERROR]   symbol:   class JsonbProperty
[ERROR]   location: class org.openapitools.client.model.Pet
[ERROR] /private/tmp/microprofile/src/main/java/org/openapitools/client/model/Pet.java:[197,4] cannot find symbol
[ERROR]   symbol:   class JsonbProperty
[ERROR]   location: class org.openapitools.client.model.Pet
[ERROR] /private/tmp/microprofile/src/main/java/org/openapitools/client/model/Pet.java:[223,4] cannot find symbol
[ERROR]   symbol:   class JsonbProperty
[ERROR]   location: class org.openapitools.client.model.Pet
[ERROR] -> [Help 1]

I wonder if you can take a look when you've time.

@Ravisankar-Challa
Copy link
Contributor Author

Ravisankar-Challa commented Jun 16, 2020

@wing328 I have fixed the issue related to withXml=true. I am able to successfully run ./bin/run-all-petstore in local. But it is failing in CI with error ./bin/run-all-petstore not found. Can you please take a look. https://cloud.drone.io/OpenAPITools/openapi-generator/9162

@wing328
Copy link
Member

wing328 commented Jun 16, 2020

@wing328 wing328 added this to the 5.0.0 milestone Jun 16, 2020
@wing328 wing328 changed the title Use jsonb imports for serializing and deserializing enum [Java][MicroProfile] Use jsonb imports for serializing and deserializing enum Jun 16, 2020
@Ravisankar-Challa
Copy link
Contributor Author

Ravisankar-Challa commented Jun 17, 2020

@wing328 I have done rebase on master branch. Now the build is successful.
I am trying to fix the issue #6028
Which has 2 more pull requests could you please verify them as well please.
https://github.com/OpenAPITools/openapi-generator/pull/6062/files
https://github.com/OpenAPITools/openapi-generator/pull/6680/files

@wing328
Copy link
Member

wing328 commented Jun 17, 2020

Tested with both withXml set to true, false and both are compiling fine without issues.

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

Successfully merging this pull request may close these issues.

2 participants