Page MenuHomePhabricator

Add a "query compiler" to convert user search queries into a form MySQL likes
ClosedPublic

Authored by epriestley on Nov 24 2016, 3:32 PM.
Tags
None
Referenced Files
F13047899: D16938.id40776.diff
Thu, Apr 18, 2:44 PM
Unknown Object (File)
Fri, Apr 12, 4:12 PM
Unknown Object (File)
Thu, Apr 11, 3:46 AM
Unknown Object (File)
Thu, Apr 11, 3:46 AM
Unknown Object (File)
Thu, Apr 11, 3:46 AM
Unknown Object (File)
Thu, Apr 11, 3:46 AM
Unknown Object (File)
Thu, Apr 11, 3:46 AM
Unknown Object (File)
Mon, Apr 1, 12:04 PM
Subscribers
None

Details

Summary

Ref T11741. Ref T10642.

By default, MySQL runs query cat dog as "cat OR dog". Users expect "cat AND dog".

For users using MySQL, we instruct them to fix this by changing ft_boolean_syntax. However:

  • You can't edit this in RDS (T10642).
  • There's no similar option for InnoDB (T11741).

Instead, we can parse the query ourselves, and submit +"cat" +"dog" to MySQL, using whatever syntax it is configured for. This will also let us simplify setup since users won't need to change this setting any more.

This just implements a parser/compiler and test cases. It does not yet change search beahvior.

Test Plan

Added and executed unit tests. Ran some of the resulting queries in MySQL to make sure it did what I was expecting.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add a "query compiler" to convert user search queries into a form MySQL likes.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
  • Improve error message for trailing operators, by showing the operator.
  • Parse cat"dog".
  • Reject cat"dog.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 24 2016, 4:41 PM
This revision was automatically updated to reflect the committed changes.