Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034242Retail ModulesStoreServerpublic2016-10-19 18:212016-10-22 16:01
ReporterAugustoMauch 
Assigned ToStoreServer 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomtaal
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034242: Improve the CreateSynchronizationPropertiesIfNeededTask so that it does not require DAL to run
DescriptionCurrently CreateSynchronizationPropertiesIfNeededTask extends DalInitializingTask, which inits DAL before running the task. This takes most of the total time of the task, and adds a lot of overhead.

DAL is only used to retrieve some info about the mobile server. If xsql is used instead, the task will be much lighter.
Steps To Reproduce-
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0033944 closed AugustoMauch Prevent local db change detection by removing/restoring triggers before and after build 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-10-19 18:21AugustoMauchNew Issue
2016-10-19 18:21AugustoMauchAssigned To => StoreServer
2016-10-19 18:21AugustoMauchTriggers an Emergency Pack => No
2016-10-19 18:44AugustoMauchReview Assigned To => mtaal
2016-10-19 18:44AugustoMauchStatusnew => scheduled
2016-10-19 18:45hgbotCheckin
2016-10-19 18:45hgbotNote Added: 0090704
2016-10-19 18:45hgbotStatusscheduled => resolved
2016-10-19 18:45hgbotResolutionopen => fixed
2016-10-19 18:45hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.replication.symmetricds/rev/a6d375a844e6bfe4cfaa7ca416869ac1b5e81006 [^]
2016-10-19 18:47AugustoMauchRelationship addedrelated to 0033944
2016-10-22 16:01mtaalNote Added: 0090781
2016-10-22 16:01mtaalStatusresolved => closed

Notes
(0090704)
hgbot   
2016-10-19 18:45   
Repository: erp/pmods/org.openbravo.replication.symmetricds
Changeset: a6d375a844e6bfe4cfaa7ca416869ac1b5e81006
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Wed Oct 19 18:40:12 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.replication.symmetricds/rev/a6d375a844e6bfe4cfaa7ca416869ac1b5e81006 [^]

Fixes issue 34242: Task to create properties file no longer required DAL to run

The CreateSynchronizationPropertiesIfNeededTask task no longer requires DAL to run. This saves a lot of time and resources. Instead of using DAL to retrieve info about the mobile servers, now the info is retrieved using xsql.

---
M build.xml
M src/org/openbravo/replication/symmetricds/SetupSymmetricListener.java
M src/org/openbravo/replication/symmetricds/process/CreateEnginePropertiesComponent.java
M src/org/openbravo/replication/symmetricds/task/CreateSynchronizationPropertiesIfNeededTask.java
M src/org/openbravo/replication/symmetricds/util/SymmetricDSUtil_data.xsql
M src/org/openbravo/replication/symmetricds/util/SynchronizationPropertiesFileBuilder.java
---
(0090781)
mtaal   
2016-10-22 16:01   
Reviewed and tested