Project

General

Profile

Refactor #10584

Replace sort.uniq with uniq.sort

Added by Tomer Brisker over 6 years ago. Updated over 3 years ago.

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

Description

Sort usually runs in O(nlogn) while uniq runs in O(n).
If n is reduced by uniq it will improve the runtime, otherwise it will make no significant difference.

Associated revisions

Revision 57efe613 (diff)
Added by Tomer Brisker over 6 years ago

Fixes #10584 - Replace sort.uniq with uniq.sort

History

#1 Updated by The Foreman Bot over 6 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2406 added
  • Pull request deleted ()

#2 Updated by Anonymous over 6 years ago

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

#3 Updated by Dominic Cleal over 6 years ago

  • Legacy Backlogs Release (now unused) set to 35

Also available in: Atom PDF