From d60226f0c6f4b200ca76605bfd0269ffbf9afe24 Mon Sep 17 00:00:00 2001 From: Mark Hahnenberg Date: Sat, 14 Jan 2017 16:47:56 -0800 Subject: [PATCH] [search-index] Don't removeQuotedHTML when indexing threads Summary: This is a very expensive operation and there seemed to be few downsides to including quoted portions in the index. We can revisit this to improve how we index when we have a proper scheduler. Test Plan: Run locally Reviewers: juan, evan Reviewed By: evan Differential Revision: https://phab.nylas.com/D3682 --- .../search-index/lib/thread-search-index-store.es6 | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/internal_packages/search-index/lib/thread-search-index-store.es6 b/internal_packages/search-index/lib/thread-search-index-store.es6 index 4eadeee27..e9907594f 100644 --- a/internal_packages/search-index/lib/thread-search-index-store.es6 +++ b/internal_packages/search-index/lib/thread-search-index-store.es6 @@ -205,11 +205,7 @@ class ThreadSearchIndexStore { thread.messages() .then((messages) => ( messages - .map(({body, snippet}) => ( - !_.isString(body) ? - {snippet} : - {body: QuotedHTMLTransformer.removeQuotedHTML(body)} - )) + .map(({body, snippet}) => (!_.isString(body) ? {snippet} : {body})) .map(({body, snippet}) => ( snippet || Utils.extractTextFromHtml(body, {maxLength: MESSAGE_BODY_LENGTH}).replace(/(\s)+/g, ' ') ))