Project: bamboo
Issue BAM-5358: Persist some calculated data for ArtifactLink
From: https://atlaseye.atlassian.com/cru/CR-BAM-997#c17472 Chai: Is there any reason why we wouldn't simply store the calculated values? One of the things was that size was not calculated for directories in fear that it was going to be too expensive (which it can be if artifacts are big). Thoughts? Krystian: If we decide to persist the url then we shall remember to update it once the buildKey will get changed (build move) which in my opinion adds complexity. We can however store a path with some kind of token to be replaced by planKey while property is being read ("/browse/%s/artifacts/build-123/whatever") If you want to persist artifact size as well, then we might calculate directory size (as it would be done once). Chai: Yeah for URL we could just store the calculated suffix (ie the thing that requires us to browse for an index.html etc) and leave out the prefix (that can just be apended, it's not expensive) + we should store the full size for the directory, though it'd be good to do some testing to see if how much more time it adds (say for a Clover report over JIRA is a good test)
Story points: 2