Preparing the ES Embeddings Rewriter

[mkr]

• Fixing parsing embeddings containing numbers without decimals (mostly for tests)
• Using the Lucene query for boosting directly to avoid re-parsing in ES

[renekrie]

Please see comment inline

[renekrie]

I'm not understanding why this has been changed. What is the benefit of not using JsonUtil.readJson()?

[mkr]

The code before was
(List<Double>) JsonUtil.readJson(con.getInputStream(), Map.class).get("embedding")

What happens here is that the Jackson Objectmapper will just read the object as a map and the embedding field as a List. It does not try to coerce into a common List element type, i.e. you get a List of (Double, Integer, Double) if you parse a JSON array [1.2, 1, 1.3]. You then get a class java.lang.Integer cannot be cast to class java.lang.Double when the element of the List is accessed as a Double.