Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 333038 - Dashboard reporting dates that can't be true
Summary: Dashboard reporting dates that can't be true
Status: RESOLVED WONTFIX
Alias: None
Product: Community
Classification: Eclipse Foundation
Component: Dashboard (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Portal Bugzilla Dummy Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-21 14:20 EST by Wayne Beaton CLA
Modified: 2013-11-25 16:54 EST (History)
0 users

See Also:


Attachments
Screenshot of the stats for eBAM (15.36 KB, image/png)
2010-12-21 17:21 EST, Wayne Beaton CLA
no flags Details
Patch to clear the buffer after writting it out (542 bytes, patch)
2010-12-22 14:12 EST, Wayne Beaton CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Wayne Beaton CLA 2010-12-21 14:20:47 EST
Dashboard is reporting 13 commits in 201007 for the eBAM project (which was provisioned in 2010).

I've scanned SVN and am pretty sure that the commits don't actually existing in SVN. I've run the Perl extraction scripts and they don't seem to be reporting these dates. Somehow, they've crept into the database.
Comment 2 Wayne Beaton CLA 2010-12-22 14:10:42 EST
The svn-parse.pl script has a bug.

It parses through the output of the corresponding svn-extract.pl script, extracting the project name, and other particulars. It also buffers the names of the files contained in the commit. When it encounters an "end of commit" line (all dashes), it dumps the buffered information to stdout.

Example extract from the input:

...
~~~~~PROJECT: dsdp.sequoyah
------------------------------------------------------------------------
r1 | (no author) | 2007-12-11 15:57:54 -0500 (Tue, 11 Dec 2007) | 1 line
Changed paths:
   A /branches
   A /tags
   A /trunk
   A /trunk/org.eclipse.sequoyah.device
   A /trunk/org.eclipse.sequoyah.device/examples
   A /trunk/org.eclipse.sequoyah.device/features
   A /trunk/org.eclipse.sequoyah.device/plugins
   A /trunk/org.eclipse.sequoyah.localization
   A /trunk/org.eclipse.sequoyah.localization/features
   A /trunk/org.eclipse.sequoyah.localization/plugins
   A /trunk/org.eclipse.sequoyah.vnc
   A /trunk/org.eclipse.sequoyah.vnc/features
   A /trunk/org.eclipse.sequoyah.vnc/plugins

New repository initialized by cvs2svn.
------------------------------------------------------------------------
~~~~~PROJECT: soa.ebam
------------------------------------------------------------------------
r206 | abernabei | 2010-12-22 08:00:16 -0500 (Wed, 22 Dec 2010) | 1 line
Changed paths:
   M /trunk/org.eclipse.ebam.engine/src/org/eclipse/ebam/customengine/rules/pingSco/PingScoNumberErrorRule.java
...

When it encounters the line of dashes above the '~~~~~PROJECT: soa.ebam' line, it dumps the information for the files in the proceeding section. When it encounters the second line of dashes, it dumps again. Only the second time, the project has been changed from dsdp.sequoyah to soa.ebam. So incorrect data gets written out.

The solution is to clear the buffer after it is written. If a second attempt is made to dump the buffer, nothing happens.

I'll attach a patch.

The better solution is to replace this whole mess :-)
Comment 3 Wayne Beaton CLA 2010-12-22 14:12:03 EST
Created attachment 185731 [details]
Patch to clear the buffer after writting it out

Matt has made some changes directly in the file system that need to be consolidated with this change.
Comment 4 Wayne Beaton CLA 2013-11-25 16:54:19 EST
SVN support is deprecated. Dash is being reimplemented without consideration for SVN repositories. Marking as WONTFIX.