Bug #13230

Regression: date range sorting is broken after ES upgrade

Added by Dan Gillean 10 months ago. Updated 9 months ago.

Status:VerifiedStart date:12/16/2019
Priority:MediumDue date:
Assignee:-% Done:

0%

Category:Search / BrowseEstimated time:8.00 hours
Target version:Release 2.6.0
Google Code Legacy ID: Tested version:2.5, 2.6
Sponsored:Yes Requires documentation:

Description

In the 2.5 release we upgraded from Elasticsearch v1.7 to ES v5.6. One side effect of this is that sorting archival description search/browse results by date range no longer works, due to changes in ES.

To reproduce

  • Navigate to Browse > Archival descriptions
  • Sort the page by start date
  • Now change to end date; try changing the ascending/descending order for each option, etc.

Resulting error

  • Results are not correctly sorted by date
  • Changing from start to end date, or changing the ascending/descending ordering, has no effect

Expected result

  • Date range sort works, and users can change between the different sort types and orderings.

Note: we should also confirm whether date range searching via the Advanced search panel is also affected.

one-hundred-simple-hierarchy.csv Magnifier (13.1 KB) Dan Gillean, 02/06/2020 11:58 AM

History

#1 Updated by Dan Gillean 10 months ago

  • Estimated time set to 16.00

#2 Updated by Dan Gillean 10 months ago

  • Estimated time changed from 16.00 to 8.00

#3 Updated by Darren Craze 9 months ago

  • Assignee set to Dan Gillean

#5 Updated by José Raddaoui Marín 9 months ago

  • Status changed from New to In progress
  • Assignee changed from Dan Gillean to José Raddaoui Marín
  • Target version set to Release 2.6.0

#6 Updated by José Raddaoui Marín 9 months ago

  • Status changed from In progress to Code Review

#7 Updated by Mike Cantelon 9 months ago

  • Status changed from Code Review to Feedback

Looks good to me!

#8 Updated by José Raddaoui Marín 9 months ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from José Raddaoui Marín to Dan Gillean

Merged in qa/2.6.x

#9 Updated by José Raddaoui Marín 9 months ago

The search index must be re-populated in order to test this changes.

#10 Updated by Dan Gillean 9 months ago

Attaching a test CSV, with date ranges in about 45-50 of the item-level descriptions.

There are occasionally weird blips - it doesn't seem to like the year date 2145 for example, and sorts it a bit strange - but i suspect these are behaviors in ES that require a deeper look (also difference in how YYYY vs YYYY-MM and YYYY-MM-DD are sorted). Overall, results were as expected - considering this complete.

Also available in: Atom PDF