Atri, Currently if user start data streaming on a client for already started cache it doesn't yield in internal cache delegate creation on the client what saves memory which is very important for client mode. With proposed change client will always create cache delegate even if it is not needed. This is why I am -1 for this change.
Vladimir. On Wed, Jul 1, 2015 at 10:06 AM, Atri Sharma <[email protected]> wrote: > I agree with that but essentially this is same as manually creating before > data streaming. I agree with your point of cache delegation but there might > be usecases where client is fine with this (low number of caches) but may > need functionality to automate cache creation when streaming. > > I feel that we should implement this and document memory implications of > this so it can be used for needed use cases. > > Thoughts? > > On Wed, Jul 1, 2015 at 12:27 PM, Vladimir Ozerov <[email protected]> > wrote: > > > Atri, > > > > I thought a little bit more about this. Calling "getOrCreateCache" is > not a > > good idea when streamer is called from a client node because it will lead > > to creating a cache delegate on this node. But with current approach this > > is not necessary and client node doesn't spent memory on it. > > > > Therefore, I think we should not implement this provided that workaround > > exists: user could call "Ignite.getOrCreateCache" manually before > starting > > data streaming in case it is really needed. > > > > Vladimir. > > > > On Tue, Jun 30, 2015 at 7:20 PM, Atri Sharma <[email protected]> > wrote: > > > > > I have created IGNITE -1066 for this. > > > > > > Please see and comment. > > > > > > On Tue, Jun 30, 2015 at 12:36 PM, Atri Sharma <[email protected]> > > wrote: > > > > > > > I think it is a good idea but I wonder if it makes sense to also > notify > > > to > > > > the user somehow that a cache was created or not by this function. > > > > > > > > Why I feel that is necessary is because if we are creating caches > most > > of > > > > the time in the function to the user opaquely, it may lead to user > > > > questioning the performance degradation (minor though it may be) > since > > > > creating a cache will be a tad more expensive than just getting it. > It > > > > would be worthwhile to atleast log so we can identify performance > > issues > > > > later on. > > > > > > > > If you are fine I can make ticket and assign it to myself. > > > > > > > > On Tue, Jun 30, 2015 at 12:01 PM, Vladimir Ozerov < > > [email protected]> > > > > wrote: > > > > > > > >> Igniters, > > > >> > > > >> Consider the following use case. > > > >> 1) User configured cache template, but has never accessed it > > explicitly > > > >> yet; > > > >> 2) User calls Ignite.dataStreamer([cacheName]) - exception is thrown > > > >> because cache is not started. > > > >> > > > >> I have a feeling that data streamer must have "getOrCreateCache" > > > semantics > > > >> so that user do not have to pre-start cache explicitly to start > > loading > > > >> data to it. > > > >> > > > >> Thoughts? > > > >> > > > >> Vladimir. > > > >> > > > > > > > > > > > > > > > > -- > > > > Regards, > > > > > > > > Atri > > > > *l'apprenant* > > > > > > > > > > > > > > > > -- > > > Regards, > > > > > > Atri > > > *l'apprenant* > > > > > > > > > -- > Regards, > > Atri > *l'apprenant* >
