Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
MIME-Version: 1.0
In-Reply-To: <CAK2_iMNnUHbSFN9+AOdT-mzrBzUMH=5uMmD32NzeHbX+iEKrkg@mail.gmail.com>
References: <CAK2_iMMT=TwVtCgWNhnOiLCtjp--x9h2DTiLDReFB8=5S1bF_g@mail.gmail.com>
 <CABuYRcrq_LomWVP8XqJWgdndJtONhGzbx9AS5NggnR6tt5bU2w@mail.gmail.com>
 <CA+0=VoWxiw5T2F=hJDgOgbzCO6_Befu7UBwj1tnCRyU1eFtyUg@mail.gmail.com> <CAK2_iMNnUHbSFN9+AOdT-mzrBzUMH=5uMmD32NzeHbX+iEKrkg@mail.gmail.com>
From: Valentin Kulichenko <valentin.kulichenko@gmail.com>
Date: Wed, 25 Jan 2017 11:45:52 -0800
Message-ID: <CABuYRcpjkwv3DpCCMRa55jXT8X3u5cqzd91XPjnLHr2zZ+tY=A@mail.gmail.com>
Subject: Re: Write IgniteUuid with BinaryMarshaller
To: dev@ignite.apache.org
Content-Type: multipart/alternative; boundary=94eb2c060070c7cb380546f07c59
archived-at: Wed, 25 Jan 2017 19:46:28 -0000

--94eb2c060070c7cb380546f07c59
Content-Type: text/plain; charset=UTF-8

Pavel,

Makes sense, I think this is the best way.

-Val

On Wed, Jan 25, 2017 at 12:27 AM, Pavel Tupitsyn <ptupitsyn@apache.org>
wrote:

> Val,
>
> Not sure about UUID.
> It is a very common thing and writing it as an object will introduce a lot
> of overhead (17 bytes now vs 40+ bytes for object).
>
> May be we should even add a special case for IgniteUuid?
>
> On Wed, Jan 25, 2017 at 1:52 AM, Dmitriy Setrakyan <dsetrakyan@apache.org>
> wrote:
>
> > Why not add IgniteUuid to BinaryContext.BINARYLIZABLE_SYS_CLSS?
> >
> > On Tue, Jan 24, 2017 at 10:55 AM, Valentin Kulichenko <
> > valentin.kulichenko@gmail.com> wrote:
> >
> > > I actually think that both UUID and IgniteUuid should be plain
> > serializable
> > > classes, I don't see any reason for special processing for them.
> > >
> > > Currently we have the following:
> > >
> > >    - UUID is Serializable, but we have special serialization logic for
> it
> > >    internally in the marshaller.
> > >    - IgniteUuid is Externalizable.
> > >
> > > This is indeed inconsistent and confusing.
> > >
> > > -Val
> > >
> > > On Tue, Jan 24, 2017 at 9:21 AM, Pavel Tupitsyn <ptupitsyn@apache.org>
> > > wrote:
> > >
> > > > Igniters,
> > > >
> > > > Currently IgniteUuid is written with OptimizedMarshaller
> > > > (it is not included in BinaryContext.BINARYLIZABLE_SYS_CLSS).
> > > >
> > > > This prevents it from being read on other platforms (.NET, C++).
> > > >
> > > > Is there any reason for this? Can we fix this in 2.0?
> > > >
> > > > Thanks,
> > > > Pavel
> > > >
> > >
> >
>

--94eb2c060070c7cb380546f07c59--