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 A5DAD18309 for ; Thu, 2 Jul 2015 15:03:36 +0000 (UTC) Received: (qmail 32418 invoked by uid 500); 2 Jul 2015 15:03:36 -0000 Delivered-To: apmail-ignite-commits-archive@ignite.apache.org Received: (qmail 32389 invoked by uid 500); 2 Jul 2015 15:03:36 -0000 Mailing-List: contact commits-help@ignite.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ignite.incubator.apache.org Delivered-To: mailing list commits@ignite.incubator.apache.org Received: (qmail 32380 invoked by uid 99); 2 Jul 2015 15:03:36 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 02 Jul 2015 15:03:36 +0000 X-ASF-Spam-Status: No, hits=-2000.7 required=5.0 tests=ALL_TRUSTED,RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO mail.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with SMTP; Thu, 02 Jul 2015 15:01:20 +0000 Received: (qmail 31966 invoked by uid 99); 2 Jul 2015 15:03:05 -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; Thu, 02 Jul 2015 15:03:05 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 6F32CE365D; Thu, 2 Jul 2015 15:03:05 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: sboikov@apache.org To: commits@ignite.incubator.apache.org Date: Thu, 02 Jul 2015 15:03:08 -0000 Message-Id: In-Reply-To: <1aebc52efecf4e09b87e4007a8ddee5b@git.apache.org> References: <1aebc52efecf4e09b87e4007a8ddee5b@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [4/5] incubator-ignite git commit: [IGNITE-1058]: GridIoPolicy: Switch from enum to collection of byte constants. X-Virus-Checked: Checked by ClamAV on apache.org [IGNITE-1058]: GridIoPolicy: Switch from enum to collection of byte constants. Project: http://git-wip-us.apache.org/repos/asf/incubator-ignite/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ignite/commit/b437ec73 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ignite/tree/b437ec73 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ignite/diff/b437ec73 Branch: refs/heads/ignite-gg-10460 Commit: b437ec73dbc15388d1a9a526dc43e7a74abdc288 Parents: 91e1cb2 Author: iveselovskiy Authored: Thu Jul 2 13:26:22 2015 +0300 Committer: iveselovskiy Committed: Thu Jul 2 13:26:22 2015 +0300 ---------------------------------------------------------------------- .../apache/ignite/internal/IgniteKernal.java | 1 - .../managers/communication/GridIoManager.java | 124 +++++++++++++++---- .../managers/communication/GridIoMessage.java | 15 ++- .../managers/communication/GridIoPolicy.java | 32 ++--- .../eventstorage/GridEventStorageManager.java | 2 +- .../internal/processors/cache/CacheType.java | 8 +- .../processors/cache/GridCacheContext.java | 4 +- .../processors/cache/GridCacheIoManager.java | 12 +- .../GridDistributedTxFinishRequest.java | 11 +- .../GridDistributedTxPrepareRequest.java | 9 +- .../GridDistributedTxRemoteAdapter.java | 3 +- .../distributed/dht/GridDhtTxFinishRequest.java | 3 +- .../cache/distributed/dht/GridDhtTxLocal.java | 3 +- .../distributed/dht/GridDhtTxLocalAdapter.java | 3 +- .../cache/distributed/dht/GridDhtTxRemote.java | 5 +- .../near/GridNearTxFinishRequest.java | 3 +- .../cache/distributed/near/GridNearTxLocal.java | 3 +- .../distributed/near/GridNearTxRemote.java | 5 +- .../cache/transactions/IgniteInternalTx.java | 3 +- .../cache/transactions/IgniteTxAdapter.java | 11 +- .../transactions/IgniteTxLocalAdapter.java | 3 +- .../internal/processors/igfs/IgfsContext.java | 5 +- .../plugin/IgnitePluginProcessor.java | 3 +- .../plugin/extensions/communication/IoPool.java | 42 +++++++ .../communication/GridIoManagerSelfTest.java | 2 +- .../cache/IgniteInternalCacheTypesTest.java | 3 +- 26 files changed, 202 insertions(+), 116 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java b/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java index e19d3d3..30931fa 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java @@ -77,7 +77,6 @@ import javax.management.*; import java.io.*; import java.lang.management.*; import java.lang.reflect.*; -import java.security.*; import java.text.*; import java.util.*; import java.util.concurrent.*; http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/managers/communication/GridIoManager.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/managers/communication/GridIoManager.java b/modules/core/src/main/java/org/apache/ignite/internal/managers/communication/GridIoManager.java index d8dcc2c..c1fb79a 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/managers/communication/GridIoManager.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/managers/communication/GridIoManager.java @@ -69,6 +69,9 @@ public class GridIoManager extends GridManagerAdapter disconnectLsnrs = new ConcurrentLinkedQueue<>(); + /** Map of {@link IoPool}-s injected by Ignite plugins. */ + private final IoPool[] ioPools = new IoPool[128]; + /** Public pool. */ private ExecutorService pubPool; @@ -255,6 +258,41 @@ public class GridIoManager extends GridManagerAdapter= 0 : "Negative policy: " + plc; + + if (isReservedGridIoPolicy(plc)) + throw new IgniteCheckedException("Failed to process message with policy of reserved" + + " range (0-31), [policy=" + plc + ']'); + + IoPool pool = ioPools[plc]; + + if (pool == null) + throw new IgniteCheckedException("Failed to process message because no pool is registered " + + "for policy. [policy=" + plc + ']'); + + assert plc == pool.id(); + + Executor ex = pool.executor(); - // Never reached. - return null; + if (ex == null) + throw new IgniteCheckedException("Failed to process message because corresponding executor " + + "is null. [id=" + plc + ']'); + + return ex; } } } @@ -643,9 +713,9 @@ public class GridIoManager extends GridManagerAdapter nodes, Object topic, Message msg, - GridIoPolicy plc, + byte plc, long timeout, boolean skipOnTimeout ) @@ -1091,7 +1160,7 @@ public class GridIoManager extends GridManagerAdapter nodes, Object topic, Message msg, - GridIoPolicy plc + byte plc ) throws IgniteCheckedException { send(nodes, topic, -1, msg, plc, false, 0, false); } @@ -1107,7 +1176,7 @@ public class GridIoManager extends GridManagerAdapter nodes, GridTopic topic, Message msg, - GridIoPolicy plc + byte plc ) throws IgniteCheckedException { send(nodes, topic, topic.ordinal(), msg, plc, false, 0, false); } @@ -1247,7 +1316,7 @@ public class GridIoManager extends GridManagerAdapter= 0 && ord < VALS.length ? VALS[ord] : null; + public static boolean isReservedGridIoPolicy(byte key) { + return key >= 0 && key <= 31; } } http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/managers/eventstorage/GridEventStorageManager.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/managers/eventstorage/GridEventStorageManager.java b/modules/core/src/main/java/org/apache/ignite/internal/managers/eventstorage/GridEventStorageManager.java index 95c5eb1..365076b 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/managers/eventstorage/GridEventStorageManager.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/managers/eventstorage/GridEventStorageManager.java @@ -986,7 +986,7 @@ public class GridEventStorageManager extends GridManagerAdapter * @throws IgniteCheckedException If sending failed. */ private void sendMessage(Collection nodes, GridTopic topic, - GridEventStorageMessage msg, GridIoPolicy plc) throws IgniteCheckedException { + GridEventStorageMessage msg, byte plc) throws IgniteCheckedException { ClusterNode locNode = F.find(nodes, null, F.localNode(ctx.localNodeId())); Collection rmtNodes = F.view(nodes, F.remoteNodes(ctx.localNodeId())); http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/CacheType.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/CacheType.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/CacheType.java index e0747b9..954bfe5 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/CacheType.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/CacheType.java @@ -17,8 +17,6 @@ package org.apache.ignite.internal.processors.cache; -import org.apache.ignite.internal.managers.communication.*; - import static org.apache.ignite.internal.managers.communication.GridIoPolicy.*; /** @@ -49,13 +47,13 @@ public enum CacheType { private final boolean userCache; /** */ - private final GridIoPolicy ioPlc; + private final byte ioPlc; /** * @param userCache {@code True} if cache created by user. * @param ioPlc Cache IO policy. */ - CacheType(boolean userCache, GridIoPolicy ioPlc) { + CacheType(boolean userCache, byte ioPlc) { this.userCache = userCache; this.ioPlc = ioPlc; } @@ -63,7 +61,7 @@ public enum CacheType { /** * @return Cache IO policy. */ - public GridIoPolicy ioPolicy() { + public byte ioPolicy() { return ioPlc; } http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java index 9b7801c..580e452 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheContext.java @@ -170,7 +170,7 @@ public class GridCacheContext implements Externalizable { private CacheType cacheType; /** IO policy. */ - private GridIoPolicy plc; + private byte plc; /** Default expiry policy. */ private ExpiryPolicy expiryPlc; @@ -459,7 +459,7 @@ public class GridCacheContext implements Externalizable { /** * @return IO policy for the given cache. */ - public GridIoPolicy ioPolicy() { + public byte ioPolicy() { return plc; } http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheIoManager.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheIoManager.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheIoManager.java index 74a4512..0707096 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheIoManager.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheIoManager.java @@ -309,7 +309,7 @@ public class GridCacheIoManager extends GridCacheSharedManagerAdapter { * @param plc grid io policy. */ private void sendResponseOnFailedMessage(UUID nodeId, GridCacheMessage res, GridCacheSharedContext cctx, - GridIoPolicy plc) { + byte plc) { try { cctx.io().send(nodeId, res, plc); } @@ -580,7 +580,7 @@ public class GridCacheIoManager extends GridCacheSharedManagerAdapter { * @throws ClusterTopologyCheckedException If receiver left. */ @SuppressWarnings("unchecked") - public void send(ClusterNode node, GridCacheMessage msg, GridIoPolicy plc) throws IgniteCheckedException { + public void send(ClusterNode node, GridCacheMessage msg, byte plc) throws IgniteCheckedException { assert !node.isLocal(); onSend(msg, node.id()); @@ -627,7 +627,7 @@ public class GridCacheIoManager extends GridCacheSharedManagerAdapter { * @throws IgniteCheckedException If send failed. */ @SuppressWarnings({"BusyWait", "unchecked"}) - public boolean safeSend(Collection nodes, GridCacheMessage msg, GridIoPolicy plc, + public boolean safeSend(Collection nodes, GridCacheMessage msg, byte plc, @Nullable IgnitePredicate fallback) throws IgniteCheckedException { assert nodes != null; assert msg != null; @@ -739,7 +739,7 @@ public class GridCacheIoManager extends GridCacheSharedManagerAdapter { * @param plc IO policy. * @throws IgniteCheckedException If sending failed. */ - public void send(UUID nodeId, GridCacheMessage msg, GridIoPolicy plc) throws IgniteCheckedException { + public void send(UUID nodeId, GridCacheMessage msg, byte plc) throws IgniteCheckedException { ClusterNode n = cctx.discovery().node(nodeId); if (n == null) @@ -757,7 +757,7 @@ public class GridCacheIoManager extends GridCacheSharedManagerAdapter { * @param timeout Timeout to keep a message on receiving queue. * @throws IgniteCheckedException Thrown in case of any errors. */ - public void sendOrderedMessage(ClusterNode node, Object topic, GridCacheMessage msg, GridIoPolicy plc, + public void sendOrderedMessage(ClusterNode node, Object topic, GridCacheMessage msg, byte plc, long timeout) throws IgniteCheckedException { onSend(msg, node.id()); @@ -807,7 +807,7 @@ public class GridCacheIoManager extends GridCacheSharedManagerAdapter { */ public void sendNoRetry(ClusterNode node, GridCacheMessage msg, - GridIoPolicy plc) + byte plc) throws IgniteCheckedException { assert node != null; http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxFinishRequest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxFinishRequest.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxFinishRequest.java index 986e211..7a84f9a 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxFinishRequest.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxFinishRequest.java @@ -18,7 +18,6 @@ package org.apache.ignite.internal.processors.cache.distributed; import org.apache.ignite.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.version.*; import org.apache.ignite.internal.util.tostring.*; @@ -68,7 +67,7 @@ public class GridDistributedTxFinishRequest extends GridDistributedBaseMessage { private boolean sys; /** IO policy. */ - private GridIoPolicy plc; + private byte plc; /** * Empty constructor required by {@link Externalizable}. @@ -99,7 +98,7 @@ public class GridDistributedTxFinishRequest extends GridDistributedBaseMessage { boolean commit, boolean invalidate, boolean sys, - GridIoPolicy plc, + byte plc, boolean syncCommit, boolean syncRollback, GridCacheVersion baseVer, @@ -135,7 +134,7 @@ public class GridDistributedTxFinishRequest extends GridDistributedBaseMessage { /** * @return IO policy. */ - public GridIoPolicy policy() { + public byte policy() { return plc; } @@ -268,7 +267,7 @@ public class GridDistributedTxFinishRequest extends GridDistributedBaseMessage { writer.incrementState(); case 12: - if (!writer.writeByte("plc", plc != null ? (byte)plc.ordinal() : -1)) + if (!writer.writeByte("plc", plc)) return false; writer.incrementState(); @@ -367,7 +366,7 @@ public class GridDistributedTxFinishRequest extends GridDistributedBaseMessage { if (!reader.isLastRead()) return false; - plc = GridIoPolicy.fromOrdinal(plcOrd); + plc = plcOrd; reader.incrementState(); http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxPrepareRequest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxPrepareRequest.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxPrepareRequest.java index 3078277..20aa833 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxPrepareRequest.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxPrepareRequest.java @@ -19,7 +19,6 @@ package org.apache.ignite.internal.processors.cache.distributed; import org.apache.ignite.*; import org.apache.ignite.internal.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.transactions.*; import org.apache.ignite.internal.processors.cache.version.*; @@ -105,7 +104,7 @@ public class GridDistributedTxPrepareRequest extends GridDistributedBaseMessage private boolean sys; /** IO policy. */ - private GridIoPolicy plc; + private byte plc; /** * Required by {@link Externalizable}. @@ -163,7 +162,7 @@ public class GridDistributedTxPrepareRequest extends GridDistributedBaseMessage /** * @return IO policy. */ - public GridIoPolicy policy() { + public byte policy() { return plc; } @@ -377,7 +376,7 @@ public class GridDistributedTxPrepareRequest extends GridDistributedBaseMessage writer.incrementState(); case 13: - if (!writer.writeByte("plc", plc != null ? (byte)plc.ordinal() : -1)) + if (!writer.writeByte("plc", plc)) return false; writer.incrementState(); @@ -510,7 +509,7 @@ public class GridDistributedTxPrepareRequest extends GridDistributedBaseMessage if (!reader.isLastRead()) return false; - plc = GridIoPolicy.fromOrdinal(plcOrd); + plc = plcOrd; reader.incrementState(); http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxRemoteAdapter.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxRemoteAdapter.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxRemoteAdapter.java index ac5395d..ef5d3d9 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxRemoteAdapter.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/GridDistributedTxRemoteAdapter.java @@ -20,7 +20,6 @@ package org.apache.ignite.internal.processors.cache.distributed; import org.apache.ignite.*; import org.apache.ignite.internal.*; import org.apache.ignite.internal.processors.affinity.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.distributed.near.*; import org.apache.ignite.internal.processors.cache.transactions.*; @@ -105,7 +104,7 @@ public class GridDistributedTxRemoteAdapter extends IgniteTxAdapter GridCacheVersion xidVer, GridCacheVersion commitVer, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, boolean invalidate, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxFinishRequest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxFinishRequest.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxFinishRequest.java index a493ebb..fe72b24 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxFinishRequest.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxFinishRequest.java @@ -19,7 +19,6 @@ package org.apache.ignite.internal.processors.cache.distributed.dht; import org.apache.ignite.internal.*; import org.apache.ignite.internal.processors.affinity.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.cache.distributed.*; import org.apache.ignite.internal.processors.cache.version.*; import org.apache.ignite.internal.util.tostring.*; @@ -111,7 +110,7 @@ public class GridDhtTxFinishRequest extends GridDistributedTxFinishRequest { boolean commit, boolean invalidate, boolean sys, - GridIoPolicy plc, + byte plc, boolean sysInvalidate, boolean syncCommit, boolean syncRollback, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocal.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocal.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocal.java index 841cac8..6a72c89 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocal.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocal.java @@ -21,7 +21,6 @@ import org.apache.ignite.*; import org.apache.ignite.internal.*; import org.apache.ignite.internal.cluster.*; import org.apache.ignite.internal.processors.affinity.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.distributed.*; import org.apache.ignite.internal.processors.cache.distributed.near.*; @@ -106,7 +105,7 @@ public class GridDhtTxLocal extends GridDhtTxLocalAdapter implements GridCacheMa boolean implicitSingle, boolean sys, boolean explicitLock, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, long timeout, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocalAdapter.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocalAdapter.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocalAdapter.java index 90edb0f..b50a010 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocalAdapter.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxLocalAdapter.java @@ -21,7 +21,6 @@ import org.apache.ignite.*; import org.apache.ignite.cluster.*; import org.apache.ignite.internal.*; import org.apache.ignite.internal.processors.affinity.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.distributed.*; import org.apache.ignite.internal.processors.cache.distributed.near.*; @@ -100,7 +99,7 @@ public abstract class GridDhtTxLocalAdapter extends IgniteTxLocalAdapter { boolean implicitSingle, boolean sys, boolean explicitLock, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, long timeout, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxRemote.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxRemote.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxRemote.java index 0a69910..ae13a10 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxRemote.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxRemote.java @@ -18,7 +18,6 @@ package org.apache.ignite.internal.processors.cache.distributed.dht; import org.apache.ignite.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.affinity.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.distributed.*; @@ -90,7 +89,7 @@ public class GridDhtTxRemote extends GridDistributedTxRemoteAdapter { GridCacheVersion xidVer, GridCacheVersion commitVer, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, boolean invalidate, @@ -149,7 +148,7 @@ public class GridDhtTxRemote extends GridDistributedTxRemoteAdapter { GridCacheVersion xidVer, GridCacheVersion commitVer, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, boolean invalidate, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxFinishRequest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxFinishRequest.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxFinishRequest.java index 15f42ff..26e4bdc 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxFinishRequest.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxFinishRequest.java @@ -18,7 +18,6 @@ package org.apache.ignite.internal.processors.cache.distributed.near; import org.apache.ignite.internal.processors.affinity.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.cache.distributed.*; import org.apache.ignite.internal.processors.cache.version.*; import org.apache.ignite.internal.util.tostring.*; @@ -84,7 +83,7 @@ public class GridNearTxFinishRequest extends GridDistributedTxFinishRequest { boolean commit, boolean invalidate, boolean sys, - GridIoPolicy plc, + byte plc, boolean syncCommit, boolean syncRollback, boolean explicitLock, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxLocal.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxLocal.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxLocal.java index fa8877a..cb391e4 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxLocal.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxLocal.java @@ -20,7 +20,6 @@ package org.apache.ignite.internal.processors.cache.distributed.near; import org.apache.ignite.*; import org.apache.ignite.cluster.*; import org.apache.ignite.internal.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.affinity.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.distributed.*; @@ -111,7 +110,7 @@ public class GridNearTxLocal extends GridDhtTxLocalAdapter { boolean implicit, boolean implicitSingle, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, long timeout, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxRemote.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxRemote.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxRemote.java index 49283cb..4ac81f8 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxRemote.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxRemote.java @@ -18,7 +18,6 @@ package org.apache.ignite.internal.processors.cache.distributed.near; import org.apache.ignite.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.distributed.*; import org.apache.ignite.internal.processors.cache.transactions.*; @@ -86,7 +85,7 @@ public class GridNearTxRemote extends GridDistributedTxRemoteAdapter { GridCacheVersion xidVer, GridCacheVersion commitVer, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, boolean invalidate, @@ -143,7 +142,7 @@ public class GridNearTxRemote extends GridDistributedTxRemoteAdapter { GridCacheVersion xidVer, GridCacheVersion commitVer, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, boolean invalidate, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteInternalTx.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteInternalTx.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteInternalTx.java index cb86e0d..b16e950 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteInternalTx.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteInternalTx.java @@ -19,7 +19,6 @@ package org.apache.ignite.internal.processors.cache.transactions; import org.apache.ignite.*; import org.apache.ignite.internal.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.affinity.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.version.*; @@ -246,7 +245,7 @@ public interface IgniteInternalTx extends AutoCloseable, GridTimeoutObject { /** * @return Pool where message for the given transaction must be processed. */ - public GridIoPolicy ioPolicy(); + public byte ioPolicy(); /** * @return Last recorded topology version. http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxAdapter.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxAdapter.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxAdapter.java index 9e8950f..709c208 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxAdapter.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxAdapter.java @@ -20,7 +20,6 @@ package org.apache.ignite.internal.processors.cache.transactions; import org.apache.ignite.*; import org.apache.ignite.cluster.*; import org.apache.ignite.internal.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.affinity.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.distributed.near.*; @@ -136,7 +135,7 @@ public abstract class IgniteTxAdapter extends GridMetadataAwareAdapter private boolean sys; /** IO policy. */ - private GridIoPolicy plc; + private byte plc; /** */ protected boolean onePhaseCommit; @@ -238,7 +237,7 @@ public abstract class IgniteTxAdapter extends GridMetadataAwareAdapter boolean implicitSingle, boolean loc, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, long timeout, @@ -296,7 +295,7 @@ public abstract class IgniteTxAdapter extends GridMetadataAwareAdapter GridCacheVersion startVer, long threadId, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, long timeout, @@ -399,7 +398,7 @@ public abstract class IgniteTxAdapter extends GridMetadataAwareAdapter } /** {@inheritDoc} */ - @Override public GridIoPolicy ioPolicy() { + @Override public byte ioPolicy() { return plc; } @@ -1716,7 +1715,7 @@ public abstract class IgniteTxAdapter extends GridMetadataAwareAdapter throw new IllegalStateException("Deserialized transaction can only be used as read-only."); } - @Override public GridIoPolicy ioPolicy() { + @Override public byte ioPolicy() { throw new IllegalStateException("Deserialized transaction can only be used as read-only."); } http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxLocalAdapter.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxLocalAdapter.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxLocalAdapter.java index 7bc9042..0a61b1a 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxLocalAdapter.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/IgniteTxLocalAdapter.java @@ -20,7 +20,6 @@ package org.apache.ignite.internal.processors.cache.transactions; import org.apache.ignite.*; import org.apache.ignite.internal.*; import org.apache.ignite.internal.cluster.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.processors.affinity.*; import org.apache.ignite.internal.processors.cache.*; import org.apache.ignite.internal.processors.cache.distributed.near.*; @@ -128,7 +127,7 @@ public abstract class IgniteTxLocalAdapter extends IgniteTxAdapter boolean implicit, boolean implicitSingle, boolean sys, - GridIoPolicy plc, + byte plc, TransactionConcurrency concurrency, TransactionIsolation isolation, long timeout, http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/igfs/IgfsContext.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/igfs/IgfsContext.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/igfs/IgfsContext.java index 475a7fc..24b56d3 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/igfs/IgfsContext.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/igfs/IgfsContext.java @@ -21,7 +21,6 @@ import org.apache.ignite.*; import org.apache.ignite.cluster.*; import org.apache.ignite.configuration.*; import org.apache.ignite.internal.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.util.typedef.*; import org.jetbrains.annotations.*; @@ -148,7 +147,7 @@ public class IgfsContext { * @param plc Policy. * @throws IgniteCheckedException In case of error. */ - public void send(UUID nodeId, Object topic, IgfsCommunicationMessage msg, GridIoPolicy plc) + public void send(UUID nodeId, Object topic, IgfsCommunicationMessage msg, byte plc) throws IgniteCheckedException { if (!kernalContext().localNodeId().equals(nodeId)) msg.prepareMarshal(kernalContext().config().getMarshaller()); @@ -163,7 +162,7 @@ public class IgfsContext { * @param plc Policy. * @throws IgniteCheckedException In case of error. */ - public void send(ClusterNode node, Object topic, IgfsCommunicationMessage msg, GridIoPolicy plc) + public void send(ClusterNode node, Object topic, IgfsCommunicationMessage msg, byte plc) throws IgniteCheckedException { if (!kernalContext().localNodeId().equals(node.id())) msg.prepareMarshal(kernalContext().config().getMarshaller()); http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/internal/processors/plugin/IgnitePluginProcessor.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/plugin/IgnitePluginProcessor.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/plugin/IgnitePluginProcessor.java index ecc6924..5e24b4b 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/processors/plugin/IgnitePluginProcessor.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/plugin/IgnitePluginProcessor.java @@ -28,7 +28,6 @@ import org.jetbrains.annotations.*; import java.io.*; import java.lang.reflect.*; -import java.security.*; import java.util.*; /** @@ -176,7 +175,7 @@ public class IgnitePluginProcessor extends GridProcessorAdapter { } /** - * Print plugin information. + * Print plugins information. */ private void ackPluginsInfo() { U.quietAndInfo(log, "Configured plugins:"); http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/main/java/org/apache/ignite/plugin/extensions/communication/IoPool.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/plugin/extensions/communication/IoPool.java b/modules/core/src/main/java/org/apache/ignite/plugin/extensions/communication/IoPool.java new file mode 100644 index 0000000..e87b82c --- /dev/null +++ b/modules/core/src/main/java/org/apache/ignite/plugin/extensions/communication/IoPool.java @@ -0,0 +1,42 @@ +/* + * 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.plugin.extensions.communication; + +import org.apache.ignite.plugin.*; + +import java.util.concurrent.*; + +/** + * The interface of IO Messaging Pool Extension. + */ +public interface IoPool extends Extension { + /** + * Gets the numeric identifier of the pool. This identifier is to be taken from serialized + * message and used to find the appropriate executor pool to process it. + * + * @return The id. + */ + public byte id(); + + /** + * Gets the Executor for this Pool. Cannot be null. + * + * @return The executor. + */ + public Executor executor(); +} http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/test/java/org/apache/ignite/internal/managers/communication/GridIoManagerSelfTest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/internal/managers/communication/GridIoManagerSelfTest.java b/modules/core/src/test/java/org/apache/ignite/internal/managers/communication/GridIoManagerSelfTest.java index 3890716..eb1fd88 100644 --- a/modules/core/src/test/java/org/apache/ignite/internal/managers/communication/GridIoManagerSelfTest.java +++ b/modules/core/src/test/java/org/apache/ignite/internal/managers/communication/GridIoManagerSelfTest.java @@ -189,7 +189,7 @@ public class GridIoManagerSelfTest extends GridCommonAbstractTest { } /** {@inheritDoc} */ - @Override public void send(ClusterNode node, GridTopic topic, Message msg, GridIoPolicy plc) + @Override public void send(ClusterNode node, GridTopic topic, Message msg, byte plc) throws IgniteCheckedException { // No-op. } http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/b437ec73/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteInternalCacheTypesTest.java ---------------------------------------------------------------------- diff --git a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteInternalCacheTypesTest.java b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteInternalCacheTypesTest.java index c9da7c5..8c2a0f8 100644 --- a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteInternalCacheTypesTest.java +++ b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/IgniteInternalCacheTypesTest.java @@ -20,7 +20,6 @@ package org.apache.ignite.internal.processors.cache; import org.apache.ignite.*; import org.apache.ignite.configuration.*; import org.apache.ignite.internal.*; -import org.apache.ignite.internal.managers.communication.*; import org.apache.ignite.internal.util.typedef.internal.*; import org.apache.ignite.spi.discovery.tcp.*; import org.apache.ignite.spi.discovery.tcp.ipfinder.*; @@ -146,7 +145,7 @@ public class IgniteInternalCacheTypesTest extends GridCommonAbstractTest { private void checkCache( Ignite ignite, String name, - GridIoPolicy plc, + byte plc, boolean user, boolean sysTx) { GridCacheAdapter cache = ((IgniteKernal)ignite).context().cache().internalCache(name);