Hi,

On Wed, Oct 1, 2014 at 3:04 PM, Guillaume Smet <[email protected]>
wrote:

> Hi !
>
> We are working on the serializer projects of Wicketstuff with several
> goals in mind:
> 1/ update the FAST dependency of the current project to the latest 1.x
> 2/ create a FAST2 serializer as FAST2 is a bit different and it might
> still be a better idea to use 1.x for some use cases;
> 3/ improve the reporting of the serializer-common stuff.
>
> For 1 and 2, we are working with RĂ¼diger, the FAST developer to fix a
> few bugs and reintroduce some stuff needed by 3/. It's in good shape
> (most PR accepted, 2 PR pending).
>

Great !!!


>
> For 3/, we are working on the D3.js part and we would like to remove
> the ugly JSON generator, which does not escape the values and is not
> very reliable.
>
> We have several solutions in mind:
> 1/ add a dependency to wicket-core to this module as wicket-core has
> the JSON.org classes already imported
> 2/ import the needed JSON.org classes in the project as done in wicket-core
> 3/ use Jackson -> this is the library we use for JSON but I don't
> think it's a very good idea in this case as it introduces a big
> dependency for a very small need.
>
> So we hesitate between 1/ and 2/ and would like your advice.
>

My vote is for 1).
I doubt the project is used for anything non-Wicket related.
(Even if it is then they better start use Wicket now! :-) )


>
> Note that the value of this inspection code for serialization is
> really really underestimated: we improved a *lot* the performances of
> one of our application since we started to inspect what is really
> serialized in our pages and we fixed a few things.
>
> We are still contemplating moving to FAST *but* even if we stay with
> the default Java serializer, using FAST and this inspection code to
> understand what is serialized is really nice.
>
> This is also a heads up that we are working on this so that we don't
> do the work twice. Expect a pull request for this in the next 2 weeks.
>
> Thanks for your feedback.
>

Reply via email to