[ 
https://issues.apache.org/jira/browse/CONTINUUM-2765?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Brent N Atkinson closed CONTINUUM-2765.
---------------------------------------
       Resolution: Fixed
    Fix Version/s: 1.5.0
         Assignee: Brent N Atkinson

I decided to reintroduce this because it was already being computed. Fixed in 
r1677905.

> Changes since last success appears to always compute no changes
> ---------------------------------------------------------------
>
>                 Key: CONTINUUM-2765
>                 URL: https://issues.apache.org/jira/browse/CONTINUUM-2765
>             Project: Continuum
>          Issue Type: Bug
>    Affects Versions: 1.4.2, 1.5.0
>            Reporter: Brent N Atkinson
>            Assignee: Brent N Atkinson
>             Fix For: 1.5.0
>
>
> While attempting to find the root cause of CONTINUUM-2762, I discovered that 
> the code to compute the changes since the last successful build (for display 
> on the build results page and in notification emails) will never result in 
> changes to display.
> Specifically, {{DefaultContinuum.getChangesSinceLastSuccess(...)}} appears to 
> be implemented with an algorithm matching the following pseudocode:
> {noformat}
> var changes = [ ]
> var results = fetchResultsBetween(lastSuccessfulResult, currentBuildResult)
> var currentBuildIndex = results.indexOf(currentBuildResult)
> if (currentBuildIndex > 0) {
>   for (var result in results, starting at currentBuildIndex) {
>     changes.addUniqueChanges(result)
>   }
> }
> return changes
> {noformat}
> The problem with this algorithm is that the fetched results never include the 
> current build result, so no changes are ever computed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to