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

Claude Warren commented on JENA-945:
------------------------------------

After further review and some head scratching I managed to reduce the change to 
adding a setGroupBy(VarExprList) method.

If the method is passed a null it resets the groupBy to an empty list (initial 
condition upon construction of the Query).  So it can not be set to null by 
this method.

It was either add that or add methods to VarExprList that would allow 
manipulation of the items in that list.  That seemed to be more work and more 
prone to error.

> 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
>
> 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