Chengxu Bian
Chengxu Bian
Hello @hmottestad, I found the root cause. In `ConstantOptimizer`, it precompiles and evaluates the constant value in the value expression. https://github.com/eclipse-rdf4j/rdf4j/blob/b33d91485502d2f5266916c0581960e41b8f28b5/core/queryalgebra/evaluation/src/main/java/org/eclipse/rdf4j/query/algebra/evaluation/optimizer/ConstantOptimizer.java#L223-L229 The example query meets the condition if `isConstant` because...
BTW, to easily reproduce it, you may the below test case to https://github.com/eclipse-rdf4j/rdf4j/blob/b33d91485502d2f5266916c0581960e41b8f28b5/testsuites/lucene/src/main/java/org/eclipse/testsuite/rdf4j/sail/lucene/AbstractLuceneSailTest.java#L71 ```java @Test public void testMaxFunction(){ StringBuffer buffer = new StringBuffer(); buffer.append("PREFIX search: \n"); buffer.append("SELECT ?subj (MAX(?score) as...
Maybe we can fix it by checking whether the parent node is a `GroupElem` because we cannot replace the child of `GroupElem` with a `ValueConstant`. The `GroupElem` requires children nodes...
@JervenBolleman Can we add an extra field in the `GroupElem` to allow it hold a `ValueConstant`? ```java public class GroupElem extends AbstractQueryModelNode { private AggregateOperator operator; private ValueConstant valueConstant; @Override...
@JervenBolleman My only concern about this approach is that `optimize` becomes `tryOptimize`. We allow the `ConsantOptimizer` to fail and do nothing. If you believe it's ok for optimizers, we can...
Could someone please assign this to me? We're currently blocked by this bug, even after trying the temporary fix.
Hi @JervenBolleman , Sorry, I may be misunderstanding where things currently stand, so I wanted to clarify: Do you already have a PR in flight? Or were you waiting on...
Thank you @JervenBolleman , I’ll need a few days to go through
Hello RDF4J maintainers, Just a quick heads-up: our team plans to contribute RDF 1.2 support to RDF4J this summer. Our intern will take the lead, with @nguyenm100 and me as...
> Does this in any way allow a remote repo (HTTPRepository) to use the existing query explanation method on a query object and retrieve the explanation from the remote server?...