Uploaded image for project: 'Apache Storm'
  1. Apache Storm
  2. STORM-2216

Favor JSONValue.parseWithException

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

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0, 1.1.0, 1.0.3
    • None
    • None

    Description

      In a number of places we will parse JSON with `JSONValue.parse` but if it cannot parse the data it just returns null, which leads to NPEs that are not descriptive and hard to debug. `JSONValue.parseWithException` throws an exception that explains what happened and makes it much simpler to debug.

      Attachments

        Activity

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

          People

            revans2 Robert Joseph Evans
            revans2 Robert Joseph Evans
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 0.5h
                0.5h

                Slack

                  Issue deployment