thomasmueller opened a new pull request, #2145:
URL: https://github.com/apache/jackrabbit-oak/pull/2145
This is longer than expected. The problem is that there was no conversion
function from "JCR property name" to "Elasticsearch field name" so far, so I
had to add it everywhere. That also affects tests etc.
Side node: ElasticIndexUtilsTest: I also added unit tests of the existing
functions. I find it really weird that we convert randomly distributed bytes to
a string like so:
new String(<byte array with completely random values>)
... and then use that as the field name for Elasticsearch. But... (a)
changing it would introduce incompatibility, and (b) it seems to work.
--
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]
