You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@polygene.apache.org by Niclas Hedhman <ni...@hedhman.org> on 2017/02/19 07:58:31 UTC
[HEADS-UP] org.json
Gang,
there was a ruling a while ago that org.json license went from CatA to
CatX, by V.P. Legal.
It means that we need to remove org.json from Polygene, and rely solely on
Jackson for now, and possibly Apache Johnzon.
Since I don't like getting dependencies into Core, I think we simply need
to move more tests out of Core and into the tests/ module.
Cheers
--
Niclas Hedhman, Software Developer
http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
Re: [HEADS-UP] org.json
Posted by Paul Merlin <pa...@apache.org>.
Niclas Hedhman a �crit :
> I haven't done much on it, and looking forward to your work reaching
> 'develop'.
>
> In CassandraEntityStoreMixin, I am using ObjectMapper to get a
> Map<String,String> for NamedAssociations to work, as yje
> JacksonDeserializer wants the format of [{ "key" : "my", "value":
> "1234-567-890" }], which I don't like.
>
> Will be good to have you back.
I'm back close to a keyboard.
It was good being away from it for more than two weeks :)
Colombia is a marvelous place, go there before it gets crowded if you
have a chance!
The changes I've made to serialization make it so that Maps are
serialized to simple JSON objects when all their keys are Strings. And
use the "array of key/value objects" format when keys are "complex",
supporting Map<SomeValue, Something>. Deserialization detects both
formats and act accordingly. No more option needed.
I still need to refactor this a bit more so that it uses the simple
object format when the map's keys can be serialized to String (e.g.
EntityReference, enums etc..).
Oh and now we'll have the ability to register custom adapters!
I'll assign all the related issues to me.
Cheers
/Paul
Re: [HEADS-UP] org.json
Posted by Niclas Hedhman <ni...@hedhman.org>.
10-4
I haven't done much on it, and looking forward to your work reaching
'develop'.
In CassandraEntityStoreMixin, I am using ObjectMapper to get a
Map<String,String> for NamedAssociations to work, as yje
JacksonDeserializer wants the format of [{ "key" : "my", "value":
"1234-567-890" }], which I don't like.
Will be good to have you back.
On Sun, Feb 19, 2017 at 10:14 PM, Paul Merlin <pa...@nosphere.org> wrote:
> Le 19 février 2017 14:06:54 GMT+01:00, Niclas Hedhman <ni...@hedhman.org>
> a écrit :
> >Let me re-phrase that; MapEntityStore uses BOTH orgjson directly AND
> >configured valueserialization.
> >
> >On Sun, Feb 19, 2017 at 9:02 PM, Niclas Hedhman <ni...@hedhman.org>
> >wrote:
> >
> >> And MapEntityStore is using org.json instead of valueserialization.
> >> Inconsistent :-(
> >>
> >> On Sun, Feb 19, 2017 at 4:06 PM, Niclas Hedhman <ni...@hedhman.org>
> >> wrote:
> >>
> >>> I should have provided the issue for this; https://issues.apache.or
> >>> g/jira/browse/POLYGENE-191
> >>>
> >>> On Sun, Feb 19, 2017 at 3:58 PM, Niclas Hedhman <ni...@hedhman.org>
> >>> wrote:
> >>>
> >>>> Gang,
> >>>> there was a ruling a while ago that org.json license went from CatA
> >to
> >>>> CatX, by V.P. Legal.
> >>>>
> >>>> It means that we need to remove org.json from Polygene, and rely
> >solely
> >>>> on Jackson for now, and possibly Apache Johnzon.
> >>>>
> >>>> Since I don't like getting dependencies into Core, I think we
> >simply
> >>>> need to move more tests out of Core and into the tests/ module.
> >>>>
> >>>> Cheers
> >>>> --
> >>>> Niclas Hedhman, Software Developer
> >>>> http://polygene.apache.org <http://zest.apache.org> - New Energy
> >for
> >>>> Java
> >>>>
> >>>
> >>>
> >>>
> >>> --
> >>> Niclas Hedhman, Software Developer
> >>> http://polygene.apache.org <http://zest.apache.org> - New Energy for
> >Java
> >>>
> >>
> >>
> >>
> >> --
> >> Niclas Hedhman, Software Developer
> >> http://polygene.apache.org <http://zest.apache.org> - New Energy for
> >Java
> >>
> >
> >
> >
> >--
> >Niclas Hedhman, Software Developer
> >http://polygene.apache.org <http://zest.apache.org> - New Energy for
> >Java
>
> Hey Niclas,
> I'm on my way back close to a keyboard.
> Before leaving I assigned this issue to me iirc, and worked on
> serialization to, among other things, remove org.json.
> There are a LOT of places where code inconsistently uses
> ValueSerialization and org.json directly. In some places it even does some
> ugly string matching to detect objects/arrays...
> I pushed my work in a serialization-3 branch two+ weeks ago. The changes
> use javax.json and Apache Johnzon. It is almost done and solve all of the
> current inconsistencies but need a bit more love. I'll get back at it next
> week.
>
> --
> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma
> brièveté.
--
Niclas Hedhman, Software Developer
http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
Re: [HEADS-UP] org.json
Posted by Paul Merlin <pa...@nosphere.org>.
Le 19 février 2017 14:06:54 GMT+01:00, Niclas Hedhman <ni...@hedhman.org> a écrit :
>Let me re-phrase that; MapEntityStore uses BOTH orgjson directly AND
>configured valueserialization.
>
>On Sun, Feb 19, 2017 at 9:02 PM, Niclas Hedhman <ni...@hedhman.org>
>wrote:
>
>> And MapEntityStore is using org.json instead of valueserialization.
>> Inconsistent :-(
>>
>> On Sun, Feb 19, 2017 at 4:06 PM, Niclas Hedhman <ni...@hedhman.org>
>> wrote:
>>
>>> I should have provided the issue for this; https://issues.apache.or
>>> g/jira/browse/POLYGENE-191
>>>
>>> On Sun, Feb 19, 2017 at 3:58 PM, Niclas Hedhman <ni...@hedhman.org>
>>> wrote:
>>>
>>>> Gang,
>>>> there was a ruling a while ago that org.json license went from CatA
>to
>>>> CatX, by V.P. Legal.
>>>>
>>>> It means that we need to remove org.json from Polygene, and rely
>solely
>>>> on Jackson for now, and possibly Apache Johnzon.
>>>>
>>>> Since I don't like getting dependencies into Core, I think we
>simply
>>>> need to move more tests out of Core and into the tests/ module.
>>>>
>>>> Cheers
>>>> --
>>>> Niclas Hedhman, Software Developer
>>>> http://polygene.apache.org <http://zest.apache.org> - New Energy
>for
>>>> Java
>>>>
>>>
>>>
>>>
>>> --
>>> Niclas Hedhman, Software Developer
>>> http://polygene.apache.org <http://zest.apache.org> - New Energy for
>Java
>>>
>>
>>
>>
>> --
>> Niclas Hedhman, Software Developer
>> http://polygene.apache.org <http://zest.apache.org> - New Energy for
>Java
>>
>
>
>
>--
>Niclas Hedhman, Software Developer
>http://polygene.apache.org <http://zest.apache.org> - New Energy for
>Java
Hey Niclas,
I'm on my way back close to a keyboard.
Before leaving I assigned this issue to me iirc, and worked on serialization to, among other things, remove org.json.
There are a LOT of places where code inconsistently uses ValueSerialization and org.json directly. In some places it even does some ugly string matching to detect objects/arrays...
I pushed my work in a serialization-3 branch two+ weeks ago. The changes use javax.json and Apache Johnzon. It is almost done and solve all of the current inconsistencies but need a bit more love. I'll get back at it next week.
--
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
Re: [HEADS-UP] org.json
Posted by Niclas Hedhman <ni...@hedhman.org>.
Let me re-phrase that; MapEntityStore uses BOTH orgjson directly AND
configured valueserialization.
On Sun, Feb 19, 2017 at 9:02 PM, Niclas Hedhman <ni...@hedhman.org> wrote:
> And MapEntityStore is using org.json instead of valueserialization.
> Inconsistent :-(
>
> On Sun, Feb 19, 2017 at 4:06 PM, Niclas Hedhman <ni...@hedhman.org>
> wrote:
>
>> I should have provided the issue for this; https://issues.apache.or
>> g/jira/browse/POLYGENE-191
>>
>> On Sun, Feb 19, 2017 at 3:58 PM, Niclas Hedhman <ni...@hedhman.org>
>> wrote:
>>
>>> Gang,
>>> there was a ruling a while ago that org.json license went from CatA to
>>> CatX, by V.P. Legal.
>>>
>>> It means that we need to remove org.json from Polygene, and rely solely
>>> on Jackson for now, and possibly Apache Johnzon.
>>>
>>> Since I don't like getting dependencies into Core, I think we simply
>>> need to move more tests out of Core and into the tests/ module.
>>>
>>> Cheers
>>> --
>>> Niclas Hedhman, Software Developer
>>> http://polygene.apache.org <http://zest.apache.org> - New Energy for
>>> Java
>>>
>>
>>
>>
>> --
>> Niclas Hedhman, Software Developer
>> http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
>>
>
>
>
> --
> Niclas Hedhman, Software Developer
> http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
>
--
Niclas Hedhman, Software Developer
http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
Re: [HEADS-UP] org.json
Posted by Niclas Hedhman <ni...@hedhman.org>.
And MapEntityStore is using org.json instead of valueserialization.
Inconsistent :-(
On Sun, Feb 19, 2017 at 4:06 PM, Niclas Hedhman <ni...@hedhman.org> wrote:
> I should have provided the issue for this; https://issues.apache.
> org/jira/browse/POLYGENE-191
>
> On Sun, Feb 19, 2017 at 3:58 PM, Niclas Hedhman <ni...@hedhman.org>
> wrote:
>
>> Gang,
>> there was a ruling a while ago that org.json license went from CatA to
>> CatX, by V.P. Legal.
>>
>> It means that we need to remove org.json from Polygene, and rely solely
>> on Jackson for now, and possibly Apache Johnzon.
>>
>> Since I don't like getting dependencies into Core, I think we simply need
>> to move more tests out of Core and into the tests/ module.
>>
>> Cheers
>> --
>> Niclas Hedhman, Software Developer
>> http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
>>
>
>
>
> --
> Niclas Hedhman, Software Developer
> http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
>
--
Niclas Hedhman, Software Developer
http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
Re: [HEADS-UP] org.json
Posted by Niclas Hedhman <ni...@hedhman.org>.
I should have provided the issue for this;
https://issues.apache.org/jira/browse/POLYGENE-191
On Sun, Feb 19, 2017 at 3:58 PM, Niclas Hedhman <ni...@hedhman.org> wrote:
> Gang,
> there was a ruling a while ago that org.json license went from CatA to
> CatX, by V.P. Legal.
>
> It means that we need to remove org.json from Polygene, and rely solely on
> Jackson for now, and possibly Apache Johnzon.
>
> Since I don't like getting dependencies into Core, I think we simply need
> to move more tests out of Core and into the tests/ module.
>
> Cheers
> --
> Niclas Hedhman, Software Developer
> http://polygene.apache.org <http://zest.apache.org> - New Energy for Java
>
--
Niclas Hedhman, Software Developer
http://polygene.apache.org <http://zest.apache.org> - New Energy for Java