Skip to content

Conversation

@daveclay
Copy link
Contributor

Hi,

I just pulled together the code from this ticket:

http://jira.codehaus.org/browse/JACKSON-211

and moved it into the new Scala module project. I've only tested the serializers today thus far; I've also made it work in RESTEasy with a custom Provider to register the ObjectMapper with the new ScalaModule (not sure where to put that little class yet. It depends on jaxrs, and gets pulled in automagically via RESTEasy/Jaxrs).

I'm using ScalaTest for my test cases. I'm new to maven, so I'm not sure the process to get general access to the ScalaTest libraries via a repository. I just installed them locally from http://scalatest.org/.

I'll see what I can do with more testing today and tomorrow.

  • Dave

@cowtowncoder
Copy link
Member

Ok, one issue is that scalatest is not available from central maven repo. I guess it should be possible to add secondary maven repository using scm tag, will try to figure it out (but why don't they publish it to central?)

@cowtowncoder
Copy link
Member

Some issues:

  • Looks like package 'com.fasterxml.jackson.module.scala.ser' is missing (and specifically class ScalaSerializers)
  • Serializers and Deserializers implementations should not extend matching factories; I can help rewrite this part (extending would cause issues with delegation)

@cowtowncoder
Copy link
Member

Ah. Actually, serializers are written in scala, so they'd have to be first compiled...

@daveclay
Copy link
Contributor Author

I'm not all that familiar with maven, but I believe I have it compiling scala first, then java. Seems to compile for me...

I updated the pom to reference the scalatest.org maven repo, according to the info at:
http://www.scalatest.org/download

I'll take a lok at the serializers/deserializers graph. I did very little to the existing code to get it into the module api. Hopefully it shouldn't be much work to remove the old implementations.

@cowtowncoder
Copy link
Member

Ok: merged this into master; more work is needed to make things work but this is a good start.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants