Author: rgodfrey
Date: Thu Apr 24 20:07:43 2014
New Revision: 1589862

URL: http://svn.apache.org/r1589862
Log:
QPID-5698 : Address review comments by Keith Wall

Modified:
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AccessControlProvider.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AuthenticationProvider.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Consumer.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Exchange.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Group.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupMember.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupProvider.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/KeyStore.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Session.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/TrustStore.java

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AccessControlProvider.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AccessControlProvider.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AccessControlProvider.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AccessControlProvider.java
 Thu Apr 24 20:07:43 2014
@@ -24,8 +24,6 @@ import org.apache.qpid.server.security.A
 @ManagedObject
 public interface AccessControlProvider<X extends AccessControlProvider<X>> 
extends ConfiguredObject<X>
 {
-    public static final String TYPE = "type";
-
     //retrieve the underlying AccessControl object
     AccessControl getAccessControl();
 }

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AuthenticationProvider.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AuthenticationProvider.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AuthenticationProvider.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/AuthenticationProvider.java
 Thu Apr 24 20:07:43 2014
@@ -32,9 +32,6 @@ import org.apache.qpid.server.security.a
 @ManagedObject( creatable = false )
 public interface AuthenticationProvider<X extends AuthenticationProvider<X>> 
extends ConfiguredObject<X>
 {
-
-    public static final String DESCRIPTION = "description";
-
     //children
     Collection<VirtualHostAlias> getVirtualHostPortBindings();
 

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Consumer.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Consumer.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Consumer.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Consumer.java
 Thu Apr 24 20:07:43 2014
@@ -28,7 +28,6 @@ public interface Consumer<X extends Cons
     public String NO_LOCAL = "noLocal";
     public String SELECTOR = "selector";
     public String SETTLEMENT_MODE = "settlementMode";
-    public String STATE = "state";
 
     @ManagedAttribute( automate = true )
     String getDistributionMode();

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Exchange.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Exchange.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Exchange.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Exchange.java
 Thu Apr 24 20:07:43 2014
@@ -28,7 +28,6 @@ import org.apache.qpid.server.message.Me
 @ManagedObject
 public interface Exchange<X extends Exchange<X>> extends ConfiguredObject<X>, 
MessageDestination
 {
-    String STATE                                = "state";
     String ALTERNATE_EXCHANGE                   = "alternateExchange";
 
     // Attributes

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Group.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Group.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Group.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Group.java
 Thu Apr 24 20:07:43 2014
@@ -22,5 +22,4 @@ package org.apache.qpid.server.model;
 @ManagedObject
 public interface Group<X extends Group<X>> extends ConfiguredObject<X>
 {
-    String STATE           = "state";
 }

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupMember.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupMember.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupMember.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupMember.java
 Thu Apr 24 20:07:43 2014
@@ -22,6 +22,4 @@ package org.apache.qpid.server.model;
 @ManagedObject
 public interface GroupMember<X extends GroupMember<X>> extends 
ConfiguredObject<X>
 {
-    String STATE           = "state";
-
 }

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupProvider.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupProvider.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupProvider.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/GroupProvider.java
 Thu Apr 24 20:07:43 2014
@@ -25,7 +25,5 @@ import java.util.Set;
 @ManagedObject
 public interface GroupProvider<X extends GroupProvider<X>> extends 
ConfiguredObject<X>
 {
-    public static final String DESCRIPTION = "description";
-
     Set<Principal> getGroupPrincipalsForUser(String username);
 }

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/KeyStore.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/KeyStore.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/KeyStore.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/KeyStore.java
 Thu Apr 24 20:07:43 2014
@@ -26,9 +26,5 @@ import javax.net.ssl.KeyManager;
 @ManagedObject( defaultType = "FileKeyStore" )
 public interface KeyStore<X extends KeyStore<X>> extends ConfiguredObject<X>
 {
-    String STATE = "state";
-    String DESCRIPTION = "description";
-
     public KeyManager[] getKeyManagers() throws GeneralSecurityException;
-
 }

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Session.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Session.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Session.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/Session.java
 Thu Apr 24 20:07:43 2014
@@ -25,8 +25,6 @@ import java.util.Collection;
 @ManagedObject
 public interface Session<X extends Session<X>> extends ConfiguredObject<X>
 {
-    String STATE = "state";
-
     String CHANNEL_ID = "channelId";
     // PRODUCER_FLOW_BLOCKED is exposed as an interim step.  We will expose 
attribute(s) that exposing
     // available credit of both producer and consumer sides.

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/TrustStore.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/TrustStore.java?rev=1589862&r1=1589861&r2=1589862&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/TrustStore.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/TrustStore.java
 Thu Apr 24 20:07:43 2014
@@ -26,12 +26,9 @@ import javax.net.ssl.TrustManager;
 @ManagedObject( defaultType = "FileTrustStore" )
 public interface TrustStore<X extends TrustStore<X>> extends 
ConfiguredObject<X>
 {
-    String DESCRIPTION = "description";
-
     @ManagedAttribute( secure = true, automate = true, mandatory = true)
     public String getPassword();
 
-
     public void setPassword(String password);
 
     public TrustManager[] getTrustManagers() throws GeneralSecurityException;



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to