zeppelin icon indicating copy to clipboard operation
zeppelin copied to clipboard

[ZEPPELIN-4124] Use optJSONArray to avoid exception

Open bd2019us opened this issue 6 years ago • 1 comments

What is this PR for?

Avoid unexpected JSONException if no key exist.

What type of PR is it?

[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring] BugFix

Todos

  • [ ] - Task

What is the Jira issue?

  • Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
  • Put link here, and add [ZEPPELIN-Jira number] in PR title, eg. [ZEPPELIN-533]

https://issues.apache.org/jira/browse/ZEPPELIN-4124

How should this be tested?

  • First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
  • Strongly recommended: add automated unit tests for any new or changed behavior
  • Outline any manual steps to test the PR here.

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update?
  • Is there breaking changes for older versions?
  • Does this needs documentation?

bd2019us avatar Apr 14 '19 01:04 bd2019us

@bd2019us Thanks for the contribution, could you add unit test and update the PR description ?

zjffdu avatar Apr 14 '19 06:04 zjffdu