downloadstats latlng feed contains data points for all versions of Firefox

RESOLVED FIXED

Status

Websites
downloadstats.mozilla.com
--
critical
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: dre, Assigned: Steve Herskovitz)

Tracking

Details

(URL)

Not sure if this is on SQLstream's side or ours.
Can we take a look at the script that generates this feed and see if anything unusual is going on?
Dave, can you link me where it is in SVN?
This is fixed in SVN, but this should get fixed on the SQLstream side.
Assignee: nobody → hersker
Steve: note that the python script in comment #1 is the script I was talking about being cronjobbed.
http://svn.mozilla.org/metrics/downloadvis/bin/updatejson.py makes a call to the servlet endpoint:
/downloadmonitor/?cmd=getLocationReport&version=%s

We would like it to be able to omit the version parameter and use the same default version that the getCountryReport endpoint uses.
(Assignee)

Comment 4

8 years ago
As of 2010-04-18 at 20:00 PDT there is a new servlet deployed which fixes this problem.

Daniel: sorry for the delay.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Steve,

to clarify I can just lob off the version parameter and I'll be recieving 3.6.4 downloads information (presumable 3.6.4 will be out soon enough).
Yes. You can now omit the version parameter just like we do for the table feed and it will use the currently configured default.  You should go ahead and do so when you can so that the data in the map and table will be in sync again.
(Assignee)

Comment 7

8 years ago
My commit collided with Daniel's! Here's some more detail...

Dave, yes, you should be able to remove the version parameter. 

I should have provided more information about the fix:

The bug report was that getLocationReport wasn't honoring the default
versionFilter provided in the properties file. I fixed this problem for both
getLocationReport and getGeographicReport (but am not sure you are still using
getGeographicReport).

The &version=n.n.n URL parameter overrides the default versionFilter property.
Removing &version directs the servlet to use the default versionFilter instead.
You can continue to use &version anytime you want to request data for a
non-default version.

The default versionFilter can be set via the shockingly bright orange page at
http://cm-metricsapp02:8080/downloadmonitor/?cmd=propertiesPage

This page displays/updates a file on cm-metricsapp02 at
/usr/local/share/sqlstream/mozdlmon/downloadmonitor.properties

This file is read by the servlet only at startup, so you should generally not
edit the properties file manually. The propertiesPage updates the properties
file and the internal versionFilter. Changes made through propertiesPage take
effect immediately with no servlet/tomcat restart required.
You need to log in before you can comment on or make changes to this bug.