Github user felixcheung commented on the issue:
https://github.com/apache/zeppelin/pull/2916
sure, I'd defer to @zjffdu
could you fix up the truncated PR title btw
---
Github user iijima-satoshi commented on the issue:
https://github.com/apache/zeppelin/pull/2916
@felixcheung Please let me know if there are any other comments. Thanks.
---
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2916
Thanks @iijima-satoshi LGTM
---
Github user iijima-satoshi commented on the issue:
https://github.com/apache/zeppelin/pull/2916
Tests for interpreters except spark one were droped in case of
ZEPPELIN_IMPERSONATE_SPARK_PROXY_USER = true.
I fixed and manually tested.
---
Github user iijima-satoshi commented on the issue:
https://github.com/apache/zeppelin/pull/2916
Could someone review this?
Is it ok that this fix has no tests? This fix only includes interpreter.sh
script. I manually tested it.
---