Github user hriviere commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/869#issuecomment-216066071
Thanks Leemonsoo and Felix for the review !
To sum up changes since the initial commit :
1/ I add the link to python.md in _navigation.html
2/ Dynamic form use the same syntax than pyspark. It uses py4j, if py4j is
not installed, displays a warning message to user to indicate he must install
py4j to use this feature
3/ Catch exceptions are now logged in logger.error or logger.warn
4/ Python code follows pep8 indent
5/ python.path is replaced by python and default value : /usr/bin/python by
python (assume python is in $PATH). See comments above for details
**No change yet about: **
1/ Set a Matplotlib backend by default (see my comment above)
2/ Method to interupt execution : currently the JVM send a SIGINT (see my
comment above)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
