Refactor #80
closed
Reports data should be accessed in a simple way
Description
Currently report status is stored in one integer, it should be broken down into multiple database tables allowing to reduce the size of the database (by not storing the entire Puppet::Transaction::Report).
this should allow to search in a more rails way for all hosts that has a report with change count > 20
Updated by Ohad Levy almost 15 years ago
- Target version deleted (
0.1-3)
as the new version of puppet is going to be a massive rewrite of how reports are handled, I'm going to push this db schema changes until its clear what info puppet will provide.
additionally, I was never really happy with my ideas for the schema structure, so any ideas are welcomed.
Updated by Ohad Levy almost 15 years ago
I've created http://github.com/ohadlevy/foreman/tree/report_scopes
I think it solves all of the topics in this issue without changing the database schema - I'll appreciate a code review :)
Updated by Ohad Levy almost 15 years ago
Ohad Levy wrote:
I've created http://github.com/ohadlevy/foreman/tree/report_scopes
I think it solves all of the topics in this issue without changing the database schema - I'll appreciate a code review :)
well.. I've just learned that indexes do not work on bitwise columns.. even though I really like that code...
Updated by Ohad Levy almost 15 years ago
- Status changed from New to Ready For Testing
- % Done changed from 0 to 100
Applied in changeset ff1cc6b1ff54003ac9916ecf389710fdfed2ed7b.
Updated by Ohad Levy almost 15 years ago
Applied in changeset 16cb774245863cfba6ceca91deec13449b7a1ce2.
Updated by Ohad Levy almost 15 years ago
- Status changed from Ready For Testing to Closed