Page MenuHomePhabricator

Skip Ferret fulltext columns in "ORDER BY" if there's no fulltext query
ClosedPublic

Authored by epriestley on Mar 18 2019, 10:52 PM.
Tags
None
Referenced Files
F15343035: D20296.diff
Sun, Mar 9, 10:08 PM
Unknown Object (File)
Mon, Mar 3, 9:04 AM
Unknown Object (File)
Mon, Mar 3, 9:04 AM
Unknown Object (File)
Mon, Mar 3, 9:04 AM
Unknown Object (File)
Mon, Mar 3, 9:04 AM
Unknown Object (File)
Mon, Mar 3, 9:03 AM
Unknown Object (File)
Feb 9 2025, 7:25 AM
Unknown Object (File)
Feb 9 2025, 7:25 AM
Subscribers
None

Details

Summary

Ref T13091. If you "Order By: Relevance" but don't actually specify a query, we currently raise a bare exception.

This operation is sort of silly/pointless, but it seems like it's probably best to just return the results for the other constraints in the fallback order (usually, by ID). Alternatively, we could raise a non-bare exception here ("You need to provide a fulltext query to order by relevance.")

Test Plan

Queried tasks by relevance with no actual query text.

Diff Detail

Repository
rP Phabricator
Branch
cursor6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22314
Build 30522: Run Core Tests
Build 30521: arc lint + arc unit