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