Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-157203411
The current push should resolve some issues with changes in the
Spark-Zeppelin interface that had created issues for users, as well as support
for 1.5.1.
Knitr support is improved, and the reason for a separate knitr interpreter
may be clearer now.
The amount of code borrowed from rscala is reduced.
I did not address issues with @author tags, or files under the R/ folder.
The reason is, to be blunt, I don't understand what the precise concerns
actually are.
Please note that I changed .travis.yml to only use spark 1.4 and later.
I'm sure there is a better way to do it, but I'm just not in a position to try
to figure out the entire Zeppelin build process.
Integrating this with the rest of zeppelin is going to take some work
regarding pom's, travis, and so forth. I can do a lot of that, but I'm going
to need to discuss it with the people who have been "owning" those files.
There are just too many moving pieces here.
Because of the size of this (which is, unfortunately, necessary), posting
here is probably not the most efficient way. That is also true because certain
people chose to use this PR as a forum to air other issues. Therefore, it
would be better if reviewers e-mail me directly.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
