NULL strings in REST-HTTP

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

NULL strings in REST-HTTP

Alexey Kuznetsov
Hi,

I would like to change serialization of null strings in our
ignite-rest-http module.

Current serialization null -> "" (empty string).
Proposed serialization null -> null.

This will make things more straightforward.

Thoughts?

--
Alexey Kuznetsov
Reply | Threaded
Open this post in threaded view
|

Re: NULL strings in REST-HTTP

Sergey Kozlov
Hi Alexey.

I vote for your fix. Definitely it's wrong to return the empty string
instead of null.

On Mon, Apr 17, 2017 at 1:30 PM, Alexey Kuznetsov <[hidden email]>
wrote:

> Hi,
>
> I would like to change serialization of null strings in our
> ignite-rest-http module.
>
> Current serialization null -> "" (empty string).
> Proposed serialization null -> null.
>
> This will make things more straightforward.
>
> Thoughts?
>
> --
> Alexey Kuznetsov
>



--
Sergey Kozlov
GridGain Systems
www.gridgain.com
Reply | Threaded
Open this post in threaded view
|

Re: NULL strings in REST-HTTP

Vladimir Ozerov
+1

On Mon, Apr 17, 2017 at 1:42 PM, Sergey Kozlov <[hidden email]> wrote:

> Hi Alexey.
>
> I vote for your fix. Definitely it's wrong to return the empty string
> instead of null.
>
> On Mon, Apr 17, 2017 at 1:30 PM, Alexey Kuznetsov <[hidden email]>
> wrote:
>
> > Hi,
> >
> > I would like to change serialization of null strings in our
> > ignite-rest-http module.
> >
> > Current serialization null -> "" (empty string).
> > Proposed serialization null -> null.
> >
> > This will make things more straightforward.
> >
> > Thoughts?
> >
> > --
> > Alexey Kuznetsov
> >
>
>
>
> --
> Sergey Kozlov
> GridGain Systems
> www.gridgain.com
>