Hi, Please review and merge pull request[1] for jira issue [2]
[1] https://github.com/wso2-dev/carbon-commons/pull/15 [2] https://wso2.org/jira/browse/CARBON-14723 Thanks, Nipuni On Tue, Jul 29, 2014 at 5:42 PM, Nipuni Perera <nip...@wso2.com> wrote: > No, Manager-node changes it's revision number with updates. (Recent > modifications to dep-sync calls checkout() method inside commit(). Just > checked with kasunG). I didn't have recent update in my local repo and > didn't noticed changes in revision numbers. According to current updates > revision numbers change with updates, in both manager and worker nodes. > > Thanks, > Nipuni > > > On Tue, Jul 29, 2014 at 4:28 PM, Kishanthan Thangarajah < > kishant...@wso2.com> wrote: > >> -- Sent from mobile -- >> >> On 29 Jul 2014 13:36, "Nipuni Perera" <nip...@wso2.com> wrote: >> > >> > Hi, >> > >> > I have updated the above patch with the attachment >> SVNBasedArtifactRepository.patch. When svn is update by a manager-node, svn >> get a new revision number with the update but manager node remains in the >> same revision number. It does not call checkout() or update() methods >> above. If I restart the manger-node, there I can observe two different >> values for newRevisionNumberLog and lastRevisionNumber inside checkout() >> method of manager-node. >> >> Is it the same behaviour without your fixes too? The revision number >> cannot change unless there is a check-in. >> >> > >> > I have sent a pull request for the above fix. >> > >> > Thanks, >> > Nipuni >> > >> > >> > On Mon, Jul 28, 2014 at 11:21 AM, Nipuni Perera <nip...@wso2.com> >> wrote: >> >> >> >> Hi, >> >> >> >> I have update the log messages and methods mentioned earlier. Find the >> attached patch. >> >> >> >> Thanks, >> >> Nipuni >> >> >> >> >> >> On Wed, Jul 9, 2014 at 3:35 PM, Nipuni Perera <nip...@wso2.com> wrote: >> >>> >> >>> >> >>> Hi, >> >>> >> >>> I am working on $subject.Three methods below >> >>> >> >>> checkout(int tenantId, String filePath, int depth) >> >>> >> >>> update(int tenantId, String rootPath, String filePathToUpdate, int >> depth) >> >>> >> >>> checkout(int tenantId, String filePath) >> >>> >> >>> in [1] uses the code, >> >>> >> >>> filesUpdated = svnClient.update(root, >> SVNRevision.HEAD,Depth.infinity, NO_SET_DEPTH,ignoreExternals, forceUpdate); >> >>> >> >>> svnClient.update(..) returns revision number but assigned to >> filesUpdated. The method return value is set to >> >>> >> >>> filesUpdated >1 >> >>> >> >>> which is incorrect. According to javadocs[2], methods should return >> true if files were checked out or updates, false otherwise. If files were >> changed, it can be detected comparing revision numbers. Therefore the >> return value would be, >> >>> >> >>> long lastRevisionNumber = >> svnClient.getInfo(root).getLastChangedRevision(); >> >>> long newRevisionNumber = svnClient.update(root, >> SVNRevision.HEAD,Depth.infinity, NO_SET_DEPTH,ignoreExternals, forceUpdate); >> >>> >> >>> return newRevisionNumber>lastRevisionNumber; >> >>> >> >>> This gives different values for newRevisionNumberLog and >> lastRevisionNumber, when I debug with a worker node, but gave same value >> for both variables for manager node. What could be the issue here?. Log >> messages should also updated accordingly. Is there any other issue to >> consider? >> >>> >> >>> [1] >> https://github.com/wso2-dev/carbon-commons/blob/master/components/deployment-synchronizer/org.wso2.carbon.deployment.synchronizer.subversion/src/main/java/org/wso2/carbon/deployment/synchronizer/subversion/SVNBasedArtifactRepository.java >> >>> >> >>> [2] >> https://svn.wso2.org/repos/wso2/carbon/platform/trunk/components/deployment-synchronizer/org.wso2.carbon.deployment.synchronizer/src/main/java/org/wso2/carbon/deployment/synchronizer/ArtifactRepository.java >> >>> >> >>> Thanks, >> >>> Nipuni >> >>> -- >> >>> Nipuni Perera >> >>> Software Engineer; WSO2 Inc.; http://wso2.com >> >>> Email: nip...@wso2.com >> >>> Git hub profile: https://github.com/nipuni >> >>> Mobile: +94 (71) 5626680 >> >>> >> >> >> >> >> >> >> >> -- >> >> Nipuni Perera >> >> Software Engineer; WSO2 Inc.; http://wso2.com >> >> Email: nip...@wso2.com >> >> Git hub profile: https://github.com/nipuni >> >> Mobile: +94 (71) 5626680 >> >> >> > >> > >> > >> > -- >> > Nipuni Perera >> > Software Engineer; WSO2 Inc.; http://wso2.com >> > Email: nip...@wso2.com >> > Git hub profile: https://github.com/nipuni >> > Mobile: +94 (71) 5626680 >> > >> > > > > -- > Nipuni Perera > Software Engineer; WSO2 Inc.; http://wso2.com > Email: nip...@wso2.com > Git hub profile: https://github.com/nipuni > Mobile: +94 (71) 5626680 > <http://wso2.com> > > -- Nipuni Perera Software Engineer; WSO2 Inc.; http://wso2.com Email: nip...@wso2.com Git hub profile: https://github.com/nipuni Mobile: +94 (71) 5626680 <http://wso2.com>
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev