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: <CAFyTW-iiUz-8oMfDphjrF72uXDfDFtq_mfMUvQJAMtJyoQpdcQ@mail.gmail.com>
References: <CAFyTW-iLutKEX0RGrBLhni7GjdWDTQ31dub6Nx915qP97o8epw@mail.gmail.com>
 <CAGt6o8moAq8OR8taTJSkTDFR3Jc7mEw0XdZO8oSUcNW7Gyp+hw@mail.gmail.com>
 <5CA257DA-AE41-4C09-BAF5-074CC0270DC5@apache.org> <CAFyTW-hb-1rKfS66FTXm=dcf_aaJFUdDJ=2n0O6xtQT+ur2BTw@mail.gmail.com>
 <9ED94A13-2DE7-4B28-9090-67C2AF0B2ACB@apache.org> <CAFyTW-iiUz-8oMfDphjrF72uXDfDFtq_mfMUvQJAMtJyoQpdcQ@mail.gmail.com>
From: Vyacheslav Daradur <daradurvs@gmail.com>
Date: Wed, 1 Mar 2017 11:11:58 +0300
Message-ID: <CAFyTW-gw_5_m7OQ=BjeUuWmhURCimjY+ZwPRv8HDydGCPP1QaQ@mail.gmail.com>
Subject: Re: IGNITE-3422 - ready for review
To: dev@ignite.apache.org
Content-Type: multipart/alternative; boundary=001a11416666d8f4510549a6ddf7
archived-at: Wed, 01 Mar 2017 08:12:08 -0000

--001a11416666d8f4510549a6ddf7
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Guys, any thoughts?

2017-02-23 13:22 GMT+03:00 Vyacheslav Daradur <daradurvs@gmail.com>:

> I think it makes a sense to define factory in IgniteConfiguration, as the
> Map<Class, InstanceFactory> which will be ised when deserializing.
>
> Interface example:
>
> public intterface InstanceFactory {
>     newInstance(Class clazz, BinaryReader reader);
> }
>
> P.S.
> About "constructor(BinraryReader reader)" - I think there is no sense,
> because it just duplicate readObject method and we can't manage a referen=
ce
> of new instance.
> How about singlton-deserialization?
>
>
>
> 2017-02-23 10:16 GMT+03:00 Denis Magda <dmagda@apache.org>:
>
>> In my understanding the goal is well-defined in the ticket. At the same
>> time we have a similar task prepared by Pavel for .NET:
>> https://issues.apache.org/jira/browse/IGNITE-3102 <
>> https://issues.apache.org/jira/browse/IGNITE-3102>
>>
>> So, what we need to agree on is how to proceed with the implementation.
>> Let us think this over and propose design.
>>
>> =E2=80=94
>> Denis
>>
>> > On Feb 22, 2017, at 2:41 AM, Vyacheslav Daradur <daradurvs@gmail.com>
>> wrote:
>> >
>> > Guys, let's discuss a goal of this task.
>> >
>> > I need the task specification.
>> >
>> >
>> > 2017-02-22 2:00 GMT+03:00 Denis Magda <dmagda@apache.org <mailto:
>> dmagda@apache.org>>:
>> > Replied.
>> >
>> > =E2=80=94
>> > Denis
>> >
>> > > On Feb 20, 2017, at 3:08 AM, Vladimir Ozerov <vozerov@gridgain.com
>> <mailto:vozerov@gridgain.com>> wrote:
>> > >
>> > > Hi Vyacheslav,
>> > >
>> > > Thank you for contribution. I reviewed implementation again and now =
I
>> am in
>> > > doubts whether our product would really benefit from it or not. See =
my
>> > > comments in the ticket. I'de prefer Denis Magda to chime in and give
>> his
>> > > feedback first.
>> > >
>> > > Vladimir.
>> > >
>> > > On Wed, Feb 15, 2017 at 5:00 PM, Vyacheslav Daradur <
>> daradurvs@gmail.com <mailto:daradurvs@gmail.com>>
>> > > wrote:
>> > >
>> > >> Hello everyone.
>> > >>
>> > >> Please, review implemented solution.
>> > >>
>> > >> https://issues.apache.org/jira/browse/IGNITE-3422 <
>> https://issues.apache.org/jira/browse/IGNITE-3422> - No way to control
>> > >> object initialization during deserialization/unmarshalling
>> > >>
>> > >> ci.tests <http://ci.ignite.apache.org/viewQueued.html?itemId=3D4610=
54
>> <http://ci.ignite.apache.org/viewQueued.html?itemId=3D461054>>
>> > >>
>> >
>> >
>>
>>
>

--001a11416666d8f4510549a6ddf7--