Opened 7 years ago
Closed 7 years ago
#12727 closed bug (duplicate)
performance regression with comma-separated selectors
| Reported by: | Owned by: | ||
|---|---|---|---|
| Priority: | low | Milestone: | None |
| Component: | selector | Version: | 1.8.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
There appears to be a performance regression in v1.8.2 with regard to complex comma separated selectors. I've prepared a demo in which such a selector takes 2 orders of magnitude more in v1.8.2 than in v1.7.2 (running into several seconds). See - http://jsfiddle.net/yuvalshirav/y97n9/1/
Change History (1)
comment:1 Changed 7 years ago by
| Component: | unfiled → selector |
|---|---|
| Priority: | undecided → low |
| Resolution: | → duplicate |
| Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
This is a known regression. On a page with fewer elements, the current implementation is actually faster. The original idea was to remove the need for document sorting. However, given positional selectors and the fact that
jQuery.uniqueis a public method, this wasn't possible so we could switch it back to recursive handling of the selector split by commas and sorted afterwards. We'll see.Ported to Sizzle issue: https://github.com/jquery/sizzle/issues/164