I do not like this change - we intentionally separated a few properties in AtomicConfiguration that make sense for Atomics, there is not need to get back to cache configuration again. In my understanding, we only need to add groupName to Atomics and Collection configuration.
Thoughts? 2017-06-01 16:32 GMT+03:00 Ilya Lantukh <ilant...@gridgain.com>: > This is how I see API to create data structures in user-defined caches: > https://github.com/apache/ignite/pull/2058 > > On Thu, Jun 1, 2017 at 2:56 PM, Ilya Lantukh <ilant...@gridgain.com> > wrote: > >> Hi all, >> >> There are other problems with current data structures implementation, >> which are related to new persistence mechanics. For example, take a look at >> this ticket: https://issues.apache.org/jira/browse/IGNITE-5324 >> I think the best approach is to store data structures not in special >> system cache, but in user defined ones. All API methods to access data >> structures will have cacheName parameter, and unique identifier will be a >> pair (cacheName, dsName). In this case we won't need a single place to >> store all data structure metadata. >> What do you think? >> >> >> On Thu, Jun 1, 2017 at 1:32 PM, Mikhail Cherkasov < >> mcherka...@gridgain.com> wrote: >> >>> On Thu, Jun 1, 2017 at 1:29 AM, Dmitriy Setrakyan <dsetrak...@apache.org >>> > >>> wrote: >>> >>> > Won't it be confusing from a user stand point to have multiple data >>> > structures with the same name? >>> > >>> >>> I won't change this, optionally I can allow to have the same name for >>> different data structures' types, >>> but it's better to keep single namespace for all data structures as it's >>> implemented now. >>> >>> >>> > Also, what is the performance impact of this >>> > change? >>> > >>> >>> I'm working on a benchmark. >>> >>> >>> -- >>> Thanks, >>> Mikhail. >>> >> >> >> >> -- >> Best regards, >> Ilya >> > > > > -- > Best regards, > Ilya >
