jena-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Damian Steer (JIRA)" <j...@apache.org>
Subject [jira] Issue Comment Edited: (JENA-22) OpAsQuery doesn't support OpGroup
Date Fri, 31 Dec 2010 10:53:45 GMT

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

Damian Steer edited comment on JENA-22 at 12/31/10 5:51 AM:
------------------------------------------------------------

This patch implements support for OpGroup in OpAsQuery. It's a little convoluted for two reasons:

 1. Projecting expressions has to be handled right at the end. Project is typically visited
before grouping, and setting project twice for a variable causes an error.
 2. Simple counts involve an internal variable assignment. I've tried to catch this, but it's
not elegant or general.

Tests are included.

      was (Author: shellac):
    This patch implements support for OpGroup in OpAsQuery. It's a little convoluted for two
reasons:

 1. Projecting expressions has to be handled right at the end. Project is typical visited
before grouping, and setting project twice for a variable causes an error.
 2. Simple counts involve an internal variable assignment. I've tried to catch this, but it's
not elegant or general.

Tests are included.
  
> OpAsQuery doesn't support OpGroup
> ---------------------------------
>
>                 Key: JENA-22
>                 URL: https://issues.apache.org/jira/browse/JENA-22
>             Project: Jena
>          Issue Type: Bug
>          Components: ARQ
>            Reporter: Damian Steer
>            Assignee: Andy Seaborne
>         Attachments: OpAsQuery_group.patch
>
>
> OpAsQuery can't create some fairly straightforward queries, for example select (count(*)
as ?cs) { ?s ?p ?o }.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message