[
https://issues.apache.org/jira/browse/CASSANDRA-10365?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14984357#comment-14984357
]
Robert Stupp edited comment on CASSANDRA-10365 at 11/1/15 11:58 AM:
--------------------------------------------------------------------
Apologies for the compilation failure. Fixed it and force-pushed an update.
WRT to the other issues around {{CREATE AGGREGATE}} (e.g. failing tests,
initcond parse errors), we definitely need CASSANDRA-10617 to get it working.
EDIT: --Regarding the NPE with the non-existing keyspace: that's a bug for
{{CREATE AGGREGATE}} and at least {{CREATE TABLE}}. Will open a separate ticket
for this.--
[~iamaleksey], the NPE is caused by the change in {{CQL3Type}}
({{CQL3Type.prepare(String)}} now always executes
{{Schema.instance.getKSMetaData(keyspace).types}} which will NPE for a
non-existing KS). Usually we check for non-existing keyspaces in
{{CQLStatement.validate}} - but that's called after the statement's been
{{prepareKeyspace}}/{{prepare}} methods have been called.
was (Author: snazy):
Apologies for the compilation failure. Fixed it and force-pushed an update.
WRT to the other issues around {{CREATE AGGREGATE}} (e.g. failing tests), we
definitely need CASSANDRA-10617 to get it working.
Regarding the NPE with the non-existing keyspace: that's a bug for {{CREATE
AGGREGATE}} and at least {{CREATE TABLE}}. Will open a separate ticket for this.
> Store types by their CQL names in schema tables instead of fully-qualified
> internal class names
> -----------------------------------------------------------------------------------------------
>
> Key: CASSANDRA-10365
> URL: https://issues.apache.org/jira/browse/CASSANDRA-10365
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Aleksey Yeschenko
> Assignee: Aleksey Yeschenko
> Labels: client-impacting
> Fix For: 3.0.0
>
>
> Consider saving CQL types names for column, UDF/UDA arguments and return
> types, and UDT components.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
