Re: {DISCUSSION] Cluster read-only mode.

2019-08-05 Thread Sergey Antonov
Maxim, I fixed issue, which you found above. Look at org.apache.ignite.internal.processors.cache.ClusterReadOnlyModeTest#testDataStreamerReadOnlyConcurrent* tests. вт, 4 июн. 2019 г. в 15:58, Maxim Muzafarov : > >> We throw CacheException on each update to read-only cluster. User code > must han

Re: {DISCUSSION] Cluster read-only mode.

2019-06-10 Thread Alexey Kukushkin
I agree with Ivan's concern - do we really need the "activation" concept in Ignite? Activation was introduced with Ignite persistence: we must prevent both the read and write operations on a cluster with persistence on until full data set is loaded (all the nodes are started). Cluster "activation"

Re: {DISCUSSION] Cluster read-only mode.

2019-06-05 Thread Павлухин Иван
Alex and all, > From a user's point of view: > Inactive cluster - deny writes, deny reads. > Read-only cluster - deny writes, allow reads. I would be great to explain what is the need to deny both reads and writes. What tasks are not possible to solve when cluster is in read-only state? ср, 5 ию

Re: {DISCUSSION] Cluster read-only mode.

2019-06-05 Thread Alex Plehanov
Ivan, >From a user's point of view: Inactive cluster - deny writes, deny reads. Read-only cluster - deny writes, allow reads. ср, 5 июн. 2019 г. в 10:44, Павлухин Иван : > Sergey, > > > On inactive cluster caches aren't started. So you can't get data from > cache. > > As a user I do not understa

Re: {DISCUSSION] Cluster read-only mode.

2019-06-05 Thread Павлухин Иван
Sergey, > On inactive cluster caches aren't started. So you can't get data from cache. As a user I do not understand what does it mean for cache to be started. IgniteCache does not provide start/stop methods. I am trying to figure out why there should be 2 modes (not from an implementation point

Re: {DISCUSSION] Cluster read-only mode.

2019-06-04 Thread Sergey Antonov
Maxim, thank you for reproducer. It looks like a bug. I will fix it! вт, 4 июн. 2019 г. в 15:58, Maxim Muzafarov : > >> We throw CacheException on each update to read-only cluster. User code > must handle CacheException correctly .You could find test on it in > ClusterReadOnlyModeTest#testDataStr

Re: {DISCUSSION] Cluster read-only mode.

2019-06-04 Thread Maxim Muzafarov
>> We throw CacheException on each update to read-only cluster. User code must >> handle CacheException correctly .You could find test on it in >> ClusterReadOnlyModeTest#testDataStreamerReadOnly() In this test, DataStreamer starts when the cluster already changes its mode, but not before. Pleas

Re: {DISCUSSION] Cluster read-only mode.

2019-06-04 Thread Sergey Antonov
Hello, Maxim! >> Do we have an IEP for this feature? No, we don't have. >> How we guarantee that all cache operations delivered (or not yet) to backups are not rejected by applied read-only request? I reused cluster activation mechanism. So enabling read-only mode generates an exchange on cluster

Re: {DISCUSSION] Cluster read-only mode.

2019-06-04 Thread Sergey Antonov
Hello, Ivan. >>What is a fundamental difference between them? On inactive cluster caches aren't started. So you can't get data from cache. вт, 4 июн. 2019 г. в 12:16, Павлухин Иван : > Sergey, Igniters, > > Sorry if my question is not very smart. > > I am trying to think about it from a perspect

Re: {DISCUSSION] Cluster read-only mode.

2019-06-04 Thread Dmitriy Pavlov
I don't remember if we agreed on it before. But it may be so, just a problem for me to remember it. Any big feature - it is quite a vague definition. My concern we'll end up with +1 inactive/uncompleted/not actual IEP. For now, we have a number of open IEPs, and no-one resolves it. I guess it shou

Re: {DISCUSSION] Cluster read-only mode.

2019-06-04 Thread Nikolay Izhikov
Hello, Dmitriy. I don't agree with you. IEP should be written for any big changes in product. We should have it for: 1. Following contributors. 2. History of decision making. 3. Easier documenting. Without IEPs our product will become a total mess. In the Apache Kafka one can't change any publ

Re: {DISCUSSION] Cluster read-only mode.

2019-06-04 Thread Dmitriy Pavlov
Folks, Sorry for offtopic. IEP is a good tool for case design is discussed before any implementation available. It is a good place to share the intent to do something and provide some details. It is a better place then mile-long emails. For now, we have code and documentation later should be upda

Re: {DISCUSSION] Cluster read-only mode.

2019-06-04 Thread Павлухин Иван
Sergey, Igniters, Sorry if my question is not very smart. I am trying to think about it from a perspective of a (newbie) user. And from the first glance it is not clear how a read-only cluster is different from a not active cluster? What is a fundamental difference between them? Can we combine tw

Re: {DISCUSSION] Cluster read-only mode.

2019-06-03 Thread Maxim Muzafarov
Sergey, Do we have an IEP for this feature? What should happen when on an active cluster with put operations if we receive a read-only state change request? How we guarantee that all cache operations delivered (or not yet) to backups are not rejected by applied read-only request? I haven't found

Re: {DISCUSSION] Cluster read-only mode.

2019-05-31 Thread Sergey Antonov
Hello, Zhenya, Maxim! Thank you for your replies! >> Should we also allow writes to the DistributedMetaStorage and if not why? Yes. DistributedMetastorage available for updates with enabled read-only mode. I added test about it to ClusterReadOnlyModeSelfTest >> What's the purpose for ignite-sys-

Re: {DISCUSSION] Cluster read-only mode.

2019-05-30 Thread Zhenya Stanilovsky
hi, Sergey. What's the purpose for ignite-sys-cache updates still be available ? thanks ! Hello Igniters! I'm working on cluster read-only mode [1] feature. In this mode cluster will be available only for read operations, all data modification operations in user caches will be rejected with Cl

Re: {DISCUSSION] Cluster read-only mode.

2019-05-30 Thread Maxim Muzafarov
Sergey, I think it will be a very useful feature for Ignite's users, my +1 here. Can you clarify what kind of updates are written to the ignite-sys-cache? Should we also allow writes to the DistributedMetaStorage and if not why? On Thu, 30 May 2019 at 19:55, Dmitriy Pavlov wrote: > > ++1 from

Re: {DISCUSSION] Cluster read-only mode.

2019-05-30 Thread Dmitriy Pavlov
++1 from my side. This mode seems to be very useful for Ignite users. чт, 30 мая 2019 г. в 19:47, Sergey Antonov : > Hello Igniters! > > I'm working on cluster read-only mode [1] feature. In this mode cluster > will be available only for read operations, all data modification > operations in user

{DISCUSSION] Cluster read-only mode.

2019-05-30 Thread Sergey Antonov
Hello Igniters! I'm working on cluster read-only mode [1] feature. In this mode cluster will be available only for read operations, all data modification operations in user caches will be rejected with ClusterReadOnlyModeCheckedException. This feature could be helpfull for maintenance works (contr