Håvard M. Ottestad

> It all looks technically correct to me, but I'm puzzled by why you'd need blank node preservation here in the first place. > > It's normally something that we...

### TODO - [ ] Add test to make sure the client still works with the old server

I think that this should be split into two branches. One with a simple fix that just changes out the format to binary, and a second that changes things up...

Let's just hope that Elastic is done inventing new clients for a while. This will be the third client, after the [Java High Level REST Client](https://www.elastic.co/guide/en/elasticsearch/client/java-rest/7.15/java-rest-high.html) and the [Transport Client](https://www.elastic.co/guide/en/elasticsearch/client/java-api/current/transport-client.html).

I'm not aware that we've copied any code verbatim. We do depend on some Jena stuff for some of the SHACL tests, in order to use the reference implementation. What...

I meant to reply to this. I was wondering if we could create a maven module with classes that extend all the ones you want to include, then use java...

Is it correct that the query you are having performance issues with is using a VALUES clause, but if you remove the VALUES clause then there is no performance difference?

Probably related to: - https://github.com/eclipse/rdf4j/issues/1405 - https://github.com/eclipse/rdf4j/issues/1642

There is a trig file in the repo that @daltontc linked too. It reproduces the issue for me. I think it's an issue with the join order from the query...

### Without VALUES clause ``` Projection (resultSizeActual=505) ╠══ProjectionElemList ║ ProjectionElem "parent" ║ ProjectionElem "child" ╚══LeftJoin (LeftJoinIterator) (resultSizeActual=505) ├──StatementPattern (resultSizeEstimate=504, resultSizeActual=504) │ Var (name=parent) │ Var (name=_const_f5e5585a_uri, value=http://www.w3.org/1999/02/22-rdf-syntax-ns#type, anonymous) │ Var...