Return-Path: X-Original-To: apmail-ignite-commits-archive@minotaur.apache.org Delivered-To: apmail-ignite-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 0CA071813D for ; Sat, 2 Jan 2016 21:22:12 +0000 (UTC) Received: (qmail 3741 invoked by uid 500); 2 Jan 2016 21:22:12 -0000 Delivered-To: apmail-ignite-commits-archive@ignite.apache.org Received: (qmail 3697 invoked by uid 500); 2 Jan 2016 21:22:11 -0000 Mailing-List: contact commits-help@ignite.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ignite.apache.org Delivered-To: mailing list commits@ignite.apache.org Received: (qmail 3683 invoked by uid 99); 2 Jan 2016 21:22:11 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 02 Jan 2016 21:22:11 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id B54A6E021A; Sat, 2 Jan 2016 21:22:11 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: vozerov@apache.org To: commits@ignite.apache.org Date: Sat, 02 Jan 2016 21:22:11 -0000 Message-Id: <7f4c0c8f0e0c4882a88c3fb8bb52da93@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [1/3] ignite git commit: IGNITE-2330: Simplified GridFunc. Repository: ignite Updated Branches: refs/heads/master dffcb834f -> 80579253f http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple3.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple3.java b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple3.java index b999e2a..e5d247a 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple3.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple3.java @@ -34,7 +34,6 @@ import org.jetbrains.annotations.Nullable; * This class doesn't provide any synchronization for multi-threaded access * and it is responsibility of the user of this class to provide outside * synchronization, if needed. - * @see GridFunc#t3() * @see GridFunc#t(Object, Object, Object) */ public class GridTuple3 implements Iterable, Externalizable, Cloneable { http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple4.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple4.java b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple4.java index c95a859..d1e69b5 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple4.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple4.java @@ -34,7 +34,6 @@ import org.jetbrains.annotations.Nullable; * This class doesn't provide any synchronization for multi-threaded access * and it is responsibility of the user of this class to provide outside * synchronization, if needed. - * @see GridFunc#t4() * @see GridFunc#t(Object, Object, Object, Object) */ public class GridTuple4 implements Iterable, Externalizable, Cloneable { http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple5.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple5.java b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple5.java index 9790f48..7d25996 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple5.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple5.java @@ -34,7 +34,6 @@ import org.jetbrains.annotations.Nullable; * This class doesn't provide any synchronization for multi-threaded access * and it is responsibility of the user of this class to provide outside * synchronization, if needed. - * @see GridFunc#t5() * @see GridFunc#t(Object, Object, Object, Object, Object) */ public class GridTuple5 implements Iterable, Externalizable, Cloneable { http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple6.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple6.java b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple6.java index 044944b..c904587 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple6.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTuple6.java @@ -34,7 +34,6 @@ import org.jetbrains.annotations.Nullable; * This class doesn't provide any synchronization for multi-threaded access * and it is responsibility of the user of this class to provide outside * synchronization, if needed. - * @see GridFunc#t5() * @see GridFunc#t(Object, Object, Object, Object, Object) */ public class GridTuple6 implements Iterable, Externalizable, http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTupleV.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTupleV.java b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTupleV.java index 225366a..58e18aa 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTupleV.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/util/lang/GridTupleV.java @@ -35,7 +35,6 @@ import org.apache.ignite.internal.util.typedef.internal.U; * This class doesn't provide any synchronization for multi-threaded access * and it is responsibility of the user of this class to provide outside * synchronization, if needed. - * @see GridFunc#tv(Object...) */ public class GridTupleV implements Iterable, Externalizable, Cloneable { /** */ http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/main/java/org/apache/ignite/lang/IgniteUuid.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/lang/IgniteUuid.java b/modules/core/src/main/java/org/apache/ignite/lang/IgniteUuid.java index 5c6bb9a..44ca067 100644 --- a/modules/core/src/main/java/org/apache/ignite/lang/IgniteUuid.java +++ b/modules/core/src/main/java/org/apache/ignite/lang/IgniteUuid.java @@ -186,7 +186,7 @@ public final class IgniteUuid implements Comparable, Iterable iterator() { - return F.iterator(Collections.singleton(this), F.identity(), true); + return F.identityIteratorReadOnly(Collections.singleton(this)); } /** {@inheritDoc} */ http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/main/java/org/apache/ignite/spi/eventstorage/memory/MemoryEventStorageSpi.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/spi/eventstorage/memory/MemoryEventStorageSpi.java b/modules/core/src/main/java/org/apache/ignite/spi/eventstorage/memory/MemoryEventStorageSpi.java index c7c635e..56a627a 100644 --- a/modules/core/src/main/java/org/apache/ignite/spi/eventstorage/memory/MemoryEventStorageSpi.java +++ b/modules/core/src/main/java/org/apache/ignite/spi/eventstorage/memory/MemoryEventStorageSpi.java @@ -216,6 +216,7 @@ public class MemoryEventStorageSpi extends IgniteSpiAdapter implements EventStor } /** {@inheritDoc} */ + @SuppressWarnings("unchecked") @Override public Collection localEvents(IgnitePredicate p) { A.notNull(p, "p"); http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheLuceneQueryIndexTest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheLuceneQueryIndexTest.java b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheLuceneQueryIndexTest.java index 59a3eca..1e868b7 100644 --- a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheLuceneQueryIndexTest.java +++ b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheLuceneQueryIndexTest.java @@ -21,6 +21,7 @@ import java.io.Serializable; import java.util.HashMap; import java.util.Map; import java.util.concurrent.Callable; +import java.util.concurrent.ThreadLocalRandom; import java.util.concurrent.atomic.AtomicInteger; import org.apache.ignite.Ignite; import org.apache.ignite.IgniteCache; @@ -335,7 +336,8 @@ public class GridCacheLuceneQueryIndexTest extends GridCommonAbstractTest { map = new HashMap<>(); } - map.put(new ObjectKey(String.valueOf(i)), F.rand(vals)); + map.put(new ObjectKey(String.valueOf(i)), + vals[ThreadLocalRandom.current().nextInt(vals.length)]); } if (!map.isEmpty()) { http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCacheEventAbstractTest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCacheEventAbstractTest.java b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCacheEventAbstractTest.java index cb24e54..52737e7 100644 --- a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCacheEventAbstractTest.java +++ b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCacheEventAbstractTest.java @@ -757,7 +757,7 @@ public abstract class GridCacheEventAbstractTest extends GridCacheAbstractSelfTe if (TEST_INFO) X.println("Cache event: " + evt.shortDisplay()); - AtomicInteger cntr = F.addIfAbsent(cntrs, evt.type(), F.newAtomicInt()); + AtomicInteger cntr = F.addIfAbsent(cntrs, evt.type(), new AtomicInteger()); assert cntr != null; http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/test/java/org/apache/ignite/internal/util/ipc/shmem/IpcSharedMemoryCrashDetectionSelfTest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/internal/util/ipc/shmem/IpcSharedMemoryCrashDetectionSelfTest.java b/modules/core/src/test/java/org/apache/ignite/internal/util/ipc/shmem/IpcSharedMemoryCrashDetectionSelfTest.java index 1aae999..3482fd6 100644 --- a/modules/core/src/test/java/org/apache/ignite/internal/util/ipc/shmem/IpcSharedMemoryCrashDetectionSelfTest.java +++ b/modules/core/src/test/java/org/apache/ignite/internal/util/ipc/shmem/IpcSharedMemoryCrashDetectionSelfTest.java @@ -517,12 +517,16 @@ public class IpcSharedMemoryCrashDetectionSelfTest extends GridCommonAbstractTes * @param shmemIds Shared memory IDs string. */ public void shmemIds(String shmemIds) { - this.shmemIds = (shmemIds == null) ? null : - F.transform(shmemIds.split(","), new C1() { - @Override public Integer apply(String s) { - return Long.valueOf(s).intValue(); - } - }); + if (shmemIds == null) + this.shmemIds = null; + else { + String[] tokens = shmemIds.split(","); + + this.shmemIds = new ArrayList<>(tokens.length); + + for (String token : tokens) + this.shmemIds.add(Long.valueOf(token).intValue()); + } } } } \ No newline at end of file http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/test/java/org/apache/ignite/lang/GridBasicPerformanceTest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/lang/GridBasicPerformanceTest.java b/modules/core/src/test/java/org/apache/ignite/lang/GridBasicPerformanceTest.java index 37e7afe..353367e 100644 --- a/modules/core/src/test/java/org/apache/ignite/lang/GridBasicPerformanceTest.java +++ b/modules/core/src/test/java/org/apache/ignite/lang/GridBasicPerformanceTest.java @@ -748,8 +748,14 @@ public class GridBasicPerformanceTest { for (int i = 0; i < MAX; i++) { if (sort) Arrays.binarySearch(arr, ThreadLocalRandom8.current().nextInt(lim)); - else - F.contains(arr, ThreadLocalRandom8.current().nextInt(lim)); + else { + int val = ThreadLocalRandom8.current().nextInt(lim); + + for (long arrItem : arr) { + if (arrItem == val) + break; + } + } } long time = System.currentTimeMillis() - start; http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/test/java/org/apache/ignite/lang/GridFuncPerformanceTest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/lang/GridFuncPerformanceTest.java b/modules/core/src/test/java/org/apache/ignite/lang/GridFuncPerformanceTest.java deleted file mode 100644 index 5afd75f..0000000 --- a/modules/core/src/test/java/org/apache/ignite/lang/GridFuncPerformanceTest.java +++ /dev/null @@ -1,102 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed with - * this work for additional information regarding copyright ownership. - * The ASF licenses this file to You under the Apache License, Version 2.0 - * (the "License"); you may not use this file except in compliance with - * the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -package org.apache.ignite.lang; - -import java.util.ArrayList; -import java.util.Collection; -import org.apache.ignite.internal.util.lang.GridIterator; -import org.apache.ignite.internal.util.typedef.F; -import org.apache.ignite.testframework.junits.common.GridCommonAbstractTest; -import org.apache.ignite.testframework.junits.common.GridCommonTest; - -/** - * GridFunc performance test. - */ -@GridCommonTest(group = "Lang") -public class GridFuncPerformanceTest extends GridCommonAbstractTest { - /** - * Creates test. - */ - public GridFuncPerformanceTest() { - super(/*start grid*/false); - } - - /** - * - */ - public void testTransformingIteratorPerformance() { - // Warmup. - testBody(); - testBody(); - testBody(); - - long r1 = testBody(); - long r2 = testBody(); - long r3 = testBody(); - - double r = (r1 + r2 + r3) / 3.f; - - System.out.println("Average result is: " + Math.round(r) + "msec."); - } - - /** - * - * @return Duration of the test. - */ - @SuppressWarnings({"UnusedDeclaration"}) - private long testBody() { - int MAX = 20000000; - - Collection l = new ArrayList<>(MAX); - - for (int i = 0; i < MAX / 10; i++) - l.add(i); - - IgniteClosure c = new IgniteClosure() { - @Override public Integer apply(Integer e) { - return e; - } - }; - - IgnitePredicate p1 = new IgnitePredicate() { - @Override public boolean apply(Integer e) { - return e % 2 == 0; - } - }; - IgnitePredicate p2 = new IgnitePredicate() { - @Override public boolean apply(Integer e) { - return e % 2 != 0; - } - }; - - GridIterator iter = F.iterator(l, c, true, p1, p2); - - long n = 0; - - long start = System.currentTimeMillis(); - - for (Integer i : iter) - n += i; - - long duration = System.currentTimeMillis() - start; - - System.out.println("Duration: " + duration + "msec."); - - return duration; - } -} \ No newline at end of file http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/core/src/test/java/org/apache/ignite/loadtest/GridLoadTestStatistics.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/loadtest/GridLoadTestStatistics.java b/modules/core/src/test/java/org/apache/ignite/loadtest/GridLoadTestStatistics.java index 5d517a4..10b140e 100644 --- a/modules/core/src/test/java/org/apache/ignite/loadtest/GridLoadTestStatistics.java +++ b/modules/core/src/test/java/org/apache/ignite/loadtest/GridLoadTestStatistics.java @@ -107,7 +107,7 @@ public class GridLoadTestStatistics { AtomicInteger cnt; synchronized (nodeCnts) { - cnt = F.addIfAbsent(nodeCnts, id, F.newAtomicInt()); + cnt = F.addIfAbsent(nodeCnts, id, new AtomicInteger()); } assert cnt != null; http://git-wip-us.apache.org/repos/asf/ignite/blob/80579253/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/opt/GridH2TreeIndex.java ---------------------------------------------------------------------- diff --git a/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/opt/GridH2TreeIndex.java b/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/opt/GridH2TreeIndex.java index 28adeee..9582df0 100644 --- a/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/opt/GridH2TreeIndex.java +++ b/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/opt/GridH2TreeIndex.java @@ -24,10 +24,10 @@ import java.util.Iterator; import java.util.NavigableMap; import java.util.concurrent.ConcurrentNavigableMap; import java.util.concurrent.ConcurrentSkipListMap; -import org.apache.ignite.internal.util.GridEmptyIterator; import org.apache.ignite.internal.util.offheap.unsafe.GridOffHeapSnapTreeMap; import org.apache.ignite.internal.util.offheap.unsafe.GridUnsafeGuard; import org.apache.ignite.internal.util.snaptree.SnapTreeMap; +import org.apache.ignite.internal.util.typedef.F; import org.apache.ignite.internal.util.typedef.internal.SB; import org.apache.ignite.internal.util.typedef.internal.U; import org.apache.ignite.spi.indexing.IndexingQueryFilter; @@ -319,7 +319,7 @@ public class GridH2TreeIndex extends GridH2IndexBase implements Comparator(); + return F.emptyIterator(); return filter(range.values().iterator()); }