cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Stupp (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (CASSANDRA-7812) DROP FUNCTION drops all function with a given name
Date Mon, 01 Sep 2014 13:26:21 GMT

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

Robert Stupp edited comment on CASSANDRA-7812 at 9/1/14 1:25 PM:
-----------------------------------------------------------------

Note: Patch rejects ambigous {{DROP FUNCTION}} with IRE.
Subject to discuss: whether to force arg name or not.


was (Author: snazy):
Note: Patch rejects ambigous {{DROP FUNCTION}} with IRE.

> DROP FUNCTION drops all function with a given name
> --------------------------------------------------
>
>                 Key: CASSANDRA-7812
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-7812
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Sylvain Lebresne
>            Assignee: Robert Stupp
>             Fix For: 3.0
>
>         Attachments: 7812.txt
>
>
> We support multiple overloads of a given function name, so that you can define:
> {noformat}
> CREATE FUNCTION div(int a, int b) RETURNS int 'myDivOnIntegerFct';
> CREATE FUNCTION div(double a, double b) RETURNS double 'myDivOnDoubleFct';
> {noformat}
> In other words, a function is uniquely identified by it's name *and* it's argument types.
> But the {{DROP FUNCTION}} syntax currently only take a function name and if there is
multiple overload for that name, drop all of them without any way to drop one without the
other.
> So we should probably allow to optionally pass the type arguments to DROP to select a
specific overloading, something like:
> {noformat}
> DROP FUNCTION div(double, double);
> {noformat}
> Also, and while that's probably debatable, I'd also suggest to change the behavior of:
> {noformat}
> DROP FUNCTION div;
> {noformat}
> to complain if the there is multiple overload of the {{div}} method instead of dropping
all overload as it feels somewhat foot-shooting otherwise.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message