Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 08635200B52 for ; Mon, 25 Jul 2016 20:01:41 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 072B3160A67; Mon, 25 Jul 2016 18:01:41 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 25595160A7D for ; Mon, 25 Jul 2016 20:01:39 +0200 (CEST) Received: (qmail 46192 invoked by uid 500); 25 Jul 2016 18:01:39 -0000 Mailing-List: contact commits-help@geode.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@geode.incubator.apache.org Delivered-To: mailing list commits@geode.incubator.apache.org Received: (qmail 46183 invoked by uid 99); 25 Jul 2016 18:01:39 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 25 Jul 2016 18:01:39 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id E95A81803EA for ; Mon, 25 Jul 2016 18:01:38 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -4.646 X-Spam-Level: X-Spam-Status: No, score=-4.646 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, KAM_LAZY_DOMAIN_SECURITY=1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.426] autolearn=disabled Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id niXPrGvuTCRI for ; Mon, 25 Jul 2016 18:01:37 +0000 (UTC) Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with SMTP id 7127B5FAEA for ; Mon, 25 Jul 2016 18:01:36 +0000 (UTC) Received: (qmail 46076 invoked by uid 99); 25 Jul 2016 18:01:36 -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; Mon, 25 Jul 2016 18:01:36 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id B794EE0100; Mon, 25 Jul 2016 18:01:35 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: klund@apache.org To: commits@geode.incubator.apache.org Date: Mon, 25 Jul 2016 18:01:36 -0000 Message-Id: <2e3dc1320d964b57962bf7b8532a2d3b@git.apache.org> In-Reply-To: <849dd694dd074902ad36c45a343b395d@git.apache.org> References: <849dd694dd074902ad36c45a343b395d@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [2/6] incubator-geode git commit: GEODE-17: fix javadoc warnings archived-at: Mon, 25 Jul 2016 18:01:41 -0000 GEODE-17: fix javadoc warnings Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/f230349f Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/f230349f Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/f230349f Branch: refs/heads/feature/GEODE-1673-PR-212 Commit: f230349f8b7994e4b2f35b06ebe417e8cf294945 Parents: fe28192 Author: Jinmei Liao Authored: Fri Jul 22 09:01:35 2016 -0700 Committer: Jinmei Liao Committed: Fri Jul 22 09:01:35 2016 -0700 ---------------------------------------------------------------------- .../gms/mgr/GMSMembershipManager.java | 63 +++++++++++++++----- .../internal/security/GeodeSecurityUtil.java | 4 +- .../apache/geode/security/GeodePermission.java | 13 ++-- .../security/templates/SamplePostProcessor.java | 2 +- 4 files changed, 57 insertions(+), 25 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/f230349f/geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/mgr/GMSMembershipManager.java ---------------------------------------------------------------------- diff --git a/geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/mgr/GMSMembershipManager.java b/geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/mgr/GMSMembershipManager.java index e6ca8e1..4ed30a4 100644 --- a/geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/mgr/GMSMembershipManager.java +++ b/geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/mgr/GMSMembershipManager.java @@ -16,17 +16,63 @@ */ package com.gemstone.gemfire.distributed.internal.membership.gms.mgr; -import com.gemstone.gemfire.*; +import java.io.IOException; +import java.io.NotSerializableException; +import java.util.ArrayList; +import java.util.Arrays; +import java.util.Collection; +import java.util.Collections; +import java.util.HashMap; +import java.util.HashSet; +import java.util.Iterator; +import java.util.LinkedHashMap; +import java.util.LinkedList; +import java.util.List; +import java.util.Map; +import java.util.Set; +import java.util.concurrent.ConcurrentHashMap; +import java.util.concurrent.CountDownLatch; +import java.util.concurrent.TimeUnit; +import java.util.concurrent.TimeoutException; +import java.util.concurrent.locks.Lock; +import java.util.concurrent.locks.ReadWriteLock; +import java.util.concurrent.locks.ReentrantReadWriteLock; + +import org.apache.logging.log4j.Logger; + +import com.gemstone.gemfire.CancelException; +import com.gemstone.gemfire.ForcedDisconnectException; +import com.gemstone.gemfire.GemFireConfigException; +import com.gemstone.gemfire.InternalGemFireError; +import com.gemstone.gemfire.SystemConnectException; +import com.gemstone.gemfire.SystemFailure; +import com.gemstone.gemfire.ToDataException; import com.gemstone.gemfire.cache.Cache; import com.gemstone.gemfire.distributed.DistributedMember; import com.gemstone.gemfire.distributed.DistributedSystem; import com.gemstone.gemfire.distributed.DistributedSystemDisconnectedException; import com.gemstone.gemfire.distributed.Locator; -import com.gemstone.gemfire.distributed.internal.*; +import com.gemstone.gemfire.distributed.internal.AdminMessageType; +import com.gemstone.gemfire.distributed.internal.DMStats; +import com.gemstone.gemfire.distributed.internal.DSClock; +import com.gemstone.gemfire.distributed.internal.DistributionConfig; +import com.gemstone.gemfire.distributed.internal.DistributionException; +import com.gemstone.gemfire.distributed.internal.DistributionManager; +import com.gemstone.gemfire.distributed.internal.DistributionMessage; +import com.gemstone.gemfire.distributed.internal.InternalDistributedSystem; +import com.gemstone.gemfire.distributed.internal.InternalLocator; +import com.gemstone.gemfire.distributed.internal.OverflowQueueWithDMStats; +import com.gemstone.gemfire.distributed.internal.SizeableRunnable; +import com.gemstone.gemfire.distributed.internal.StartupMessage; import com.gemstone.gemfire.distributed.internal.direct.DirectChannel; import com.gemstone.gemfire.distributed.internal.direct.DirectChannelListener; import com.gemstone.gemfire.distributed.internal.direct.ShunnedMemberException; -import com.gemstone.gemfire.distributed.internal.membership.*; +import com.gemstone.gemfire.distributed.internal.membership.DistributedMembershipListener; +import com.gemstone.gemfire.distributed.internal.membership.InternalDistributedMember; +import com.gemstone.gemfire.distributed.internal.membership.MembershipManager; +import com.gemstone.gemfire.distributed.internal.membership.MembershipTestHook; +import com.gemstone.gemfire.distributed.internal.membership.NetView; +import com.gemstone.gemfire.distributed.internal.membership.QuorumChecker; import com.gemstone.gemfire.distributed.internal.membership.gms.GMSMember; import com.gemstone.gemfire.distributed.internal.membership.gms.Services; import com.gemstone.gemfire.distributed.internal.membership.gms.SuspectMember; @@ -51,16 +97,6 @@ import com.gemstone.gemfire.internal.shared.StringPrintWriter; import com.gemstone.gemfire.internal.tcp.ConnectExceptions; import com.gemstone.gemfire.internal.tcp.MemberShunnedException; import com.gemstone.gemfire.internal.util.Breadcrumbs; -import org.apache.logging.log4j.Logger; - -import java.io.IOException; -import java.io.NotSerializableException; -import java.util.*; -import java.util.concurrent.ConcurrentHashMap; -import java.util.concurrent.CountDownLatch; -import java.util.concurrent.TimeUnit; -import java.util.concurrent.TimeoutException; -import java.util.concurrent.locks.*; public class GMSMembershipManager implements MembershipManager, Manager { @@ -2034,7 +2070,6 @@ public class GMSMembershipManager implements MembershipManager, Manager * * Concurrency: protected by {@link #latestViewLock} ReentrantReadWriteLock * - * @guarded.By latestViewLock * @return true if the given member is a zombie */ public boolean isShunned(DistributedMember m) { http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/f230349f/geode-core/src/main/java/com/gemstone/gemfire/internal/security/GeodeSecurityUtil.java ---------------------------------------------------------------------- diff --git a/geode-core/src/main/java/com/gemstone/gemfire/internal/security/GeodeSecurityUtil.java b/geode-core/src/main/java/com/gemstone/gemfire/internal/security/GeodeSecurityUtil.java index ead82e5..0d5e701 100644 --- a/geode-core/src/main/java/com/gemstone/gemfire/internal/security/GeodeSecurityUtil.java +++ b/geode-core/src/main/java/com/gemstone/gemfire/internal/security/GeodeSecurityUtil.java @@ -378,7 +378,7 @@ public class GeodeSecurityUtil { * @param className * @param expectedClazz * @param - * @return + * @return the expected object loadded by using the className */ public static T getObjectOfTypeFromClassName(String className, Class expectedClazz) { Class actualClass = null; @@ -407,7 +407,7 @@ public class GeodeSecurityUtil { * @param factoryMethodName * @param expectedClazz * @param - * @return + * @return the expected object loaded by the factory method */ public static T getObjectOfTypeFromFactoryMethod(String factoryMethodName, Class expectedClazz){ T actualObject = null; http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/f230349f/geode-core/src/main/java/org/apache/geode/security/GeodePermission.java ---------------------------------------------------------------------- diff --git a/geode-core/src/main/java/org/apache/geode/security/GeodePermission.java b/geode-core/src/main/java/org/apache/geode/security/GeodePermission.java index 0a777a8..9f0ce2d 100644 --- a/geode-core/src/main/java/org/apache/geode/security/GeodePermission.java +++ b/geode-core/src/main/java/org/apache/geode/security/GeodePermission.java @@ -43,32 +43,29 @@ public class GeodePermission extends WildcardPermission { } /** - * Returns the resource, could be either DATA or CLUSTER - * @return + * Returns + * @return the resource, could be either DATA or CLUSTER */ public Resource getResource() { return resource; } /** - * Returns the operation, could be either MANAGE, WRITE or READ - * @return + * @return the operation, could be either MANAGE, WRITE or READ */ public Operation getOperation() { return operation; } /** - * returns the regionName, could be "*", meaning all regions - * @return + * @return the regionName, could be "*", meaning all regions */ public String getRegionName() { return regionName; } /** - * returns the key, could be "*" meaning all keys. - * @return + * @return the key, could be "*" meaning all keys. */ public String getKey() { return key; http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/f230349f/geode-core/src/main/java/org/apache/geode/security/templates/SamplePostProcessor.java ---------------------------------------------------------------------- diff --git a/geode-core/src/main/java/org/apache/geode/security/templates/SamplePostProcessor.java b/geode-core/src/main/java/org/apache/geode/security/templates/SamplePostProcessor.java index 8f61db7..5eca744 100644 --- a/geode-core/src/main/java/org/apache/geode/security/templates/SamplePostProcessor.java +++ b/geode-core/src/main/java/org/apache/geode/security/templates/SamplePostProcessor.java @@ -42,7 +42,7 @@ public class SamplePostProcessor implements PostProcessor{ * the key of the value that's been accessed. This could be null. * @param value * the value, this could be null. - * @return + * @return the processed value */ @Override public Object processRegionValue(Principal principal,