Do you mind if I clear up to remove the duplication of functionality in Query?

        Andy

On 21/05/15 07:18, Claude Warren wrote:
They are not both needed.  It was a case of simultaneous development.  You
added the the two methods while I added the one. I ran a  complete build
and test on my (s...ll...o...w...) system and when it worked committed.
You and made your changes and committed at the same time.

On Wed, May 20, 2015 at 11:11 PM, Andy Seaborne (JIRA) <j...@apache.org>
wrote:


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

Andy Seaborne commented on JENA-945:
------------------------------------

Replacing the call of {{Query.setGroupBy}} in {{SolutionModifierHandler}},
that is.

Add Seven methods to Query class
--------------------------------

                 Key: JENA-945
                 URL: https://issues.apache.org/jira/browse/JENA-945
             Project: Apache Jena
          Issue Type: Improvement
          Components: Core
    Affects Versions: Jena 3.0.0
            Reporter: Claude Warren
            Assignee: Claude Warren
            Priority: Minor
             Fix For: Jena 3.0.0


The QueryBuilder uses reflection to perform some of its data
manipulation.  This reflection will fail in an environment with Java
Security Manager enabled.
This change is to add methods so that QueryBuilder no longer needs to
use reflection to get and set Query internal variables.
The variables are:
namedGraphURIs  -- get and set required. parameter/return: List<String>
graphURIs -- get and set required. parameter/return: List<String>
projectVars -- get and set required. parameter/return: VarExprList
groupVars -- set required. parameter: VarExprList



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)





Reply via email to