JSON-P, JSON-B and JAX-RS upgrades (2nd attempt) #10843
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
retest this please |
…he default server will continue to use JSON-P 1.0.
asoldano
force-pushed
the
resteasy-3.5-jamezp
branch
from
730dbc8
to
64e6bc9
Compare
|
rebased to solve conflict, for the records all test runs passed before. |
jamezp
added a commit
to jamezp/wildfly
that referenced
this pull request
Feb 12, 2018
[WFLY-7185] JSON-B 1.0 smoke test
|
Execution timeout for Elytron run. Linux run has one failed test - failed org.jboss.as.test.clustering.cluster.web ConcurrentFineWebFailoverTestCase.testGracefulSimpleFailover. Probably related to JGroups + ISPN upgrade which was merge lately. |
jamezp
added a commit
to jamezp/wildfly
that referenced
this pull request
Feb 12, 2018
[WFLY-7185] JSON-B 1.0 smoke test
|
I've merged #10872 which is going to make this out of date. It includes the 2 JSON commits and the 2 JSON test commits. I'll create a new branch on my repo where I'll handle the rebase for this so that it's easier of you guys to rebase in the future. I'll leave this open for now and we can close once a new PR is opened or you can just force push to this branch. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This should replace #10834 and fix the jaxrs failures in that.
Overview
This PR includes changes for JSON-P, JSON-B and JAX-RS with the Java EE 8 tech preview. @stuartwdouglas please let me know if the module configurations look correct as well.
JSON-P 1.1 WFLY-6460
This replaces JSON-P 1.0 with JSON-P 1.1 when the
ee8.preview.modesystem property is set totrue. One thing to note about this is the system property only seems to work when the user specifies it in theJAVA_OPTS. This may be because elytron accesses the module before the property is set.Also note that in the JMS client we hard-code the version to 1.0.4 since client itself still relies on JSON-P 1.0. This library is shaded into the client which is the reason for doing this.
Also note that JSON-P 1.0 implementation library includes both the API and the implementation which is my the
module.xmlexports only thejavax.json.*packages.JSON-B 1.0 WFLY-7185
This introduces the new JSON-B 1.0 API. The module itself will be added to all deployments however the resource will only be added if the
ee8.preview.modesystem property is set totrue.There are currently not tests for JSON-B however I did test manually and I will follow up with tests after #10833 gets merged.
JAX-RS Upgrade Information
This is a preview of the upgrade that I'm soon submitting for bringing JAX-RS 2.1 RESTEasy implementation in WildFly. I'd like the reviewers to start having a look and letting me know of any additional requirements / changes needed for merging before I tag 3.5.0.Final.
I'm dealing with the needed analysis / test plan in the mean time; besides JSR 370 (JAX-RS 2.1), the RESTEasy component upgrade also pulls few other planned new features.
Few JIRA links:
https://issues.jboss.org/browse/WFLY-9789
https://issues.jboss.org/browse/WFLY-6459
https://issues.jboss.org/browse/WFLY-9786
https://issues.jboss.org/browse/WFLY-9787
Dev analysis:
wildfly/wildfly-proposals#16
wildfly/wildfly-proposals#18
wildfly/wildfly-proposals#19