You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by or...@apache.org on 2017/03/03 14:08:31 UTC
svn commit: r1785308 - in /qpid/java/branches/6.1.x: ./
broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhost/jdbc/
broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhostnode/jdbc/
broker-plugins/jdbc-store/src/...
Author: orudyy
Date: Fri Mar 3 14:08:31 2017
New Revision: 1785308
URL: http://svn.apache.org/viewvc?rev=1785308&view=rev
Log:
QPID-7558: [Java Broker] Make attributes for table name prefix unsecure
and immutable, and modify corresponding UI to allow entering
of table prefixes.
merged from trunk using
svn merge -c 1785269 ^/qpid/java/trunk
merge conflicts resolved manually
Modified:
qpid/java/branches/6.1.x/ (props changed)
qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhost/jdbc/JDBCVirtualHost.java
qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhostnode/jdbc/JDBCVirtualHostNode.java
qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhost/jdbc/add.html
qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhostnode/jdbc/add.html
Propchange: qpid/java/branches/6.1.x/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Mar 3 14:08:31 2017
@@ -9,5 +9,5 @@
/qpid/branches/java-broker-vhost-refactor/java:1493674-1494547
/qpid/branches/java-network-refactor/qpid/java:805429-821809
/qpid/branches/qpid-2935/qpid/java:1061302-1072333
-/qpid/java/trunk:1766544,1766547,1766553,1766666,1766796-1766797,1766806,1767251,1767267-1767268,1767275,1767310,1767326,1767329,1767332,1767514,1767523,1767738,1767825,1767847-1767849,1767882,1767909,1767914,1768016-1768017,1768065,1768643,1768704,1768854,1768875,1768914,1768963,1768967,1768976,1769007,1769009,1769087,1769138-1769139,1769597,1769879,1770236,1770716,1772050,1772241,1772365,1772574,1773057,1774039,1774446,1774564,1774885,1775087,1775100,1777939,1780947,1782302,1782735,1785117,1785270
+/qpid/java/trunk:1766544,1766547,1766553,1766666,1766796-1766797,1766806,1767251,1767267-1767268,1767275,1767310,1767326,1767329,1767332,1767514,1767523,1767738,1767825,1767847-1767849,1767882,1767909,1767914,1768016-1768017,1768065,1768643,1768704,1768854,1768875,1768914,1768963,1768967,1768976,1769007,1769009,1769087,1769138-1769139,1769597,1769879,1770236,1770716,1772050,1772241,1772365,1772574,1773057,1774039,1774446,1774564,1774885,1775087,1775100,1777939,1780947,1782302,1782735,1785117,1785269-1785270
/qpid/trunk/qpid:796646-796653
Modified: qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhost/jdbc/JDBCVirtualHost.java
URL: http://svn.apache.org/viewvc/qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhost/jdbc/JDBCVirtualHost.java?rev=1785308&r1=1785307&r2=1785308&view=diff
==============================================================================
--- qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhost/jdbc/JDBCVirtualHost.java (original)
+++ qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhost/jdbc/JDBCVirtualHost.java Fri Mar 3 14:08:31 2017
@@ -45,13 +45,15 @@ public interface JDBCVirtualHost<X exten
@ManagedOperation(description = "Resets statistics on this object and all child objects", changesConfiguredObjectState = false, nonModifying = true)
void resetStatistics();
- @ManagedContextDefault(name = "jdbcvirtualhost.tableNamePrefix")
+ @ManagedContextDefault(name = "jdbcvirtualhost.tableNamePrefix",
+ description = "Default value for optional database table prefix")
String DEFAULT_JDBC_VIRTUALHOST_TABLE_NAME_PREFIX = "";
- @ManagedAttribute(secure = true,
+ @ManagedAttribute(
description = "Optional database table prefix so multiple VirtualHosts can share the same database",
defaultValue = "${jdbcvirtualhost.tableNamePrefix}",
- validValuePattern = "[a-zA-Z_0-9]*")
+ validValuePattern = "[a-zA-Z_0-9]*",
+ immutable = true)
String getTableNamePrefix();
}
Modified: qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhostnode/jdbc/JDBCVirtualHostNode.java
URL: http://svn.apache.org/viewvc/qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhostnode/jdbc/JDBCVirtualHostNode.java?rev=1785308&r1=1785307&r2=1785308&view=diff
==============================================================================
--- qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhostnode/jdbc/JDBCVirtualHostNode.java (original)
+++ qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/org/apache/qpid/server/virtualhostnode/jdbc/JDBCVirtualHostNode.java Fri Mar 3 14:08:31 2017
@@ -48,12 +48,14 @@ public interface JDBCVirtualHostNode<X e
defaultValue = "{\"type\": \"Provided\"}")
PreferenceStoreAttributes getPreferenceStoreAttributes();
- @ManagedContextDefault(name = "jdbcvirtualhostnode.tableNamePrefix")
+ @ManagedContextDefault(name = "jdbcvirtualhostnode.tableNamePrefix",
+ description = "Default value for optional database table prefix")
String DEFAULT_JDBC_VIRTUALHOSTNODE_TABLE_NAME_PREFIX = "";
- @ManagedAttribute(secure = true,
+ @ManagedAttribute(
description = "Optional database table prefix so multiple VirtualHostNodes can share the same database",
defaultValue = "${jdbcvirtualhostnode.tableNamePrefix}",
- validValuePattern = "[a-zA-Z_0-9]*")
+ validValuePattern = "[a-zA-Z_0-9]*",
+ immutable = true)
String getTableNamePrefix();
}
Modified: qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhost/jdbc/add.html
URL: http://svn.apache.org/viewvc/qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhost/jdbc/add.html?rev=1785308&r1=1785307&r2=1785308&view=diff
==============================================================================
--- qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhost/jdbc/add.html (original)
+++ qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhost/jdbc/add.html Fri Mar 3 14:08:31 2017
@@ -69,7 +69,20 @@
placeHolder: 'Select pool type'" />
</div>
</div>
-
+ <div class="clear">
+ <div class="formLabel-labelCell tableContainer-labelCell">Table name prefix:</div>
+ <div class="formLabel-controlCell tableContainer-valueCell">
+ <input type="text" id="addVirtualHost.tableNamePrefix"
+ data-dojo-type="dijit/form/ValidationTextBox"
+ data-dojo-props="
+ name: 'tableNamePrefix',
+ placeHolder: 'table name prefix',
+ required: false,
+ regExp: '[a-zA-Z_0-9]*',
+ promptMessage: 'Optional database table prefix so multiple VirtualHosts can share the same database',
+ title: 'Optional database table prefix so multiple VirtualHosts can share the same database'" />
+ </div>
+ </div>
<div class="clear"></div>
<div id="addVirtualHost.poolSpecificDiv"></div>
<div class="infoMessage">The virtual host will have the same name as the node.</div>
Modified: qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhostnode/jdbc/add.html
URL: http://svn.apache.org/viewvc/qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhostnode/jdbc/add.html?rev=1785308&r1=1785307&r2=1785308&view=diff
==============================================================================
--- qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhostnode/jdbc/add.html (original)
+++ qpid/java/branches/6.1.x/broker-plugins/jdbc-store/src/main/java/resources/virtualhostnode/jdbc/add.html Fri Mar 3 14:08:31 2017
@@ -71,7 +71,20 @@
placeHolder: 'Select pool type'" />
</div>
</div>
-
+ <div class="clear">
+ <div class="formLabel-labelCell tableContainer-labelCell">Table name prefix:</div>
+ <div class="formLabel-controlCell tableContainer-valueCell">
+ <input type="text" id="addVirtualHostNode.tableNamePrefix"
+ data-dojo-type="dijit/form/ValidationTextBox"
+ data-dojo-props="
+ name: 'tableNamePrefix',
+ placeHolder: 'table name prefix',
+ required: false,
+ regExp: '[a-zA-Z_0-9]*',
+ promptMessage: 'Optional database table prefix so multiple VirtualHostNodes can share the same database',
+ title: 'Optional database table prefix so multiple VirtualHostNodes can share the same database'" />
+ </div>
+ </div>
<div class="clear"></div>
<div id="addVirtualHostNode.poolSpecificDiv"></div>
</div>
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org