No, I can not edit other pages. Screenshots menu: https://www.dropbox.com/s/hbhuk3tci7k77qr/%D0%A1%D0%BA%D1%80%D0%B8%D0%BD%D1%88%D0%BE%D1%82%202017-03-13%2019.51.30.png?dl=0 https://www.dropbox.com/s/43nr384icfb6xwp/%D0%A1%D0%BA%D1%80%D0%B8%D0%BD%D1%88%D0%BE%D1%82%202017-03-13%2019.50.24.png?dl=0
> 13 марта 2017 г., в 19:44, Denis Magda <[email protected]> написал(а): > > Weird, can you edit any other page? I can’t find any configuration parameters > that should prevent you from editing. > > — > Denis > >> On Mar 13, 2017, at 9:14 AM, Kozlov Maxim <[email protected]> wrote: >> >> Denis, I can not edit this page. >> >> confluence username: dreamx >> >>> 13 марта 2017 г., в 19:01, Kozlov Maxim <[email protected] >>> <mailto:[email protected]>> написал(а): >>> >>> Denis, >>> >>> ok >>> >>>> 13 марта 2017 г., в 18:59, Denis Magda <[email protected] >>>> <mailto:[email protected]> <mailto:[email protected] >>>> <mailto:[email protected]>>> написал(а): >>>> >>>> Maxim, >>>> >>>> Please update Apache Ignite 2.0 migration guide: >>>> https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide >>>> >>>> <https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide><https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide >>>> >>>> <https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide>> >>>> >>>> <https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide >>>> >>>> <https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide><https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide >>>> >>>> <https://cwiki.apache.org/confluence/display/IGNITE/Apache+Ignite+2.0+Migration+Guide>>> >>>> >>>> You need to say that the parameter has been discontinued and the users can >>>> use CacheAtomicWriteOrderMode.PRIMARY instead. >>>> >>>> Agreed? >>>> >>>> — >>>> Denis >>>> >>>>> On Mar 10, 2017, at 12:06 AM, Kozlov Maxim <[email protected] >>>>> <mailto:[email protected]> <mailto:[email protected] >>>>> <mailto:[email protected]>>> wrote: >>>>> >>>>> Andrey, Alexey, please review >>>>> PR - https://github.com/apache/ignite/pull/1521 >>>>> <https://github.com/apache/ignite/pull/1521> >>>>> <https://github.com/apache/ignite/pull/1521 >>>>> <https://github.com/apache/ignite/pull/1521>> >>>>> <https://github.com/apache/ignite/pull/1521 >>>>> <https://github.com/apache/ignite/pull/1521> >>>>> <https://github.com/apache/ignite/pull/1521 >>>>> <https://github.com/apache/ignite/pull/1521>>> >>>>> tests - >>>>> http://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests_RunAll&branch_IgniteTests=pull%2F1521%2Fhead&tab=buildTypeStatusDiv >>>>> >>>>> <http://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests_RunAll&branch_IgniteTests=pull%2F1521%2Fhead&tab=buildTypeStatusDiv><http://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests_RunAll&branch_IgniteTests=pull%2F1521%2Fhead&tab=buildTypeStatusDiv >>>>> >>>>> <http://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests_RunAll&branch_IgniteTests=pull%2F1521%2Fhead&tab=buildTypeStatusDiv>> >>>>> >>>>> <http://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests_RunAll&branch_IgniteTests=pull/1521/head&tab=buildTypeStatusDiv >>>>> >>>>> <http://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests_RunAll&branch_IgniteTests=pull/1521/head&tab=buildTypeStatusDiv><http://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests_RunAll&branch_IgniteTests=pull/1521/head&tab=buildTypeStatusDiv >>>>> >>>>> <http://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests_RunAll&branch_IgniteTests=pull/1521/head&tab=buildTypeStatusDiv>>> >>>>> >>>>>> 7 марта 2017 г., в 14:15, Andrey Gura <[email protected] >>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>> <mailto:[email protected]>>> написал(а): >>>>>> >>>>>> Maxim, >>>>>> >>>>>> all GridClockSyncProcessor related code should be remove (objects, >>>>>> messages, etc) >>>>>> >>>>>> On Tue, Mar 7, 2017 at 12:23 PM, Kozlov Maxim <[email protected] >>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>> <mailto:[email protected]>>> wrote: >>>>>>> Andrey, or better remove GridTimeSyncProcessorSelfTest class? >>>>>>> >>>>>>>> 7 марта 2017 г., в 12:21, Kozlov Maxim <[email protected] >>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>> <mailto:[email protected]>>> написал(а): >>>>>>>> >>>>>>>> Andrey, in GridTimeSyncProcessorSelfTest class methods: testTimeSync() >>>>>>>> and testTimeSyncChangeCoordinator() also removed? >>>>>>>> >>>>>>>> >>>>>>>>> 6 марта 2017 г., в 18:42, Andrey Gura <[email protected] >>>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>>> <mailto:[email protected]>>> написал(а): >>>>>>>>> >>>>>>>>> Maxim, >>>>>>>>> >>>>>>>>> About SER_VER_COMPARATOR. You can use code branch that executes when >>>>>>>>> times are equal: >>>>>>>>> >>>>>>>>> int nodeOrder1 = ver1.nodeOrder(); >>>>>>>>> int nodeOrder2 = ver2.nodeOrder(); >>>>>>>>> >>>>>>>>> if (nodeOrder1 == nodeOrder2) { >>>>>>>>> long order1 = ver1.order(); >>>>>>>>> long order2 = ver2.order(); >>>>>>>>> >>>>>>>>> assert order1 != order2; >>>>>>>>> >>>>>>>>> return order1 > order2 ? 1 : -1; >>>>>>>>> } >>>>>>>>> else >>>>>>>>> return nodeOrder1 > nodeOrder2 ? 1 : -1; >>>>>>>>> >>>>>>>>> On Mon, Mar 6, 2017 at 6:32 PM, Alexey Goncharuk >>>>>>>>> <[email protected] <mailto:[email protected]> >>>>>>>>> <mailto:[email protected] >>>>>>>>> <mailto:[email protected]>>> wrote: >>>>>>>>>> Maxim, >>>>>>>>>> >>>>>>>>>> Global time comparison is only needed for CLOCK mode, so you should >>>>>>>>>> modify >>>>>>>>>> the code as if ignoreTime is always true. >>>>>>>>>> >>>>>>>>>> 2017-03-06 18:13 GMT+03:00 Kozlov Maxim <[email protected] >>>>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>>>> <mailto:[email protected]>>>: >>>>>>>>>> >>>>>>>>>>> ok, >>>>>>>>>>> in GridCacheAtomicVersionComparator class, method >>>>>>>>>>> compare(GridCacheVersion one, GridCacheVersion other, boolean >>>>>>>>>>> ignoreTime) >>>>>>>>>>> if (globalTime == otherGlobalTime || ignoreTime) { // => if >>>>>>>>>>> (ignoreTime) { >>>>>>>>>>> ..... >>>>>>>>>>> } >>>>>>>>>>> else >>>>>>>>>>> return globalTime > otherGlobalTime ? 1 : -1; // => return -1; >>>>>>>>>>> >>>>>>>>>>> and, >>>>>>>>>>> GridCacheMvcc class, >>>>>>>>>>> SER_VER_COMPARATOR is comparator by globalTime var. His remove and >>>>>>>>>>> remove >>>>>>>>>>> compareSerializableVersion? >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>>> 6 марта 2017 г., в 16:51, Andrey Gura <[email protected] >>>>>>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>>>>>> <mailto:[email protected]>>> написал(а): >>>>>>>>>>>> >>>>>>>>>>>> Maxim, >>>>>>>>>>>> >>>>>>>>>>>> updateTime() method should be removed. >>>>>>>>>>>> >>>>>>>>>>>> On Mon, Mar 6, 2017 at 12:12 PM, Kozlov Maxim >>>>>>>>>>>> <[email protected] <mailto:[email protected]> >>>>>>>>>>>> <mailto:[email protected] <mailto:[email protected]>>> >>>>>>>>>>> wrote: >>>>>>>>>>>>> In CacheEntryImplEx class use ver.globalTime() in >>>>>>>>>>>>> >>>>>>>>>>>>> @Override public long updateTime() { >>>>>>>>>>>>> return ver.globalTime(); >>>>>>>>>>>>> } >>>>>>>>>>>>> >>>>>>>>>>>>> Than is better to replace this variable? >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>> 3 марта 2017 г., в 19:19, Andrey Gura <[email protected] >>>>>>>>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>>>>>>>> <mailto:[email protected]>>> написал(а): >>>>>>>>>>>>>> >>>>>>>>>>>>>> Maxim, >>>>>>>>>>>>>> >>>>>>>>>>>>>> I think the next implementation will be good enough: >>>>>>>>>>>>>> >>>>>>>>>>>>>> public IgniteUuid asGridUuid() { >>>>>>>>>>>>>> return new IgniteUuid(new UUID(nodeOrderDrId, topVer), order); >>>>>>>>>>>>>> } >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> Serialization/deserialization of GridCacheVersion.globalTime >>>>>>>>>>>>>> field >>>>>>>>>>>>>> should be removed. >>>>>>>>>>>>>> >>>>>>>>>>>>>> On Fri, Mar 3, 2017 at 5:57 PM, Kozlov Maxim >>>>>>>>>>>>>> <[email protected] <mailto:[email protected]> >>>>>>>>>>>>>> <mailto:[email protected] <mailto:[email protected]>>> >>>>>>>>>>> wrote: >>>>>>>>>>>>>>> Alexey, >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> public IgniteUuid asGridUuid() { >>>>>>>>>>>>>>> return new IgniteUuid(new UUID(nodeOrderDrId << 32, topVer << >>>>>>>>>>>>>>> 32), >>>>>>>>>>> order); >>>>>>>>>>>>>>> } >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> So you want to change or not? >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> And >>>>>>>>>>>>>>> - GridCacheVersion.writeTo(ByteBuffer buf, MessageWriter writer) >>>>>>>>>>>>>>> - GridCacheVersion.readFrom(ByteBuffer buf, MessageReader >>>>>>>>>>>>>>> reader) >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> use globalTime variable, must be removed case 0: (in both >>>>>>>>>>>>>>> methods) or >>>>>>>>>>> replace globalTime? >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> 2 марта 2017 г., в 16:58, Andrey Gura <[email protected] >>>>>>>>>>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>>>>>>>>>> <mailto:[email protected]>>> написал(а): >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> +1 >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Removing of asGridUuid() method can lead to much code changes >>>>>>>>>>>>>>>> but it >>>>>>>>>>>>>>>> should be avoided on this step. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> On Thu, Mar 2, 2017 at 4:56 PM, Alexey Goncharuk >>>>>>>>>>>>>>>> <[email protected] >>>>>>>>>>>>>>>> <mailto:[email protected]> >>>>>>>>>>>>>>>> <mailto:[email protected] >>>>>>>>>>>>>>>> <mailto:[email protected]>>> wrote: >>>>>>>>>>>>>>>>> Maxim, >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> I see several usages of asGridUuid() method, so I would just >>>>>>>>>>>>>>>>> remove >>>>>>>>>>> global >>>>>>>>>>>>>>>>> time and use nodeOrderDrId and topVer as different parts of >>>>>>>>>>>>>>>>> high >>>>>>>>>>> and low >>>>>>>>>>>>>>>>> parts of the embedded UUID. >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> --AG >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> 2017-03-02 12:39 GMT+03:00 Kozlov Maxim <[email protected] >>>>>>>>>>>>>>>>> <mailto:[email protected]><mailto:[email protected] >>>>>>>>>>>>>>>>> <mailto:[email protected]>>>: >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> Andrey, >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> When removed parameter globalTime, in method: >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> public IgniteUuid asGridUuid() { >>>>>>>>>>>>>>>>>> return new IgniteUuid(new UUID(((long)topVer << 32) | >>>>>>>>>>> nodeOrderDrId, >>>>>>>>>>>>>>>>>> globalTime), order); >>>>>>>>>>>>>>>>>> } >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> globalTime parameter replaced by something or remove this >>>>>>>>>>>>>>>>>> method? >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> 2 марта 2017 г., в 12:07, Kozlov Maxim >>>>>>>>>>>>>>>>>>> <[email protected] <mailto:[email protected]> >>>>>>>>>>>>>>>>>>> <mailto:[email protected] <mailto:[email protected]>>> >>>>>>>>>>>>>>>>>> написал(а): >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Andrey, >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Please review PR again. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> 1 марта 2017 г., в 18:47, Andrey Gura <[email protected] >>>>>>>>>>>>>>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>>>>>>>>>>>>>> <mailto:[email protected]>>> >>>>>>>>>>> написал(а): >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> I think that it is ok. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> On Wed, Mar 1, 2017 at 6:34 PM, Kozlov Maxim < >>>>>>>>>>> [email protected] <mailto:[email protected]> >>>>>>>>>>> <mailto:[email protected] <mailto:[email protected]>>> >>>>>>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>>>>>>> Ok. What do you say for the rest? >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> 1 марта 2017 г., в 18:15, Andrey Gura <[email protected] >>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected]>>> >>>>>>>>>>> написал(а): >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> Maxim, >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> I think that during renaming we should not lose "Atomic" >>>>>>>>>>> prefix. >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>> On Wed, Mar 1, 2017 at 5:16 PM, Kozlov Maxim < >>>>>>>>>>> [email protected] <mailto:[email protected]> >>>>>>>>>>> <mailto:[email protected] <mailto:[email protected]>>> >>>>>>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>>>>>>>>> Andrey, ok. >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> Also remove in the modules/platform/dotnet >>>>>>>>>>>>>>>>>> CacheAtomicWriteOrderMode.cs? >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> Rename classes: >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicNearCacheSelfTest.startGrids -> >>>>>>>>>>>>>>>>>> GridCacheAtomicNearCacheSelfTest.startGridsLocal (commit) >>>>>>>>>>>>>>>>>>>>>>> IgniteCacheAtomicPrimaryWriteOrderWithStoreInvokeTest -> >>>>>>>>>>>>>>>>>> IgniteCacheWithStoreInvokeTest >>>>>>>>>>>>>>>>>>>>>>> IgniteCacheAtomicPrimaryWriteOrderInvokeTest -> >>>>>>>>>>>>>>>>>> IgniteCacheInvokeTest >>>>>>>>>>>>>>>>>>>>>>> IgniteCacheAtomicPrimaryWriteOrderNearEnabledStoreValueTest >>>>>>>>>>> -> >>>>>>>>>>>>>>>>>> IgniteCacheNearEnabledStoreValueTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWriteOrderNearRemoveFailureTest -> >>>>>>>>>>>>>>>>>> GridCacheNearRemoveFailureTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWriteOrderRemoveFailureTest -> >>>>>>>>>>>>>>>>>> GridCacheRemoveFailureTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWriteOrderFailoverSelfTest -> >>>>>>>>>>>>>>>>>> GridCacheFailoverSelfTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheValueConsistencyAtomicPrimaryWriteOrderNearEnabledS >>>>>>>>>>> elfTest >>>>>>>>>>>>>>>>>> -> GridCacheValueConsistencyNearEnabledSelfTest >>>>>>>>>>>>>>>>>>>>>>> CacheContinuousQueryAsyncFailoverAtomicPrimaryWriteOrderSelfTest >>>>>>>>>>> -> >>>>>>>>>>>>>>>>>> CacheContinuousQueryAsyncFailoverSelfTest >>>>>>>>>>>>>>>>>>>>>>> CacheContinuousQueryFailoverAtomicPrimaryWriteOrderSelfTest >>>>>>>>>>> -> >>>>>>>>>>>>>>>>>> CacheContinuousQueryFailoverSelfTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicNearCacheSelfTest.testNoBackupsPrimaryWriteOrder >>>>>>>>>>> -> >>>>>>>>>>>>>>>>>> GridCacheAtomicNearCacheSelfTest.testNoBackups >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicNearCacheSelfTest. >>>>>>>>>>> testWithBackupsPrimaryWriteOrder >>>>>>>>>>>>>>>>>> -> GridCacheAtomicNearCacheSelfTest.testWithBackups >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> Remove classes: >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> IgniteCacheAtomicPrimaryWriteOrderStoreValueTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheReplicatedAtomicPrimaryWriteOrderMultiNodeFullApiSe >>>>>>>>>>> lfTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWriteOrderMultiNodeFullApiSelfTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWriteOrderMultiNodeP2PDisabledFullApiS >>>>>>>>>>> elfTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWrityOrderOffHeapMultiNodeFullApiSelfT >>>>>>>>>>> est >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWriteOrderOffHeapFullApiSelfTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWriteOrderFullApiSelfTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheAtomicPrimaryWriteOrderReloadAllSelfTest >>>>>>>>>>>>>>>>>>>>>>> IgniteCachePutRetryAtomicPrimaryWriteOrderSelfTest >>>>>>>>>>>>>>>>>>>>>>> GridCacheValueConsistencyAtomicPrimaryWriteOrderSelfTest >>>>>>>>>>>>>>>>>>>>>>> IgniteCacheAtomicPrimaryWriteOrderExpiryPolicyTest >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> ok? :) >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> 1 марта 2017 г., в 2:04, Andrey Gura <[email protected] >>>>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected]> <mailto:[email protected] >>>>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected]>>> >>>>>>>>>>> написал(а): >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> No, it should be removed. If somebody use entry last >>>>>>>>>>>>>>>>>>>>>>>> update >>>>>>>>>>> time >>>>>>>>>>>>>>>>>> (e.g. >>>>>>>>>>>>>>>>>>>>>>>> for conflict resolving) they should store this time as >>>>>>>>>>>>>>>>>>>>>>>> entry >>>>>>>>>>> field. >>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>> On Wed, Mar 1, 2017 at 12:57 AM, Dmitriy Setrakyan >>>>>>>>>>>>>>>>>>>>>>>> <[email protected] <mailto:[email protected]> >>>>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected] >>>>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected]>>> wrote: >>>>>>>>>>>>>>>>>>>>>>>>> Do we still need GridClockSyncProcessor? >>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>> On Tue, Feb 28, 2017 at 5:26 AM, Andrey Gura < >>>>>>>>>>> [email protected] <mailto:[email protected]> <mailto:[email protected] >>>>>>>>>>> <mailto:[email protected]>>> >>>>>>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> Maxim, >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> Yes, this setting doesn't make sense anymore. So we >>>>>>>>>>>>>>>>>>>>>>>>>> need >>>>>>>>>>> remove >>>>>>>>>>>>>>>>>> all >>>>>>>>>>>>>>>>>>>>>>>>>> related methods. >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> Also there is component called >>>>>>>>>>>>>>>>>>>>>>>>>> GridClockSyncProcessor that >>>>>>>>>>> also >>>>>>>>>>>>>>>>>> should >>>>>>>>>>>>>>>>>>>>>>>>>> be removed. It will lead to removing globalTime >>>>>>>>>>>>>>>>>>>>>>>>>> field from >>>>>>>>>>>>>>>>>>>>>>>>>> GridCacheVersion class and some related methods. >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> On Tue, Feb 28, 2017 at 3:21 PM, Kozlov Maxim < >>>>>>>>>>>>>>>>>> [email protected] <mailto:[email protected]> >>>>>>>>>>>>>>>>>> <mailto:[email protected] <mailto:[email protected]>>> >>>>>>>>>>>>>>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>>>>>>>>>>>>> Valentin, >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> Then there is no need for setting >>>>>>>>>>>>>>>>>>>>>>>>>>> CacheConfiguration. >>>>>>>>>>>>>>>>>> atomicWriteOrderMode. >>>>>>>>>>>>>>>>>>>>>>>>>> What do you think, remove it and and related methods? >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> 28 февр. 2017 г., в 2:49, Valentin Kulichenko < >>>>>>>>>>>>>>>>>>>>>>>>>> [email protected] >>>>>>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected]> >>>>>>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected] >>>>>>>>>>>>>>>>>>>>>>>>>> <mailto:[email protected]>>> написал(а): >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> Hi Max, >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> In case we remove the CLOCK mode, I think we should >>>>>>>>>>> remove the >>>>>>>>>>>>>>>>>> enum >>>>>>>>>>>>>>>>>>>>>>>>>> too, as >>>>>>>>>>>>>>>>>>>>>>>>>>>> well as configuration properties and other code >>>>>>>>>>>>>>>>>>>>>>>>>>>> using >>>>>>>>>>> this >>>>>>>>>>>>>>>>>> enum. Having >>>>>>>>>>>>>>>>>>>>>>>>>>>> enum with one value doesn't make sense to me. >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> -Val >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>> On Mon, Feb 27, 2017 at 5:09 AM, Kozlov Maxim < >>>>>>>>>>>>>>>>>> [email protected] <mailto:[email protected]> >>>>>>>>>>>>>>>>>> <mailto:[email protected] <mailto:[email protected]>>> >>>>>>>>>>>>>>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> Hi Igniters, >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> After remove CLOCK mode, >>>>>>>>>>>>>>>>>>>>>>>>>>>>> CacheAtomicWriteOrderMode enum >>>>>>>>>>>>>>>>>> contains now >>>>>>>>>>>>>>>>>>>>>>>>>> only >>>>>>>>>>>>>>>>>>>>>>>>>>>>> one value PRIMARY. Andrey Gura, proposition remove >>>>>>>>>>>>>>>>>>>>>>>>>>>>> CacheAtomicWriteOrderMode enum. Will there be >>>>>>>>>>>>>>>>>>>>>>>>>>>>> something >>>>>>>>>>>>>>>>>> special for >>>>>>>>>>>>>>>>>>>>>>>>>> this >>>>>>>>>>>>>>>>>>>>>>>>>>>>> purpose is enum? >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> jira: >>>>>>>>>>>>>>>>>>>>>>>>>>>>> https://issues.apache.org/jira/browse/IGNITE-4587 >>>>>>>>>>>>>>>>>>>>>>>>>>>>> <https://issues.apache.org/jira/browse/IGNITE-4587><https://issues.apache.org/jira/browse/IGNITE-4587 >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> <https://issues.apache.org/jira/browse/IGNITE-4587>> >>>>>>>>>>> < >>>>>>>>>>>>>>>>>>>>>>>>>>>>> https://issues.apache.org/jira/browse/IGNITE-4587 >>>>>>>>>>>>>>>>>>>>>>>>>>>>> <https://issues.apache.org/jira/browse/IGNITE-4587>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>>>>>>>>>>>>>> Max K. >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>>>>>>>>>>>> Max K. >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>>>>>>>> Max K. >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>>>>>> Max K. >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>>>> Max K. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>>>>> Max K. >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>>>> Max K. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> -- >>>>>>>>>>>>> Best Regards, >>>>>>>>>>>>> Max K. >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> -- >>>>>>>>>>> Best Regards, >>>>>>>>>>> Max K. >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>> >>>>>>>> -- >>>>>>>> Best Regards, >>>>>>>> Max K. >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Best Regards, >>>>>>> Max K. >>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>> >>>>> -- >>>>> Best Regards, >>>>> Max K. >>>>> >>>>> >>>>> >>>>> >>>> >>> >>> -- >>> Best Regards, >>> Max K. >>> >>> >>> >>> >> >> -- >> Best Regards, >> Max K. > -- Best Regards, Max K.
