Project

Profile

Help

Issue #385

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 over 5 years ago. Updated over 1 year 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

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

History

#1 Updated by bmbouter over 5 years ago

  • Severity changed from Medium to 2. Medium

#2 Updated by mhrivnak over 5 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.

#3 Updated by mhrivnak over 5 years ago

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

#4 Updated by bmbouter over 1 year ago

  • Status changed from NEW to CLOSED - WONTFIX

#5 Updated by bmbouter over 1 year 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.

#6 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF