Upgrade to jackson 2.3.1 (renaming packages); addition for swift api.#137
Upgrade to jackson 2.3.1 (renaming packages); addition for swift api.#137bluebossa63 wants to merge 6 commits intowoorea:masterfrom
Conversation
|
This is very huge! Anyway thank you for the pull, I will merge maintaining Jackson compatibility. I tried with Jersey before and I made it work, but I have no tried the new Jackson with RestEasy. I plan to move to a new version ASAP but i want to check with resteasy people before. Options are: Jackson 2 I know XML is less important for android stuff, so another posibility is (ie. keystone) keystone-client keystone-model (only interfaces) What do you think guys? |
|
I do not think that it makes a lot of sense to split the code because of the libraries. http://docs.jboss.org/resteasy/docs/3.0.2.Final/userguide/html/json.html ( Using Jackson 2.2.x Inside of JBoss AS7 ) I would leave it as it is for now, and if you guys decide to move forward with jackson 2, I would integrate the pull fully... And my pull is not huge, most of the changes are done by removing the old libraries and with a simple eclipse / source / organize imports. I'm afraid, that a lot of formatting changes were done this way. Then there were basically 3 exceptions to this rule: SerializationConfiguration.Feature -> SerializationFeature And for jersey I had an issue with the HeadersFactory, please check this! The rest are the changes I already provided as a patch for the Android Client https://github.com/bluebossa63/OpenStackIntegration In the changes of the model there is only one thing a little bit strange: com.woorea.openstack.keystone.model.User I had some problems with the password field name |
JBoss EAP 6.2.0.GA (AS 7.3.0.Final-redhat-14) and Jackson 2.3.1
|
complete refactoring and successfully tested with |
|
I had to roll back! The issue was that too much refactoring would be needed: { was not possible to deserialize with the actual model. Switching back, everything was parsed as expected. There are some heavy differences in parsing between the jackson libraries 1 and 2. |
|
Hi, @woorea do you plan to accept this pull request in the near future? I tried the SDK with @bluebossa63's support for Jackson 2.3 and it is great. |
I made a fork for this because the newer jackson library won't be compatible for every resteasy implementation. As I use the SDK as part of an OpenStack Client for Android, I have a focus on performance and the storage cloud.
I'm sorry but there are a lot of changes that are simply formatting issues.