Getting "type not supported as root type by protobuf" for serialization of short and UUID types about jackson-dataformats-binary HOT 10 CLOSED

I can't say off-hand what could be causing this. Which version is this with?

from jackson-dataformats-binary.

Currently, 2.8.1 but also tried the latest version 2.9.0.pr2

from jackson-dataformats-binary.

@eldadru then it probably exists in 2.8.7 too. If easy enough would be nice to double-check, but if not this is fine.

from jackson-dataformats-binary.

Sorry, I already refactored the code

from jackson-dataformats-binary.

@eldadru that's ok, I should be able to reproduce this easily.

from jackson-dataformats-binary.

Yes, I can reproduce this. Should work, not sure yet why it does not.

from jackson-dataformats-binary.

Ok. Fixing may take a while since this is part of code I did not write (it's a contribution), and some of the logic is bit difficult to follow. Fix for short is relatively easy (it just needs to be handled like int, basically). And in theory UUID and similar types really should be handled as either String, or, perhaps, byte array (since format supports native binary content).
But for some reason current schema generator is pickier...

from jackson-dataformats-binary.

There was a bit bigger issue, but things should work significantly better after fix. Will be in 2.9.0.pr3.

from jackson-dataformats-binary.

@cowtowncoder Thanks!

from jackson-dataformats-binary.

@eldadru Thank you for reporting this: handling was basically wrong and could only support a small number of primitives, regular POJOs and Lists/arrays. What needed to be added was actual handling of typing as indicated by callbacks, so that (for example) date/time values are indicate as types that they would be encoded in JSON (if written as textual timestamps, then String, if long timestamps, 64-bit and so on).

I suspect there may some other rough edges but I hope these can be located and fixed over time as well.

from jackson-dataformats-binary.