Jackson JSON Processor

Autodetect JSON vs Smile on deserialization

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Duplicate
  • Affects Version/s: None
  • Fix Version/s: 2.1
  • Component/s: JsonParser
  • Labels:
  • Number of attachments :
    0

Description

It would be super cool if it were possible to configure a JsonParser / ObjectMapper to auto detect SMILE vs JSON.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Tatu Saloranta added a comment -

Yes.

And there is in fact quite of bit of groundwork for this – observe existence of JsonFactory.hasFormat(...), as well as InputAccessor.Std – which are both implemented fully for basic JSON and Smile.
In fact there may already be a unit test that does this, figure out which of multiple formats (as defined by JsonFactory.hasFormat(...) returning best match) is most likely the correct one.

Question then is what would be the best way to make this feature as convenient to use as possible, above and beyond basic support that is in place.

Show
Tatu Saloranta added a comment - Yes. And there is in fact quite of bit of groundwork for this – observe existence of JsonFactory.hasFormat(...), as well as InputAccessor.Std – which are both implemented fully for basic JSON and Smile. In fact there may already be a unit test that does this, figure out which of multiple formats (as defined by JsonFactory.hasFormat(...) returning best match) is most likely the correct one. Question then is what would be the best way to make this feature as convenient to use as possible, above and beyond basic support that is in place.
Tatu Saloranta made changes -
Field Original Value New Value
Status Open [ 1 ] In Progress [ 3 ]
Hide
Permalink
Tatu Saloranta added a comment -

Actually, implemented it as:

https://github.com/FasterXML/jackson-databind/issues/15

should now add unit tests...

Show
Tatu Saloranta added a comment - Actually, implemented it as: https://github.com/FasterXML/jackson-databind/issues/15 should now add unit tests...
Hide
Permalink
Tatu Saloranta added a comment -

as per above, closing as dup.

Show
Tatu Saloranta added a comment - as per above, closing as dup.
Tatu Saloranta made changes -
Status In Progress [ 3 ] Closed [ 6 ]
Fix Version/s 2.1 [ 18536 ]
Resolution Duplicate [ 3 ]

People

  • Assignee:
    Tatu Saloranta
    Reporter:
    Steven Schlansker
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: