Github user aspen01 commented on the issue:
https://github.com/apache/zeppelin/pull/1600
@EronWright Thank you for your feedback. When I first developed the
feature, I assumed that HDFS would not be used as default storage because
remote FS failure could affect zeppelin usage. So ext
Github user EronWright commented on the issue:
https://github.com/apache/zeppelin/pull/1600
Seems overcomplicated for `HDFSNotebookRepo` to extend `VFSNotebookRepo`,
and to be synchronizing the local FS with the remote FS. Just implement
`NotebookRepo` directly.
---
If your projec
Github user keithchambers commented on the issue:
https://github.com/apache/zeppelin/pull/1600
Is this ready to be merged?
---
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
Github user placeybordeaux commented on the issue:
https://github.com/apache/zeppelin/pull/1600
I built with ```mvn clean package```, but got the same results with ```mvn
clean package -Pspark-1.6 -Phadoop-2.6 -Pyarn -Ppyspark -Psparkr
-Pscala-2.10```. It's likely this is just a probl
Github user aspen01 commented on the issue:
https://github.com/apache/zeppelin/pull/1600
@placeybordeaux How did you build a package?
```
$ mvn clean package -Pspark-1.6 -Phadoop-2.6 -Pyarn -Ppyspark -Psparkr
-Pscala-2.10
...
Results :
Tests
Github user placeybordeaux commented on the issue:
https://github.com/apache/zeppelin/pull/1600
Not sure why the CI hasn't kicked in, but I tried compiling this on my
local and I am getting some test failures:
```
Results :
Failed tests:
NotebookTest.testAbo
Github user aspen01 commented on the issue:
https://github.com/apache/zeppelin/pull/1600
I committed this branch on latest master and force-push it.
---
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 no
Github user aspen01 commented on the issue:
https://github.com/apache/zeppelin/pull/1600
@placeybordeaux Thank you.
@bzz Do you mean I have to to merge these commits to master not branch-0.6?
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1600
Thank you @aspen01, @placeybordeaux
In order to be merged, this branch have to be rebased on top of the master,
which itself already includes CI fixes, i.e 1c7d8fb.
This branch must
Github user placeybordeaux commented on the issue:
https://github.com/apache/zeppelin/pull/1600
Looks like you're missing a hotfix commit:
https://github.com/apache/zeppelin/commit/1c7d8fb0f7f8dd3cb6ce4837053c11c6453b5f18
---
If your project is set up for it, you can reply
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1600
I see! This looks like something failing on the TravisCI side though.
Could you please rebase this branch on latest master and force-push it here
again? This will trigger CI \w latest code\fixes f
Github user aspen01 commented on the issue:
https://github.com/apache/zeppelin/pull/1600
@bzz Thank you for advice. But the build is failed even though I fixed
license and checkstyle.
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1600
@aspen01 I wonder if any progress on this one?
---
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
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1600
@aspen01 thank you for contribution!
Could you please double-check that all new files, as any other file in
Apache Zeppelin, have [ASF's Apache 2.0 license
header](http://www.apache.org/legal
14 matches
Mail list logo
