Page MenuHomePhabricator

insource: queries can't search for the return character
Open, LowPublic

Description

Perhaps a minor thing, but lucene regex doesn't appear to support \n natively. Instead cirrus needs to parse \n into a literal return charater which will then be re-encoded to \n at the json level when making requests to elasticsearch. Currently we pass the \n through all the way to lucene and don't find anything. Should be an easy fix, but we might want to consider if there are other special characters beyond \n (\r, \t? others? i dunno...) that we want to handle

Event Timeline

Restricted Application added subscribers: Zppix, Aklapper. · View Herald Transcript
Aqwww23 renamed this task from insource: queries can't search for the return character to rikvip.May 13 2016, 9:34 PM
Aqwww23 triaged this task as High priority.
Aqwww23 set the point value for this task to 12.
Aqwww23 removed subscribers: Aklapper, Zppix, EBernhardson.

ss

Krenair renamed this task from rikvip to insource: queries can't search for the return character.May 13 2016, 9:53 PM
Krenair raised the priority of this task from High to Needs Triage.
Krenair removed the point value for this task.
Krenair edited subscribers, added: EBernhardson, Aklapper, Zppix; removed: Aqwww23.
debt triaged this task as Low priority.Oct 27 2016, 8:49 PM
debt moved this task from needs triage to This Quarter on the Discovery-Search board.