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.

Sign up for GitHub

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

Jump to bottom

Replace Gson by Jackson #755

Open
abourdon opened this issue May 10, 2016 · 2 comments
Open

Replace Gson by Jackson #755

abourdon opened this issue May 10, 2016 · 2 comments
Labels
Nat : refactor Perim : OTHERS Prio : 3 minor

Comments

@abourdon
Copy link
Member

The Gson library is used to map objects to JSON representation. But this library lacks a key feature: let user to use getter/setter to get attribute value. Actually, only fields are used during mapping.

An issue has been opened from the Gson Github: google/gson#232

I propose to use an other library, the powerful Jackson library, where this feature is already developed. As the Gson one, the Jackson library is already integrated to the Spring framework for latter refactoring if any.

Let me know
abourdon added a commit that referenced this issue May 17, 2016
@abourdon
feat(capabilities): robot capabilities have now a dedicated database
53f7fc7 
- Create the new RobotCapabilities object to give more flexibility in
  defining capabilities (relates #393)
- Use the Jackson library to serialize Robot and RobotCapabilities
  objects (relates #755)
abourdon referenced this issue Jan 17, 2017
@abourdon
feat(run multiple): add the ReadExecutionPools servlet
e58d45d 
To read information about execution pools

References #914
@abourdon
Copy link
Member Author

abourdon commented Feb 1, 2017

Another argument, Gson cannot handle anonymous classes: http://stackoverflow.com/questions/39538677/why-does-gson-tojson-always-return-null

abourdon added a commit that referenced this issue Feb 1, 2017
@abourdon
feat(multiple run): get information about a single ExecutionThreadPool
d90db64 
Also:
        - Create the IExecutionThreadPoolService and move
          implementations to a dedicated package
        - Use the Jakson library for Json/Object mapping because of
          limited Gson library
        - ExecutionThreadReset servlet is now deprecated

References #914
References #755
@abourdon
Copy link
Member Author

Library is approved, we just have to have additional time to do it ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Nat : refactor Perim : OTHERS Prio : 3 minor
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@abourdon @vertigo17