[LIVY-995][REPL] JsonParseException is thrown when closing Livy session when using python profile

[jianzhenwu]

What changes were proposed in this pull request?

When Livy closes PythonInterpreter, the output may not be in JSON format, so there is no need to deserialize json.
https://issues.apache.org/jira/browse/LIVY-995

How was this patch tested?

manual tests
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Please review https://livy.incubator.apache.org/community/ before opening a pull request.

[codecov-commenter]

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (86fc823) 65.45% compared to head (4336eab) 28.56%.

Files	Patch %	Lines
...scala/org/apache/livy/repl/PythonInterpreter.scala	0.00%	9 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##             master     #439       +/-   ##
=============================================
- Coverage     65.45%   28.56%   -36.89%     
+ Complexity      954      380      -574     
=============================================
  Files           103      103               
  Lines          6084     6091        +7     
  Branches        922      923        +1     
=============================================
- Hits           3982     1740     -2242     
- Misses         1546     3994     +2448     
+ Partials        556      357      -199     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jianzhenwu]

hi @jerryshao I noticed you contributed to Livy's REPL module. Can you help review this PR?

[lmccay]

@jianzhenwu - can you add a unit test to show that the issue is resolved and ensure that it doesn't regress in the future?
It would also be good to point to coverage or add new tests to ensure that previous functional expectations remain intact.

[jianzhenwu]

@lmccay thank you for your reply. I added unit tests to ensure no impact on original functionality and to verify expected functionality.

[lmccay]

LGTM
+1

[lmccay]

@jianzhenwu - thanks for your contribution! I have merged this and it will be available in the next release.

[jianzhenwu]

thank you