Homec4science

Reduce Maniphest fulltext limit from PHP_INT_MAX to 10k

Authored by epriestley <git@epriestley.com> on Sep 24 2013, 19:50.

Description

Reduce Maniphest fulltext limit from PHP_INT_MAX to 10k

Summary:
PHP_INT_MAX is rejected by ElasticSearch since it's outside of the representable integer range (see: https://gist.github.com/JustinTulloss/c4ac0e1c93d6d1e91744).

Just use 10K, as matching more than 10K results probably isn't useful to anyone.

Test Plan: Confirmed this fixes the issue in IRC. Ran a fulltext search.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D7097

Details

Committed
epriestley <git@epriestley.com>Sep 24 2013, 19:50
Pushed
aubortJan 31 2017, 17:16
Parents
rPH6b245c0665f4: Further smooth "Add CC" interaction in Maniphest
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH79ec5222a579: Reduce Maniphest fulltext limit from PHP_INT_MAX to 10k (authored by epriestley <git@epriestley.com>).Sep 24 2013, 19:50