[jira] [Comment Edited] (JENA-628) TransformFilterPlacement does not always push down through assign/extend when it could

["Andy Seaborne (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/JENA-628?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13884050#comment-13884050 ] 

Andy Seaborne edited comment on JENA-628 at 1/28/14 2:42 PM:
-------------------------------------------------------------

Commit 1562046 from [~andy.seaborne] in branch 'jena/trunk'
[ https://svn.apache.org/r1562046 ]

JENA-628 Expose BGP/filter placement functional in public functions for use in TDB.


was (Author: jira-bot):
Commit 1562046 from [~andy.seaborne] in branch 'jena/trunk'
[ https://svn.apache.org/r1562046 ]

JENA-628 Expose BG/filter placement functional in public functions for use in TDB.

> 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
>             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.1.5#6160)