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
Unknown Object (File)
Sat, Dec 14, 4:15 PM
Unknown Object (File)
Sat, Dec 14, 9:07 AM
Unknown Object (File)
Sat, Dec 14, 2:09 AM
Unknown Object (File)
Thu, Dec 12, 11:30 AM
Unknown Object (File)
Fri, Dec 6, 10:03 AM
Unknown Object (File)
Thu, Dec 5, 2:10 AM
Unknown Object (File)
Sun, Dec 1, 6:36 AM
Unknown Object (File)
Thu, Nov 28, 2:39 AM
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
Branch
search1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14659
Build 19145: Run Core Tests
Build 19144: 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.