Dmitry,

I don't think we need to show warnings until there is an attempt to create
a local cache. This is a very rare use case, not worth it. We should
properly document it though.

-Val

On Fri, Feb 2, 2018 at 12:18 AM, Dmitry Karachentsev <
dkarachent...@gridgain.com> wrote:

> Sounds pretty reasonable for me, so fix will entail just in proper
> exception.
> Maybe it worth to show warning that says that's no data region configured
> and it's not possible to create LOCAL cache?
>
> Thanks!
>
> 02.02.2018 00:08, Valentin Kulichenko пишет:
>
> I meant "they should *explicitly* provide data region configuration", of
>> course.
>>
>> -Val
>>
>> On Thu, Feb 1, 2018 at 10:58 AM, Valentin Kulichenko <
>> valentin.kuliche...@gmail.com> wrote:
>>
>> Agree with Mike. I don't think it's a good idea to implicitly create data
>>> regions on client node. If one wants to have a local there, they should
>>> implicitly provide data region configuration (default or otherwise). If
>>> data region is not configured, then we should throw proper exception
>>> instead of NPE.
>>>
>>> -Val
>>>
>>> On Thu, Feb 1, 2018 at 10:54 AM, Michael Cherkasov <
>>> michael.cherka...@gmail.com> wrote:
>>>
>>> Hi Dmitry,
>>>>
>>>> I think we should make a user to explicitly create data region on a
>>>> client.
>>>>
>>>> Client nodes aren't supposed to be used for data storing, so if someone
>>>> what to use a local cache on a client node, let's make him/her create
>>>> data
>>>> region explicitly. Just to make sure that user knows what he/she is
>>>> doing.
>>>>
>>>> Thanks,
>>>> Mike.
>>>>
>>>>
>>>> 2018-02-01 7:36 GMT-08:00 Dmitry Karachentsev <
>>>> dkarachent...@gridgain.com
>>>>
>>>>> :
>>>>> Hello everyone!
>>>>>
>>>>> We have an erroneous use case when client tries to create LOCAL cache,
>>>>>
>>>> but
>>>>
>>>>> by default it does not initiates default data region. So client gets
>>>>>
>>>> NPE.
>>>>
>>>>> [1]
>>>>>
>>>>> I think it should be a lazy data region initialization on client. Do
>>>>> you
>>>>> have any concerns about this approach or other proposals?
>>>>>
>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-7572
>>>>>
>>>>> Thanks!
>>>>>
>>>>>
>>>>>
>>>
>

Reply via email to