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/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
reply
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 also
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/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/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/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 GitHub
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 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/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
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
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 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?
Improvement
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
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 of step 3. Could
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?
Bug
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
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/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/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-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 well
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 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 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 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 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
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
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
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 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/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
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
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
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/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 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 interpreter
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/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
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/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 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/1104
re-trigger CI
---
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 reopened 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/1104
---
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 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/919
https://github.com/apache/zeppelin/pull/1218 is better way.
---
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 closed the pull request at:
https://github.com/apache/zeppelin/pull/919
---
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/1110
Is this PR going 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1185
@AhyoungRyu Thank you for your opinion.
Your idea is great but now is redirected all to home when requested wrong
path. (you can see the related code
[here](https://github.com/apache
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1227
[ZEPPELIN-1198][Spark Standalone] Documents for running zeppelin on
production environments.
### What is this PR for?
This PR is for documentation for running zeppelin on production
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1223
@prabhjyotsingh Thank you for share your great idea.
This PR is for interpreter authorization but your idea seems include user
management.
I think it should be separated and I agree
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1223
@Leemoonsoo Thank you for your review and I think your opinion makes sense.
I'll change the UI.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1117
Ping
---
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 closed the pull request at:
https://github.com/apache/zeppelin/pull/1229
---
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/1229
re-trigger CI
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1229
[BugFix] Show checkbox for "Connect to existing process" on interpreter
menu.
### What is this PR for?
This PR fixes a bug of showing checkbox for "Connect to e
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1229
---
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/1229
re-trigger CI
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1229
[BugFix] Show checkbox for "Connect to existing process" on interpreter
menu.
### What is this PR for?
This PR fixes a bug of showing checkbox for "Connect to e
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1229
[BugFix] Show checkbox for "Connect to existing process" on interpreter
menu.
### What is this PR for?
This PR fixes a bug of showing checkbox for "Connect to e
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1248
tested. 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/zeppelin/pull/1193
@bzz I added log and tested. Thank you for taking care of 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1318
Spark2.0 & hadoop2.3 is working well.
![image](https://cloud.githubusercontent.com/assets/3348133/17588549/d611d7ca-600a-11e6-87f6-4a692fbae273.png)
---
If your project is se
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1318
Do you know what version of spark is popular? I will test 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1257
@AhyoungRyu I fixed you pointed out and I'll create the document for this
separated PR.
Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1257
@AhyoungRyu Let me check. 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1257
@prabhjyotsingh Great. Thank you for giving me the guidance.
---
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/1257
Thank you for detail review @bzz and @felizbear
I'll fix the codes.
---
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/1318
Then what about supporting latest verison(Spark2.0 & hadoop2.7)?
---
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 pro
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1318
Spark2.0 & hadoop2.7 is working well.
- YARN
![image](https://cloud.githubusercontent.com/assets/3348133/17622873/fc1b827c-60d7-11e6-80ec-e8e9890a5c99.png)
- hdfs
![i
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1193
@bzz I rebased. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1318
@AhyoungRyu I'll update. Thank you for giving me the informations.
---
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 opened a pull request:
https://github.com/apache/zeppelin/pull/1318
[ZEPPELIN-1280][Spark on Yarn] Documents for running zeppelin on production
environments.
### What is this PR for?
This PR is for the documentation of running zeppelin on production
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1171
Build error has gone and working good!
---
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/1176
Great feature!
Can you add documents for this or please explain more detail to review?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1117
@bzz I totally agree with you so I did try to remove things you mentioned
but it's not easy.
Let me try one more. Thanks!
---
If your project is set up for it, you can reply to this email
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1117
@bzz Thank you kindly for telling like that.
What do you think about tagging meta value than removing keywords like
following image?
![image](https://cloud.githubusercontent.com
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1191
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/zeppelin/pull/1193
please review 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 feature
enabled
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1193
[ZEPPELIN-1192] Block pyspark paragraph hang.
### What is this PR for?
This PR block pyspark paragraph hang.
### What type of PR is it?
Bug Fix
### What
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1196
Oh sorry.
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1185
Redirect home when requested a wrong notename path.
### What is this PR for?
This PR for redirecting to zeppelin home when requested wrong path.
### What type of PR
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 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/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
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
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1104
---
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/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 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
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
1 - 100 of 435 matches
Mail list logo