[COMMONS-109] Miss-implication in logs for Deployment Synchronizor Created: 07/Mar/14 Updated: 01/Sep/14 Status: Project: Component/s: Affects Version/s: Fix Version/s: In Progress WSO2 Commons None None Type: Reporter: Resolution: Labels: Remaining Estimate: Time Spent: Original Estimate: Bug Priority: Pushpalanka Jayawardhana Assignee: Unresolved Votes: None Not Specified Attachments: SVNBasedArtifactRepository.patch Major Moderate Severity: Estimated Complexity: Attachment License: None Highest Kishanthan Thangarajah 0 Not Specified Not Specified patch.diff I agree to grant a license to WSO2 for this work for inclusion in WSO2 works as per the WSO2 Contributor License Agreement and the Apache License 2.0 Description This is regarding the code at [1], which is miss-interpreting the revision number of the svn update to be the numb } else { filesUpdated = svnClient.update(root, SVNRevision.HEAD, Depth.infinity, NO_SET_DEPTH, ignoreExternals, forceUpdate); if (log.isDebugEnabled()) { log.debug(filesUpdated + " files were updated using SVN Kit"); } } According to [2], what is actually returned is the revision number. The logic inside this function also consider th is incorrect. [1] - https://svn.wso2.org/repos/wso2/carbon/platform/branches/turing/components/deploymentsynchronizer/org.wso2.carbon.deployment.synchronizer.subversion/4.2.0/src/main/java/org/wso2/carbon/deploy [2] - https://svn.wso2.org/repos/wso2/carbon/platform/trunk/dependencies/svnclient/1.6.18.wso2v2/src/main/jav Comments Comment by Nipuni Perera [ 01/Sep/14 ] Hi, Pull request[1] sent to wso2-dev/carbon-commons. [1] https://github.com/wso2-dev/carbon-commons/pull/15 Thanks, Nipuni Generated at Wed Feb 10 07:38:11 IST 2016 using JIRA 6.0.1#6096sha1:e4a48bd73c6b8a4d99c824976ce5808b4c85857d.