Can't serialize LinkedMap with Jackson Databind

[ikitommi]

isEmpty() is not implemented:

java.lang.AbstractMethodError: Method linked/map/LinkedMap.isEmpty()Z is abstract
  linked.map.LinkedMap.isEmpty(map.cljc)
  com.fasterxml.jackson.databind.ser.std.MapSerializer.serialize(MapSerializer.java:627)
  com.fasterxml.jackson.databind.ser.std.MapSerializer.serialize(MapSerializer.java:33)
  com.fasterxml.jackson.databind.ser.std.MapSerializer.serializeFields(MapSerializer.java:718)
  com.fasterxml.jackson.databind.ser.std.MapSerializer.serialize(MapSerializer.java:639)
  com.fasterxml.jackson.databind.ser.std.MapSerializer.serialize(MapSerializer.java:33)
  com.fasterxml.jackson.databind.ser.DefaultSerializerProvider._serialize(DefaultSerializerProvider.java:480)
  com.fasterxml.jackson.databind.ser.DefaultSerializerProvider.serializeValue(DefaultSerializerProvider.java:319)
  com.fasterxml.jackson.databind.ObjectMapper._configAndWriteValue(ObjectMapper.java:3893)
  com.fasterxml.jackson.databind.ObjectMapper.writeValueAsBytes(ObjectMapper.java:3231)
  jsonista.core$write_value_as_bytes.invokeStatic(core.clj:227)
  jsonista.core$write_value_as_bytes.invoke(core.clj:219)

[ikitommi]

also, .entrySet() is needed.

[alexandergunnarson]

Related are serialization issues with Transit, in which .keySet() is needed.

[ikitommi]

Porting all our libs to use Jsonista, and tests with linked are currently failing. Would it be possible to cut a release with tye fix?

[frankiesardo]

I've added a deps.edn to the repo if you can use that

…

On Mon, 18 Jun 2018, 14:56 Tommi Reiman, ***@***.***> wrote: Porting all our libs to use Jsonista, and tests with linked are currently failing. Would it be possible to cut a release with tye fix? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#8 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABaH0XJWxG4v1x0Rp7wDaC_YGOpSimwrks5t97GRgaJpZM4UQGIe> .

[ikitommi]

The projects are still on Leiningen and about to release new versions that depend on this. Would it be possible to cut a release?

[frankiesardo]

Sure I can, but the app is broken on the boot infrastructure and I have no idea why https://circleci.com/gh/frankiesardo/linked/63 If you can help me fix this I have an automated job that pushes it to clojars

[ikitommi]

not a boot fan and no idea why it fails. but could it be pushed manually? Off to vacation today, would like to flush all things out before that

[ro6]

@frankiesardo Any more progress on a release incorporating ikitommi@ea58ccf? I'm depending on the @ikitommi fork ([ikitommi/linked "1.3.1-alpha1"]) for now, but would love to get back on the main line.

[martinffx]

@ikitommi was this fixed in Jsonista, I'm still getting this error when setting up a swagger in compojure api?

[ikitommi]

What was the fix in jsonista? Can't recall..

[martinffx]

I excluded this lib and pulled in your copy of the jar with the fix in it.