[
https://issues.apache.org/jira/browse/JENA-628?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13974904#comment-13974904
]
Rob Vesse commented on JENA-628:
--------------------------------
It's a single quad pattern with a complex filter that is affected by both the
change in behaviour in 2.11.1 (filter wasn't placed at all previously but is
with 2.11.1) and also gets horribly mangled with 2.11.1 if you turn on filter
placement and tell it to exclude BGPs (the filter disappears entirely)
So it is intended primarily as a regression test case
With the various fixes to filter placement in current trunk the filter gets
placed suitably with filter placement on and with BGPs excluded the query no
longer gets mangled
> TransformFilterPlacement does not always push down through assign/extend when
> it could
> --------------------------------------------------------------------------------------
>
> Key: JENA-628
> URL: https://issues.apache.org/jira/browse/JENA-628
> Project: Apache Jena
> Issue Type: Bug
> Components: ARQ
> Affects Versions: Jena 2.11.1
> Reporter: Rob Vesse
> Assignee: Rob Vesse
> Fix For: Jena 2.11.2
>
>
> I've noticed a bug in the improved TransformFilterPlacement where it doesn't
> always do the correct thing when pushing through assign/extend in cases where
> the placement into BGPs is disabled.
> Consider the following example algebra:
> {noformat}
> (filter (= ?x 123) (assign ((?z 123)) (bgp (?s ?p ?x)) ))
> {noformat}
> With optFilterPlacementBGP enabled this produces the following improved
> algebra:
> {noformat}
> (assign ((?z 123)) (filter (= ?x 123) (bgp (?s ?p ?x)) ))
> {noformat}
> However with optFilterPlacementBGP disabled this leaves the algebra as is
> even though it could safely push it through the bind without pushing it into
> the BGP itself.
> I think the fix for this is trivial and I will apply it shortly. I will also
> look and see if there are other cases where it fails to push down as far as
> it could.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
