Liran Zelkha has uploaded a new change for review.

Change subject: engine: Datasource configuration fixes
......................................................................

engine: Datasource configuration fixes

Fix database configuration based on JBoss team suggestion. Allow to recover 
from DB failure.

Change-Id: Ib7cbe97f0baf85f8b6785726380a7a4dcb9d20f8
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=995084
Signed-off-by: lzel...@redhat.com <liran.zel...@gmail.com>
---
M backend/manager/conf/standalone.xml
M packaging/services/ovirt-engine/ovirt-engine.conf.in
M packaging/services/ovirt-engine/ovirt-engine.xml.in
3 files changed, 8 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/35/19735/1

diff --git a/backend/manager/conf/standalone.xml 
b/backend/manager/conf/standalone.xml
index 241473a..009e518 100644
--- a/backend/manager/conf/standalone.xml
+++ b/backend/manager/conf/standalone.xml
@@ -145,7 +145,7 @@
         <subsystem xmlns="urn:jboss:domain:configadmin:1.0"/>
         <subsystem xmlns="urn:jboss:domain:datasources:1.0">
             <datasources>
-                <datasource jndi-name="java:/ENGINEDataSource" 
pool-name="ENGINEDataSource" enabled="true">
+                <datasource jndi-name="java:/ENGINEDataSource" 
pool-name="ENGINEDataSource" enabled="true" use-ccm="false">
                     <connection-url>
                         jdbc:postgresql://localhost:5432/engine
                     </connection-url>
@@ -157,7 +157,7 @@
                     </transaction-isolation>
                     <pool>
                         <min-pool-size>
-                            1
+                           75
                         </min-pool-size>
                         <max-pool-size>
                             100
@@ -175,9 +175,11 @@
                         <prepared-statement-cache-size>
                             100
                         </prepared-statement-cache-size>
+                        <share-prepared-statements/>
                     </statement>
                     <validation>
                         <check-valid-connection-sql>select 
1</check-valid-connection-sql>
+                        <background-validation>true</background-validation>
                     </validation>
                 </datasource>
                 <drivers>
diff --git a/packaging/services/ovirt-engine/ovirt-engine.conf.in 
b/packaging/services/ovirt-engine/ovirt-engine.conf.in
index c4c1aa4..0cdc9c6 100644
--- a/packaging/services/ovirt-engine/ovirt-engine.conf.in
+++ b/packaging/services/ovirt-engine/ovirt-engine.conf.in
@@ -180,7 +180,7 @@
 #
 # Size of the database connection pool:
 #
-ENGINE_DB_MIN_CONNECTIONS=1
+ENGINE_DB_MIN_CONNECTIONS=75
 ENGINE_DB_MAX_CONNECTIONS=100
 
 #
diff --git a/packaging/services/ovirt-engine/ovirt-engine.xml.in 
b/packaging/services/ovirt-engine/ovirt-engine.xml.in
index f11cbfb..2397383 100644
--- a/packaging/services/ovirt-engine/ovirt-engine.xml.in
+++ b/packaging/services/ovirt-engine/ovirt-engine.xml.in
@@ -132,7 +132,7 @@
 
       <datasources>
 
-        <datasource jndi-name="java:/ENGINEDataSource" 
pool-name="ENGINEDataSource" enabled="true">
+        <datasource jndi-name="java:/ENGINEDataSource" 
pool-name="ENGINEDataSource" enabled="true" use-ccm="false">
           
<connection-url><![CDATA[$getstring('ENGINE_DB_URL')]]></connection-url>
           <driver>postgresql</driver>
           
<transaction-isolation>TRANSACTION_READ_COMMITTED</transaction-isolation>
@@ -147,9 +147,11 @@
           </security>
           <statement>
             <prepared-statement-cache-size>100</prepared-statement-cache-size>
+            <share-prepared-statements/>
           </statement>
           <validation>
             <check-valid-connection-sql>select 1</check-valid-connection-sql>
+            <background-validation>true</background-validation>
           </validation>
         </datasource>
 


-- 
To view, visit http://gerrit.ovirt.org/19735
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib7cbe97f0baf85f8b6785726380a7a4dcb9d20f8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liran Zelkha <liran.zel...@gmail.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to