Project

General

Profile

Actions

Refactor #3016

closed

accessible_hostgroups method unnecessarily sorts in memory

Added by Ohad Levy almost 11 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Performance
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

since we added the full label to the db, we can ask the db to sort for us.

see source:app/helpers/hosts_and_hostgroups_helper.rb#L20

Actions #1

Updated by Ohad Levy almost 11 years ago

also, changing the order to check first for an admin should improve the query a bit (when you are an admin).

Actions #2

Updated by The Foreman Bot over 9 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2215 added
Actions #3

Updated by Tomer Brisker over 9 years ago

  • Assignee set to Tomer Brisker
Actions #4

Updated by Anonymous over 9 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #5

Updated by Marek Hulán over 9 years ago

  • Category set to Performance
  • Translation missing: en.field_release set to 35
Actions

Also available in: Atom PDF