Use raw ID fields for the LocalSite fields in the admin UI.

Review Request #2047 — Created Jan. 16, 2011 and submitted

Information

Review Board

Reviewers

Use raw ID fields for the LocalSite fields in the admin UI.

The LocalSite fields may become very large up on certain large-scale
deployments. This changes them to be raw ID fields instead of a drop-down of
options, which will speed up browsing the admin UI.
Saw the fields in each admin page.
david
  1. I'm kind of dubious about this, but if it's really noticeable, okay.
    1. It all depends on how many we end up having. We'd be fine short-term, but I'd rather not hit a point down the road where things are suddenly slowing to a crawl because Django is loading and deserializing all these objects every time we hit an admin page. This was the case back in the day with other admin pages on RB, and the raw field made things much better.
      
      You can always click the little search icon next to the field and get a full browser interface with filtering to find the LocalSite you want.
    2. Okay, cool.
  2. 
      
Loading...