Re: svn commit: r1130060 - in /camel/trunk: camel-core/src/main/java/org/apache/camel/impl/ camel-core/src/test/java/org/apache/camel/impl/ components/camel-jms/src/test/java/org/apache/camel/componen

2011-06-01 Thread Freeman Fang
Hi Claus, I think it's a little bit hard to determine if a collection/map is Serializable. Let's say there's a map1, a valule of this map1 is also a map2, so we should also recursively check if all the values in the map2 is Serializable, also map2 could have collection/map as values, and s

Re: svn commit: r1130060 - in /camel/trunk: camel-core/src/main/java/org/apache/camel/impl/ camel-core/src/test/java/org/apache/camel/impl/ components/camel-jms/src/test/java/org/apache/camel/componen

2011-06-02 Thread Claus Ibsen
Hi Freeman Yeah circular references is a problem. However the code only checks at 1st level. Its not recursive checking deep down. The reason is that this 1st level check is sufficient in most cases. In fact camel-core stores information in a Map that are not serializable so we need this check to