thomasmueller opened a new pull request, #1187:
URL: https://github.com/apache/jackrabbit-oak/pull/1187

   I had a to change a lot of tests, because they depended (too much) on 
implementation details.
   
   Even thought this would be a good idea, I did not switch to Json output so 
far, because:
   * some upstream UI tools depend on the query plans to have a certain form, 
which would break,
   * it would make the patch even larger,
   * the Lucene query string might contain double quotes, which would then be 
escaped, and so the output hard to read
   
   So basically, first the upstream UI tools would 
   
   The Elasticsearch query is Json, which wouldn't need to be escaped if we add 
it as a child object.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to