You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Sylvain Lebresne (JIRA)" <ji...@apache.org> on 2015/11/23 15:57:11 UTC
[jira] [Commented] (CASSANDRA-10747) CQL.textile syntax incorrectly
includes optional keyspace for aggregate SFUNC and FINALFUNC
[ https://issues.apache.org/jira/browse/CASSANDRA-10747?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15022205#comment-15022205 ]
Sylvain Lebresne commented on CASSANDRA-10747:
----------------------------------------------
Are we sure we don't want to update the syntax instead to allow the keyspace? It's certainly very rarely useful but I suppose this could be useful in the case of a name conflict between the current keyspace and the system one at least? Would be more consistent also in a way.
> CQL.textile syntax incorrectly includes optional keyspace for aggregate SFUNC and FINALFUNC
> -------------------------------------------------------------------------------------------
>
> Key: CASSANDRA-10747
> URL: https://issues.apache.org/jira/browse/CASSANDRA-10747
> Project: Cassandra
> Issue Type: Bug
> Components: Documentation and Website
> Reporter: dan jatnieks
> Assignee: Robert Stupp
> Priority: Trivial
> Fix For: 2.2.x
>
> Attachments: 10747.txt
>
>
> [CQL.textile|https://github.com/apache/cassandra/blob/trunk/doc/cql3/CQL.textile] incorrectly includes an optional keyspace for the {{SFUNC}} and {{FINALFUNC}} parts of a {{CREATE AGGREGATE}} statement.
> The grammar for 2.2 and 3.0 does not allow a keyspace to be specified here.
> From the CQL.textile:
> {noformat}
> <create-aggregate-stmt> ::= CREATE ( OR REPLACE )?
> AGGREGATE ( IF NOT EXISTS )?
> ( <keyspace> '.' )? <aggregate-name>
> '(' <arg-type> ( ',' <arg-type> )* ')'
> SFUNC ( <keyspace> '.' )? <state-functionname>
> STYPE <state-type>
> ( FINALFUNC ( <keyspace> '.' )? <final-functionname> )?
> ( INITCOND <init-cond> )?
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)