Merge lp://staging/~brian-murray/oops-repository/bucketversion into lp://staging/~ev/oops-repository/whoopsie-daisy

Proposed by Brian Murray
Status: Needs review
Proposed branch: lp://staging/~brian-murray/oops-repository/bucketversion
Merge into: lp://staging/~ev/oops-repository/whoopsie-daisy
Diff against target: 46 lines (+17/-0)
3 files modified
oopsrepository/oopses.py (+4/-0)
oopsrepository/schema.py (+4/-0)
oopsrepository/tests/test_oopses.py (+9/-0)
To merge this branch: bzr merge lp://staging/~brian-murray/oops-repository/bucketversion
Reviewer Review Type Date Requested Status
Evan Needs Information
Review via email: mp+149180@code.staging.launchpad.net

Description of the change

This adds the bucketversion cf which uses the custom comparator.

To post a comment you must log in.
Revision history for this message
Evan (ev) wrote :

Would this still work for your needs if it was a composite type of (DpkgVersionComparator, UTF8) where the latter part of the tuple represents the Ubuntu release that version is responsible for? The ordering matters here, as we want to sort on the dpkg version primarily.

Getting the first version this issue was ever seen in would still be:
bucketversions_cf.get(column_count=1) -> ('1.0-0ubuntu1', 'Ubuntu 12.04')

Getting the total count of instances for the first version this issue was ever seen in, regardless of release, would require two operations as far as I can tell [1]:
bucketversions_cf.get(column_count=1) -> ('1.0-0ubuntu1', 'Ubuntu 12.04')
bucketversions_cf.get_range(column_start='1.0-0ubuntu1', column_finish='1.0-0ubuntu2~')

But if we grabbed the entire row, we could build an accurate table of "package versions with this error", as described in the specification:
https://wiki.ubuntu.com/ErrorTracker#Bucket_page

And mentioned in the following bug report:
https://bugs.launchpad.net/errors/+bug/1078801

Alternatively we could have another column family responsible for this finer granularity.

1: http://mail-archives.apache.org/mod_mbox/cassandra-user/201208.mbox/%<email address hidden>%3E

review: Needs Information

Unmerged revisions

58. By Brian Murray

add and use bucketversion cf which uses a dpkgversiontype custom comparator

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: