Github user funes79 commented on the issue:
https://github.com/apache/zeppelin/pull/2016
-> Tried to compile
mvn -X clean package -pl 'zeppelin-web' -DskipTests;
-> But failed. Using Centos 7 and the master branch.
[ERROR] error
Github user lwmonster commented on the issue:
https://github.com/apache/zeppelin/pull/2016
Still have the same problem on master at 2017-09-20 ...
---
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2016
LGTM.
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2016
CI failure is not related (fixed by #2033).
LGTM and merge to master and branch-0.7 if no further discussions
---
If your project is set up for it, you can reply to this email and have
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2016
@prabhjyotsingh
- tested and it works well.
- I'v changed implementation of this PR according to your suggestion.
---
If your project is set up for it, you can reply to this email
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2016
@prabhjyotsingh Sure. I will test and give you feedback soon.
---
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
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2016
@1ambda I Just tried this patch, I and it worked for me, do you want to
try it out ?
```
---
zeppelin-web/pom.xml | 3 +++
1 file changed, 3 insertions(+)
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2016
Hope to merge into 0.7.0 branch as well to help 0.7.1+ users
---
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2016
@felixcheung
Yes, build time increase 40 secs ~ 1 min approximately per build profile.
---
If your project is set up for it, you can reply to this email and have your
reply appear on