Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2578 LGTM, just one concern about `PythonCondaInterpreterTest.java`. It is using mock which may not found potential bugs. I think we can do unit without mock. But anyway this could be done in a following up ticket.
---