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

Knut Anders Hatlen commented on DERBY-672:
------------------------------------------

Hi Rick,

I saw you made this change in revision 1400083:

     /** Get the SQL name of the aggregate */
     public  String  getSQLName()
-        throws StandardException
     {
         if ( isUserDefinedAggregate() )
         {
-            return ((UserAggregateDefinition) 
uad).getAliasDescriptor().getQualifiedName();
+            try {
+                return ((UserAggregateDefinition) 
uad).getAliasDescriptor().getQualifiedName();
+            } catch (StandardException se)
+            {
+                return aggregateName;
+            }
         }
         else { return aggregateName; }
     }

Could you add a comment explaining what could cause exceptions here and why 
it's OK to ignore those exceptions?
Thanks.
                
> Re-enable user defined aggregates
> ---------------------------------
>
>                 Key: DERBY-672
>                 URL: https://issues.apache.org/jira/browse/DERBY-672
>             Project: Derby
>          Issue Type: Improvement
>          Components: SQL
>            Reporter: Rick Hillegas
>            Assignee: Rick Hillegas
>         Attachments: derby-672-01-aa-ddl.diff, 
> derby-672-02-ac-nonDistinct.diff, derby-672-03-aa-distinct.diff, 
> derby-672-03-ab-distinct.diff, derby-672-04-aa-fixJSR169test.diff, 
> derby-672-05-aa-java7testOrderProblem.diff, derby-672-06-aa-grantRevoke.diff, 
> derby-672-07-aa-fixJSR169again.diff, derby-672-08-aa-fixJSR169yetAgain.diff, 
> derby-672-09-ab-udtAggregates.diff, derby-672-10-af-typeBounds.diff, 
> derby-672-11-aa-tests.diff, derby-672-11-ab-tests.diff, 
> derby-672-12-aa-implicitCasts.diff, derby-672-13-aa-differentReturnType.diff, 
> derby-672-14-aa-collations.diff, derby-672-15-aa-setCollation.diff, 
> derby-672-16-aa-forbidInGroupBy.diff, derby-672-17-aa-moreTests.diff, 
> derby-672-18-aa-udaInJar.diff, UserDefinedAggregates.html, 
> UserDefinedAggregates.html, UserDefinedAggregates.html
>
>
> Nicolas Dufour in an email thread titled "functions and list" started on 
> November 2, 2005 requests the ability to create user defined aggregates.
> This functionality used to be in Cloudscape. It was disabled presumably 
> because it was considered non-standard. However, most of the machinery needed 
> for this feature is still in the code. We should re-enable user defined 
> aggregates after we agree on acceptable syntax.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to