Uploaded image for project: 'Atlas'
  1. Atlas
  2. ATLAS-1666

Improve on Overly Broad Throws

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 0.8.1, 1.0.0
    • atlas-core
    • None

    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.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            amestry Ashutosh Mestry
            amestry Ashutosh Mestry
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment