Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/908#issuecomment-220874760
I tested this funtionality but I got failed.
[My test process]
1. I removed ```interpreter/md``` path
2. start zeppelin server
3
GitHub user astroshim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/910
move noteName method from main to navbar.
### What is this PR for?
Notebook list doesn't show when you reload(F5) the zeppelin web page not in
home.
### What ty
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/908#issuecomment-221145936
@cloverhearts I tested and really nice working. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-221151596
I tesed and working well. but it seems need a "initial statement".
---
If your project is set up for it, you can reply to this email and have
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-221156654
Thanks for the detail explanation AhyoungRyu.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user astroshim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/912
Hotfix - does not working notebook filter on navibar.
### What is this PR for?
It doesn't working the notebook filter on navibar.
### What type of PR is it?
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221449317
@minahlee Thank you for review and finding the problem. I'll take a look
this.
---
If your project is set up for it, you can reply to this emai
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/913#issuecomment-221480263
+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
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221508138
@minahlee I fixed the problem. Can you review?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221547615
@bzz Okay. I'll make 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 pr
Github user astroshim closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/912
---
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 astroshim reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/912
Hotfix - does not working notebook filter on navibar.
### What is this PR for?
It doesn't working the notebook filter on navibar.
### What type of PR is it?
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221553248
@bzz I couldn't find the reason of build failure so I just retrigger CI.
---
If your project is set up for it, you can reply to this email and have
GitHub user astroshim reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/912
Hotfix - does not working notebook filter on navibar.
### What is this PR for?
It doesn't working the notebook filter on navibar.
### What type of PR is it?
Github user astroshim closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/912
---
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 astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/918#issuecomment-221772165
@cloverhearts maybe you need to set up the smtp informations. you can set
the variables in the "conf/zeppelin-site.xml".
---
If your project
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221788304
@minahlee Yes, you're right. It would be better supporting the filter
subdirectory names.
How about creating new PR for filtering subdirectory
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221791290
@bzz I made issue(https://issues.apache.org/jira/browse/ZEPPELIN-886).
---
If your project is set up for it, you can reply to this email and have your
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221803140
@bzz I rebased.
---
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 astroshim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/919
support changing tabsize of paragraph
### What is this PR for?
The code editor tab size is fixed as 4.
It would be better to support changing tabsize to users
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-222044643
@minahlee It's my mistake. I reverted.
---
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 astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/919#issuecomment-222048031
@Leemoonsoo Good point. I agree with you that changing tab size in the
interpreter or notebook is better than paragraph. so I think Configuring tab
size
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/919#issuecomment-222195524
@Leemoonsoo It really makes sense and I thought that too.
The reason why I thought about configuring tab size in notebook is better
was just because
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/921#issuecomment-222408542
@cloverhearts Really nice job. but I got an follwing error when i click the
paragraph in job
![image](https://cloud.githubusercontent.com/assets
GitHub user astroshim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/929
Hotfix - initial notebook name does not increase.
### What is this PR for?
To increase initial notebook name.
### What type of PR is it?
Bug Fix
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/941
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
Github user astroshim commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/942
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
GitHub user astroshim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/946
ZEPPELIN-937 : broken layout of Interpreter and Configurations table in
mobile view.
### What is this PR for?
Current contents table of Interpreter and Configurations menu are
Github user astroshim commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/938
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
Github user astroshim commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/959
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
GitHub user astroshim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/964
filtering subdirectory names.
### What is this PR for?
It needs to filtering subdirectory names in navbar.
### What type of PR is it?
Bug Fix
GitHub user astroshim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/965
[ZEPPELIN-957] Normalize note name.
### What is this PR for?
It's better if notebook name can be normalized.
### What type of PR is it?
Improv
Github user astroshim commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/972
It's working nicely.
but Is the correct behavior that the lastResponseUnixTime is increased
whenever call the curl?
Then i can't understand what's the purpose o
Github user astroshim commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/972
@cloverhearts Thank you for your explanation. You mean that the
"LastResponseUnixTime" is for getting notebook information of lastest updated.
---
If your project is set
GitHub user astroshim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/979
Alert message when trying to delete last paragraph.
### What is this PR for?
This PR is for giving proper information to users and blocking unnecessary
communication with zeppelin
Github user astroshim commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/919
@corneadoug Thank you for your concern! I'm thinking to find right way how
to handle this issue.
---
If your project is set up for it, you can reply to this email and have your
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@corneadoug Thank you very much for your review and opinion. I'll follow
your advice.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/964
@corneadoug Thank you for your review.
I'll refactor the code as you mentioned.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@minahlee @corneadoug Then we need two ways to solve this issue?
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
Thank you for clarification.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/964
@corneadoug I refactored the codes that you pointed out. 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@corneadoug @minahlee I fixed the code to hide "Remove" menu on the last
paragraph menu. please review.
---
If your project is set up for it, you can reply to this email and have
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/965
@felixcheung Thank you for pointing out that i missed. I just fixed 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@minahlee I'll take care about that you mentioned. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/965
I really appreciate your detailed review. @Leemoonsoo
I will fix 1,2,3 but in case 4,
I thought ```/MyNote/sub``` and ```MyNote/sub``` are the same from
directory point of view.
Do
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/965
@Leemoonsoo I fixed 1,2,3,4. 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 project does not have this
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@minahlee Could you tell me how can i get your test failure?
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@corneadoug Thank you for your kind explanation.
I know the reason about the error but I couldn't reproduce the error even
if i build like ```mvn clean package```.
Did you build
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@corneadoug Okay. Thank you!!
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1011
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1006
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/979
@corneadoug I really appreciate for your effort. Thank you. I reset my
commits.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/976
LGTM.
@minahlee The selenium test of travis has a problem so @corneadoug is
fixing now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/972
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 astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1018
Blocking the zeppelin-web build error.
### What is this PR for?
This PR is for blocking unnecessary zeppelin-web build errors.
```
[ERROR] npm WARN package.json zeppelin-web@0.0.0
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1018
It's not actual error just error messsage.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1018
@corneadoug Thank you very much for the clarification!
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1021
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 astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1030
[ZEPPELIN-1023] Add more credential apis.
### What is this PR for?
This PR is for supporting various Credential APIs for users.
### What type of PR is it?
Improvement
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1036
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1025
The job page looks greate and working good.
I think it seems to be better if the Job description is changed. :)
Then how can i control the the notebook action in the Job page?
---
If
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1025
@cloverhearts Thanks for the quick response. I understand. :+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
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1037
fix typo the description of interpreter menu.
### What is this PR for?
This PR is for fixing typo of the interpreter menu description.
### What type of PR is it?
Bug Fix
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1037
@minahlee Thanks for pointing out. I just fixed.
---
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 astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1048
[ZEPPELIN-1033] HotFixing of paragraph deletion and re-ordering broken
### What is this PR for?
This PR is for fixing [this
issue](https://issues.apache.org/jira/browse/ZEPPELIN-1033
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1048
@randerzander Thank you for your review. Let me check.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1048
@randerzander Did you build this PR?
You can get this PR from ```git clone https://github.com/astroshim/zeppelin
-b ZEPPELIN-1033```.
My zeppelin is working well as following screenshot
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1048
nice @randerzander thanks for addressing this issue and 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/919
@bzz Thank you for your detail review!
I was worring about changing tab size per interpreter as @Leemoonsoo
suggests is more hard to config than per notebook or per paragraph.
How about
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1030
Thank you for your review @bzz and @AhyoungRyu and @cloverhearts !
And It really makes sense @bzz comments.
Let me fix them.
---
If your project is set up for it, you can reply to this
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1058
BugFix of Automatically adds %.* of previous paragraph's typing.
### What is this PR for?
This PR is to fix bug from https://github.com/apache/zeppelin/pull/806.
The default interp
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/919
Thank you for your detailed explanation @Leemoonsoo and I agree with moving
interpreter-setting.json.
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1064
BugFix-blocking of blank values insertion on the Credential page.
### What is this PR for?
This PR blocks the blank values insertion on the Credential page and
changes the success
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1064
@bzz I fixed as your comments. 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 project does not have this
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1058
@Leemoonsoo I rebased.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1058
The selenium test has passed!!
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1058
@Leemoonsoo Sorry for late response. I fixed.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1079
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1030
@bzz @AhyoungRyu Could you 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 astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1090
[MINOR] Blocking the minus cursor value.
### What is this PR for?
This PR is for blocking minus cursor value on the paragraph.
If we put the ```ctrl+.``` for auto completion on the
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1030
@bzz okay I will. Thank you!
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1090
Thank you for your review! @bzz
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1030
@bzz I fixed code and the build problem that you mentioned is gone.
but another following build fail occurred.
I think following problem seems not related this PR, what do you think
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1097
init newRepoSetting values
### What is this PR for?
This PR fixes initialize value bug of ```Add New Repository```.
### What type of PR is it?
Bug Fix
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1030
---
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
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1030
[ZEPPELIN-1023] Add more credential apis.
### What is this PR for?
This PR is for supporting various Credential APIs for users.
### What type of PR is it?
Improvement
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1097
It seems that the CI error is not related this PR.
```
Tests in error:
ZeppelinIT.testSparkInterpreterDependencyLoading:217->AbstractZeppelinIT.createNewNote:119
û Time
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1100
Really great feature! Thanks for making this.
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1030
The CI fail has gone.. I just did trigger CI 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
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1097
[Bugfix] init RepoSetting values
### What is this PR for?
This PR fixes initialize value bug of ```Add New Repository```.
### What type of PR is it?
Bug Fix
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1097
---
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
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1104
[ZEPPELIN-1078] Does not show the text result in the paragraph sometimes.
### What is this PR for?
This PR fixes the problem that does not show the text result in the
paragraph sometimes
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1064
---
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
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1064
BugFix-blocking of blank values insertion on the Credential page.
### What is this PR for?
This PR blocks the blank values insertion on the Credential page and
changes the success
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1097
I re-triggered but other error occurred.
```
Tests run: 5, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.579 sec
<<< FAILURE! - in org.apache.zeppelin.python.PythonInterp
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1104
It seems that the CI fail is related to
https://github.com/apache/zeppelin/pull/1084.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1099
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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1097
[Bugfix] init RepoSetting values
### What is this PR for?
This PR fixes initialize value bug of ```Add New Repository```.
### What type of PR is it?
Bug Fix
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1097
---
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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1097
CI has passed. 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
1 - 100 of 541 matches
Mail list logo
