x/stackalytics
Code Issues Proposed changes

Fixed traceback in get_release

Browse Source 
Error occurred when record timestamp was after the latest configured release.

Also corrected Icehouse release in test default_data

Change-Id: I0bbb4abb6ecc474e29ec63ae7aff7da625252896
This commit is contained in:
Ilya Shakhat 2014-04-09 15:44:52 +04:00
parent 1449e662fe
commit eba5e35df4
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
etc/test_default_data.json
View File

@ -141,7 +141,7 @@
},
{
"release_name": "IceHouse",
"end_date": "2014-Apr-04"
"end_date": "2014-Apr-17"
}
],

5
stackalytics/processor/record_processor.py
View File

@ -41,6 +41,11 @@ class RecordProcessor(object):
def _get_release(self, timestamp):
release_index = bisect.bisect(self.releases_dates, timestamp)
if release_index >= len(self.releases):
LOG.warn('Timestamp %s is beyond releases boundaries, the last '
'release will be used. Please consider adding a '
'new release into default_data.json', timestamp)
release_index = len(self.releases) - 1
return self.releases[release_index]['release_name']
def _get_modules(self):