Project

Profile

Help

Issue #385

closed

Our base Model class indexes the id field as a unique field by default, causing duplicate indexes on at least one model

Added by rbarlow about 9 years ago. Updated almost 5 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version:
Master
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

We have an opportunity to reduce our MongoDB memory usage. I discovered recently that our base Model class indexes the id field as a unique index. The id field is defaulted to a string representation of the _id field (which probably isn't terribly useful). Since _id must always be indexed in MongoDB, we essentially index that same information two different ways (which uses more RAM).

I am not sure how many models this affects, but I did find at least one: the CallResource Model. It doesn't override the id field, and it does not override the unique_indexes class attribute.

+ This bug was cloned from Bugzilla Bug #1033195 +


Related issues

Related to Pulp - Refactor #765: Convert Pulp to use MongoEngineCLOSED - WONTFIX

Actions
Actions #1

Updated by bmbouter about 9 years ago

  • Severity changed from Medium to 2. Medium
Actions #2

Updated by mhrivnak almost 9 years ago

  • Severity changed from 2. Medium to 1. Low

Since we are moving to mongoengine, we should probably just let this bug get completed as part of that work.

Actions #3

Updated by mhrivnak almost 9 years ago

  • Related to Refactor #765: Convert Pulp to use MongoEngine added
Actions #4

Updated by bmbouter almost 5 years ago

  • Status changed from NEW to CLOSED - WONTFIX
Actions #5

Updated by bmbouter almost 5 years ago

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

Actions #6

Updated by bmbouter almost 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF