Page MenuHomePhabricator

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

Authored by epriestley on Nov 24 2016, 3:32 PM.
Referenced Files
F11622420: D16938.diff
Mon, Mar 20, 8:45 AM
Unknown Object (File)
Mon, Mar 20, 12:53 AM
Unknown Object (File)
Thu, Mar 16, 6:42 PM
Unknown Object (File)
Thu, Mar 16, 6:41 PM
Unknown Object (File)
Mon, Feb 20, 12:51 PM
Unknown Object (File)
Mon, Feb 20, 12:35 PM
Unknown Object (File)
Feb 14 2023, 6:32 PM
Unknown Object (File)
Feb 7 2023, 12:58 PM



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

rPHU libphutil
Lint Passed
Tests Passed
Build Status
Buildable 14658
Build 19143: Run Core Tests
Build 19142: arc lint + arc unit

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.