search

eclipse-emfcloud / emfjson-jackson

emfjson-jackson
Other
16 stars 15 forks source link

Make EMap serializer more configurable for NPE fix #33

Closed vhemery closed 2 years ago

vhemery commented 2 years ago

Deprecate EMapStringSerializer (keep it for external usages). Use EMapSerializer instead, more generic and delegates to Jackson MapSerializer for configurability. Configure by default with NullKeySerializer using '' for null EMap keys. (hence no more NPE)

closes https://github.com/eclipse-emfcloud/emfjson-jackson/issues/32

vhemery commented 2 years ago

The PR looks stuck waiting for an executor... https://ci.eclipse.org/emfcloud/job/eclipse-emfcloud/job/emfjson-jackson/job/PR-33/1/console While I can't see any busy executor on https://ci.eclipse.org/emfcloud/ Is this normal ? If not, can someone check whether any other execution starts and eventually restart the job ? Thanks in advance.

martin-fleck-at commented 2 years ago

@vhemery Restarting the job did not solve the issue but we are looking into it!

martin-fleck-at commented 2 years ago

@vhemery A job should be running now!