activemq-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ARTEMIS-1327) Support checked exceptions from ActiveMQServerPlugin
Date Mon, 07 Aug 2017 18:57:00 GMT

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

ASF GitHub Bot commented on ARTEMIS-1327:
-----------------------------------------

Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1445
  
    What about expose activemqexception.  But swallow others?
    
    
    Let me amend my PR and will and you feedback. 


> Support checked exceptions from ActiveMQServerPlugin
> ----------------------------------------------------
>
>                 Key: ARTEMIS-1327
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1327
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>          Components: Broker
>            Reporter: Christopher L. Shannon
>            Assignee: Christopher L. Shannon
>             Fix For: 2.3.0
>
>
> After I was writing a couple custom plugins I realized it would be beneficial to support
checked exceptions.  This makes error handling simpler for plugin writers as they can throw
various exceptions and not have to always wrap them in a RuntimeException.  Almost every place
in the broker where plugin methods are currently called already support handling checked Exceptions
so this is pretty simple and mostly we just need to add a "throws Exception" to each of the
methods in the ActiveMQServerPlugin interface and make sure the methods used to execute the
plugin methods also support it.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message