atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ashutosh Mestry (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (ATLAS-1666) Improve on Overly Broad Throws
Date Fri, 17 Mar 2017 21:21:41 GMT

     [ https://issues.apache.org/jira/browse/ATLAS-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Ashutosh Mestry updated ATLAS-1666:
-----------------------------------
    Description: 
Methods in the following classes throws generic exception. This makes it hard for callers
to effectively handle errors and recover. Although, this does not impact functionality. It
would be valuable to improve on this.

Below is a files that have this pattern: 
1. *ActiveInstanceState.java*
2. *AbstractParam.java*
3. *QuickStart.java*
4. *SetupSteps.java*
5. *FalconHook.java*
6. *AtlasLdapAuthenticationProvider.java*
7. *FalconEventPublisher.java*
8. *AtlasADAuthenticationProvider.java*
9. *StormHook.java*
10. *FalconBridge.java*
11. *EmbeddedServer.java*
12. *StormTopologyUtil.java*
13. *SqoopHook.java*
14. *HiveHook.java*
15. *HiveMetaStoreBridge.java*

General approach to refactoring would be define an error code in *AtlasErrorCode* and throw
*AtlasBaseException*.


> Improve on Overly Broad Throws
> ------------------------------
>
>                 Key: ATLAS-1666
>                 URL: https://issues.apache.org/jira/browse/ATLAS-1666
>             Project: Atlas
>          Issue Type: Improvement
>          Components:  atlas-core
>            Reporter: Ashutosh Mestry
>            Assignee: Ashutosh Mestry
>             Fix For: trunk
>
>
> Methods in the following classes throws generic exception. This makes it hard for callers
to effectively handle errors and recover. Although, this does not impact functionality. It
would be valuable to improve on this.
> Below is a files that have this pattern: 
> 1. *ActiveInstanceState.java*
> 2. *AbstractParam.java*
> 3. *QuickStart.java*
> 4. *SetupSteps.java*
> 5. *FalconHook.java*
> 6. *AtlasLdapAuthenticationProvider.java*
> 7. *FalconEventPublisher.java*
> 8. *AtlasADAuthenticationProvider.java*
> 9. *StormHook.java*
> 10. *FalconBridge.java*
> 11. *EmbeddedServer.java*
> 12. *StormTopologyUtil.java*
> 13. *SqoopHook.java*
> 14. *HiveHook.java*
> 15. *HiveMetaStoreBridge.java*
> General approach to refactoring would be define an error code in *AtlasErrorCode* and
throw *AtlasBaseException*.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message