mashupbots / socko

A Scala web server powered by Netty networking and AKKA processing.
Other
255 stars 51 forks source link

ReflectUtil don't check some scala class will contain "$" in classname #72

Closed marsxu1984 closed 10 years ago

marsxu1984 commented 10 years ago

I find this bug when I use socko-rest framework. Some time the reflect will not success, because classname has "$" tag, and won't load into jvm with package full namespace?

veebs commented 10 years ago

Can you please give us an example?

Also, have you instancing RestRegistry in a single thread?

This module relies on Scala reflection. Reflection is tagged as experimental in Scala 2.10. There is also an issue with thread safety. For unit tests to work, you have to execute them serially (in sbt set parallelExecution in Test := false). Also, instancing RestRegistry in Step #5 must be performed in a single thead.
veebs commented 10 years ago

See #78