GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/1381
[ZEPPELIN-1390] SparkInterpreter does not work for Spark2 version of HDP 2.5
### What is this PR for?
Spark2 version of HDP 2.5 is "2.0.0.2.5.0.0-1245". Currently, Zeppelin
p
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1381
I upgraded HDP to 2.5 from 2.4 today. sc.version printed 1.6.1 when HDP 2.4
and now HDP 2.5 spark.version prints 2.0.0.2.5.0.0-1245
---
If your project is set up for it, you can reply to this
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1381
@Leemoonsoo Thanks for comment!
I made some changes.
How does it look?
Thanks again.
---
If your project is set up for it, you can reply to this email and have your
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1381
[ZEPPELIN-1390] SparkInterpreter does not work for Spark2 version of HDP 2.5
### What is this PR for?
Spark2 version of HDP 2.5 is "2.0.0.2.5.0.0-1245". Currently, Zeppelin
p
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1381
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1381
@Leemoonsoo Yep, I did 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 not have this feature
enabled
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1381
[ZEPPELIN-1390] SparkInterpreter does not work for Spark2 version of HDP 2.5
### What is this PR for?
Spark2 version of HDP 2.5 is "2.0.0.2.5.0.0-1245". Currently, Zeppelin
p
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1381
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1381
@Leemoonsoo Hi, thanks for your guide.
I merged master branch, but it failed again.
And I tried to build again via reopening, but it fails again as you see.
However, it
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1381
Ok thanks
Let me know when it is done.
I will reopen PR :-)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1381
---
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
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1381
[ZEPPELIN-1390] SparkInterpreter does not work for Spark2 version of HDP 2.5
### What is this PR for?
Spark2 version of HDP 2.5 is "2.0.0.2.5.0.0-1245". Currently, Zeppelin
p
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1381
Thanks everyone!
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1525
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 if the
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1523
+1
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1522
ð
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1525
CI is green light!
Merge please @AhyoungRyu ! ð
---
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1540
Wow, this is a really cool feature for me. Thanks! ð
---
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
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/1570
[MINOR] High definition build status badge
### What is this PR for?
CI badge has low definition now. This change will use .svg image as badge
(instead of .png!)
before:
But, I don't know h
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@minahlee Awesome! I just changed status to `IN PROGRESS` now. Thanks ð
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1567
I also tested and was able to reproduce screenshot above.
When I was another user, it returned `403` status code with message:
```json
{"status":"FORB
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1567
@anthonycorbacho Nice! I agree with you :)
I will open the issue on JIRA. But since I don't know much about the code
structure, I may need some help ð
---
If your project is s
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1567
@anthonycorbacho Thanks :-) Don't worry! I do this because I want to do
ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1567
I guess it's same CI error with #1518.
There is a log which is:
```bash
gzip: stdin: unexpected end of file
tar: Unexpected EOF in archive
tar: Unexpected EOF in ar
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1578
ð
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@minahlee I have a question about REST API style. Is there any purpose that
you used `PUT` method?
In my opinion, `DELETE` method is more proper since it does not `PUT`
`clear` under a
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@minahlee Oh, in that context, it makes sense. LGTM.
But, after looking all around Notebook REST API, it works well now but it
doesn't seem RESTful. I summarized current list of RES
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1565
@anthonycorbacho @khalidhuseynov Thanks for the comment! I agree with you
guys.
This is a breaking change, so yeah we must be careful. So, if we proceed
this, new API URL's prefix
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1606
So cl! ð
---
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
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/1609
[ZEPPELIN-1628] Enable renaming note from the main page
### What is this PR for?
Now users can rename a note from the main page! This new feature will
improve UX.
I divided
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1609
@AhyoungRyu Thanks for the quick review! You are right! I will fix it.
Please wait ð
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1609
@AhyoungRyu I fixed it! It took some time to improve UX ð
Now when an input text is not valid, users can't click the button and the
modal shows error me
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1339
Tested and worked as expected ð
Fantastic work! But, there are other benefits caused by this change.
Before this change, users couldn't use `SPARK_SUBMIT_OPTIONS` env var
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1609
CI is green!
---
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1609
@AhyoungRyu Thanks for the review! :)
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1615
@Leemoonsoo Nice! When I was at Zeppelin meetup, every beginner tried to
build Zeppelin. Because they didn't know about existence of Zeppelin binary :-)
So I suggest adding a link to
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1627
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 if
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1634
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 if
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
Fixed error and CI is green!
Ready for review! ð
---
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1639
Nice move! ð I've always wanted to use ES6.
I tested and everything worked as expected.
But it seems like you turned off opening a browser when livereload serve
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1639
@1ambda +1 for webpack
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1639
@felizbear That makes sense! Then it's ok with me :)
I tested three commands (`serve` `test` `build`) and also worked with me.
I had removed `bower_components` and `node_mo
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1639
@felizbear Of course! But I do not completely understand about Zeppelin's
Travis CI, so we gonna need some help from committers ð
Oh, it just failed again with the same error. Bu
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1645
@Leemoonsoo Awesome! I will test it after updating the doc.
By the way, can I list installed packages of conda?
---
If your project is set up for it, you can reply to this email and have
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1649
@1ambda Nice catch!
Does that happen in IE? Because my Zeppelin didn't happen that way :)
I use Macbook Pro and tested on Chrome, Safari, and Firefox.
---
If your proje
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1649
@1ambda I just built Zeppelin from master branch and took screenshots :)
### Chrome secret tab:
### Chrome:
54.0.2840.98 (64-bit)
Maybe because of
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Thanks for review ð Don't need to hurry! I will check and
ping you :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu ping ping~ I tested!
But it just
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1339
@AhyoungRyu Thanks for taking care of my feedback ð
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Thanks for your kind review. I appreciate it.
Actually, I didn't test with Shiro ^^; and didn't know what `op` parameter
means. So thanks very much.
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Thanks! I also think `notebook` is a little bit weird :) I
merged your PR thanks.
This change will affect other insufficient privileges error messages as
well
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu I fixed what you mentioned ð Please review.
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
I guess CI fails because it only recompiles changed files. (since build
time is too short)
Could you please share your building command line and error log?
I don't know why, but I guess MathJax library wasn't
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1606
@giaosudau Thanks for the prompt response!
So you mean it doesn't happen without ZEPPELIN_SERVER_CONTEXT_PATH?
I've never used that option so I might not be helpful ^^;
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1606
@giaosudau You clarified the issue! Would you like to create a
[JIRA](https://issues.apache.org/jira/browse/ZEPPELIN) ticket for that? :-)
Then other developers will see the issue. Since this PR
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1639
You guys rock! ð
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1671
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 if
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Yeah!!! Finally!!!
CI is green!!! It was my fault! I didn't import `Folder` at
`zeppelin-server`.
(Why was it green before...?)
Please take a look!
-
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Fixed what you mentioned :) Always thanks for the review ð
---
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
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/1676
[ZEPPELIN-1705] Exclude unnecessary source file when check style on scio
### What is this PR for?
When building Zeppelin with `mvn clean package install -DskipTests
-DskipRat`, it fails on
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1676
@AhyoungRyu Thanks for the review again! ð
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1676
@prabhjyotsingh Thanks! ð
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Sorry for my late response! But it's due to tests :)
I reproduced the error you mentioned and you were right!
Furthermore, chasing a reason of the error, I notice
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
CI failure is because of me. I will fix 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 not have this
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1606
@giaosudau Thanks! ð I appreciate 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 not have this
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Yeah! CI is green ð Please review! Thanks
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1676
@Nabajyoti001 Thanks for sharing your issue! But it seems irrelevant with
this PR. However, of course, that issue should be resolved!
Since I could build current master branch
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/1706
[HOTFIX] Exclude org/apache/zeppelin/scio/avro/* once at pom.xml
### What is this PR for?
Previously on #1676, I excluded path **twice**. It's my mistake!
@khalidhuseynov pointed tha
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1706
@zjffdu @khalidhuseynov Thanks! ð
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1706
@astroshim @Leemoonsoo Thanks for review!
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@soralee Thanks for the review! ð That's a great idea! I will give some
effect on the front-end and ping you again. Thanks :)
---
If your project is set up for it, you can reply to this
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@soralee Hi! I added the feature you suggested. What do you think? Is title
or content of warning message appropriate? Thanks for the cool suggestion!
![rename_folder_merging_warning
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Always thanks for the review ð
---
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
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1630
[ZEPPELIN-1629] Enable renaming folder from the main page
### What is this PR for?
This PR will enable renaming folder from the main page.
I wrote some codes. Hope this PR is the
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1630
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu @soralee Yeah! CI is green!
---
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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1715
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 if
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
Thanks @Leemoonsoo! I appreciate your review ð
---
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1692
@cloverhearts Nice! Hope the new version will be released 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
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
I've tested more and it was fine most times. But occasionally it doesn't
work well. I think it should be tested more! I don't know why for now but I
will figure it out and ping
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
I found out bugs by adding loggers and fixed them!
CI is green and it seems to work perfectly. I think it's good to go now
ð
Please when you have some time. Thanks!
-
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1723
I tested on Firefox and the error is gone. LGTM ð
And I guess
1 - 100 of 206 matches
Mail list logo
