Github user mwkang closed the pull request at:
https://github.com/apache/zeppelin/pull/1290
---
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 mwkang closed the pull request at:
https://github.com/apache/zeppelin/pull/1350
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1350
I am not sure what is my next step. And this PR was created very long time
ago. So enviroment changed before. I think other contributor can resolve better
than me. So I closed this PR.
Thanks
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
I am not sure what is my next step. And this PR was created very long time
ago. So enviroment changed before. I think other contributor can resolve better
than me. So I closed this PR.
Thanks
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1350
Can anyone help this PR?
I don't know the next step.
---
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 doe
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
@corneadoug
Thanks for you review. I checked them.
Before I fix it, I have several questions.
Do you mean do not change CTRL + ENTER function?
And Is this PR not useful for
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1350
@Leemoonsoo When paragraph received a SIGTERM, Job result code is
INCOMPLETE.
Before When Job result code is INCOMPLETE, Status is FINISHED.
I think in this situation user can not realize
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
@corneadoug Thanks for your comment. I rebased my branch.
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1350
I added test. Please review PR, thanks for your consideration.
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
There are 2 errors.
1. `ZeppelinSparkClusterTest.zRunTest:204 expected: but
was:` does not reproduce.
2.
`AuthenticationIT.testGroupPermission:179->AbstractZeppelinIT.pollingWait
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1330
I'm sorry for no-reply long time. my plate was full.
I will finish other 2 PRs and then If ZEPPELIN-1437 and ZEPPELIN-1438 are
status is unresolved, I will
try to resolve.
If
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1350
@felixcheung Okay! I will add unit test. Thanks for your comment!
---
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 mwkang opened a pull request:
https://github.com/apache/zeppelin/pull/1350
[ZEPPELIN-1351]Paragraph status should be ERROR if Paragraph is terminated
with SIGTERM
### What is this PR for?
When Paragraph received a SIGTERM. ExitValue: 143, Status of the paragraph
GitHub user mwkang reopened a pull request:
https://github.com/apache/zeppelin/pull/1290
[ZEPPELIN-708]shift positions automatically to the next cell
### What is this PR for?
* Add and modify shortcut function
* Shift + Enter: Run paragraph, move focus to next
* Ctrl
Github user mwkang closed the pull request at:
https://github.com/apache/zeppelin/pull/1290
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
`[ERROR] Failed to execute goal
com.github.eirslett:frontend-maven-plugin:0.0.25:npm (npm install) on project
zeppelin-web: Failed to run task: 'npm install --color=false' failed. (err
Github user mwkang closed the pull request at:
https://github.com/apache/zeppelin/pull/1290
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
CI Fail msg is `[ERROR] Failed to execute goal
com.github.eirslett:frontend-maven-plugin:0.0.25:npm (npm install) on project
zeppelin-web: Failed to run task: 'npm install --color=false&#x
GitHub user mwkang reopened a pull request:
https://github.com/apache/zeppelin/pull/1290
[WIP][ZEPPELIN-708]shift positions automatically to the next cell
### What is this PR for?
* Add and modify shortcut function
* Shift + Enter: Run paragraph, move focus to next
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1330
@bzz https://github.com/apache/zeppelin/pull/1330#issuecomment-239997747 It
did not reproduce in local. Can you check this?
---
If your project is set up for it, you can reply to this email and
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
ping @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
enabled and wishes so, or if
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
@resec That is more make sense. I will change when New cell added, Cursor
position is next line.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1330
@bzz I will check it. Thanks for review.
When I fix CI, I'll mention you.
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1330
CI error
```
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 76.102 sec
- in org.apache.zeppelin.integration.SparkParagraphIT
Results :
Tests in error
GitHub user mwkang opened a pull request:
https://github.com/apache/zeppelin/pull/1330
[ZEPPELIN-1144]Zeppelin home page should only list notebooks with read or
write permission
### What is this PR for?
If logged in user does not have Read and Write permission for a notebook
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
I remove `angular.element('#' + focusedParagraph +
'_paragraphColumn_main').scope().goToLineEnd()`, When paragraph runs.
Because each ace editor keeps cursor position.
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
@bzz I think This PR have some trouble merging to master.
Because When bzz leave comment, I also leave comment immediately.
So bot did not recognize this PR.
Could you mind if you check
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
@bzz 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 and wishes so, or
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
@bzz Thanks for your review. I rebase on to master and force push.
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
I am not sure. But I think it is persist error.
```
Failed tests:
ParallelSchedulerTest.testRun:63 expected: but was:
```
Anyway.. @Leemoonsoo I change code that it does
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
@bzz No, It does not persists every time.
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
> If you re-trigger CI more (close&re-open this PR) - does the same issue
persists every time?
I'm not sure if it persists. I will re-tigger after fix the issue.
---
If your proje
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
I am not sure it is related with this PR.
When I run it, it passed.
```
Failed tests:
ParallelSchedulerTest.testRun:63 expected: but was:
```
---
If your project is set up
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
I am not sure this error is related with this issue.
```
Failed tests:
ZeppelinSparkClusterTest.zRunTest:204 expected: but was:
```
---
If your project is set up for it, you can
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
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, or
Github user mwkang closed the pull request at:
https://github.com/apache/zeppelin/pull/1290
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
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, or
GitHub user mwkang reopened a pull request:
https://github.com/apache/zeppelin/pull/1290
[ZEPPELIN-708]shift positions automatically to the next cell
### What is this PR for?
Add and modify shortcut function
* Shift + Enter: Run paragraph, move focus to next
* Ctrl
GitHub user mwkang reopened a pull request:
https://github.com/apache/zeppelin/pull/1248
[ZEPPELIN-1069]Ignore implicit interpreter when user enter wrong
interpreter name
### What is this PR for?
Ignore implicit interpreter when user enter wrong interpreter name
linked to
Github user mwkang closed the pull request at:
https://github.com/apache/zeppelin/pull/1248
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
I handle that unnecessary persisted in note.json.
I find some problem.
While new paragraph is added in runParagraph, focus is not end of line.
It ignore space character
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
@jongyoul Sorry, I missed it. I will check. 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
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
@Leemoonsoo Yes, I agree with you.
I will change that. 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
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1290
Could you mind if I am going to resolve that consider running paragraph is
successful on new issue?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user mwkang opened a pull request:
https://github.com/apache/zeppelin/pull/1290
[ZEPPELIN-708]shift positions automatically to the next cell
### What is this PR for?
Add and modify shortcut function
* Shift + Enter: Run paragraph, move focus to next
* Ctrl + Enter
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
@bzz I changed StringUtils is static import. Can you check the code?
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
@bzz I'm worried about that `static import` confused.
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
I refactor method name.
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1248
@jongyoul Can you review this PR?
---
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 mwkang opened a pull request:
https://github.com/apache/zeppelin/pull/1248
[ZEPPELIN-1069]Ignore implicit interpreter when user enter wrong
interpreter name
### What is this PR for?
Ignore implicit interpreter when user enter wrong interpreter name
linked to
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1113
@AhyoungRyu Thanks for your advice. Next time, I will try to do that.
---
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 mwkang closed the pull request at:
https://github.com/apache/zeppelin/pull/1113
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1113
This PR is temporarily closed, While I refactor my 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 project does not
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/1113
@jongyoul Sure, I will do that.
I have a question. Is it okay that wrong interpreter is ignored in
[here](https://github.com/apache/zeppelin/pull/1113/files#diff
GitHub user mwkang opened a pull request:
https://github.com/apache/zeppelin/pull/1113
[ZEPPELIN-1069]Ignore implicit interpreter when user enter wrong
interpreter name
### What is this PR for?
Ignore implicit interpreter when user enter wrong interpreter name
linked to
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
Wow thanks for your patience. This PR is a long period and lots of
comments. I'll keep on trying my best.
---
If your project is set up for it, you can reply to this email and have your
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
- [ ] https://travis-ci.org/apache/zeppelin/jobs/138874097
`ZeppelinSparkClusterTest.sparkRTest:90`
- [ ] https://travis-ci.org/apache/zeppelin/jobs/138874103
```
Tests run: 3
Github user mwkang closed the pull request at:
https://github.com/apache/zeppelin/pull/806
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
I reopened this PR. But I think I have to fix some test case.
---
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 mwkang reopened a pull request:
https://github.com/apache/zeppelin/pull/806
[ZEPPELIN-707]Automatically adds %.* of previous paragraph's typing
### What is this PR for?
Automatically adds %pyspark in the code area after paragraph created, if
previous paragr
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
@jongyoul I rebase this PR onto master. I'll create new PR about It ignore
wrong interpreter name
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
@jongyoul If user enter wrong name, I ignore that name. Is it okay?
Or do I try to figure similar name out ?
---
If your project is set up for it, you can reply to this email and have your
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
I am not sure that this test case is you expected
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
@jongyoul @corneadoug
Can you review my codes?
(I had rebased my branch because branch had conflict)
(Um.. CI is failed. :cry: )
---
If your project is set up for it, you can
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
I'll fix after this branch rebases onto develop.
---
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 mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
Okay. I am going to fix it.
I show real interpreter name instead of '%..'
@jongyoul I think about single responsibility. But if you have thought it
is redundant, I will c
Github user mwkang commented on the issue:
https://github.com/apache/zeppelin/pull/806
@corneadoug I fix whole issue. Can you check my code?
---
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 mwkang commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/904
Oh.. I closed JIRA issue.
I am so sorry that I act at my discretion.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user mwkang commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/904
Related PR: https://github.com/apache/incubator-zeppelin/pull/860
Related Issue: https://issues.apache.org/jira/browse/ZEPPELIN-828
---
If your project is set up for it, you can reply
Github user mwkang closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/904
---
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 mwkang commented on the issue:
https://github.com/apache/incubator-zeppelin/pull/904
While I rebase this PR, I found this issue was been resolved
[AuthenticationInfo in master].
So I closed PR.
[AuthenticationInfo in master]:
https://github.com/apache
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-222498405
@corneadoug
Hello, I have a question.
Only after running a query, the cursor go to line end in new paragraph.
If It is added manually, the cursor
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-222479738
- [ ] When you create a new notebook, the interpreter is listed as: %...
Any way to get the default interpreter from back-end and insert it instead
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-221424620
@corneadoug
Oh... I don't consider about that.
Thanks for mention that,
I will fix my code.
---
If your project is set up for it, you can
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-221421745
@corneadoug
Thank you for your comment.
I am sorry I don't understand clearly.
Could you explain more?
What is the meaning get the de
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/904#issuecomment-221245539
I made 2 issue for failing CI.
[ZEPPELIN-876](https://issues.apache.org/jira/browse/ZEPPELIN-876),
[ZEPPELIN-877](https://issues.apache.org/jira/browse
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/904#issuecomment-221210218
@bzz Thanks for your comment. It is really helpful. and now I understand
100%! :smile:
---
If your project is set up for it, you can reply to this email and
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/904#issuecomment-221204328
@bzz I think I don't understand 100%. So I want to more clearly what you
are saying.
If I am impolite, I am so sorry.
Is the meaning I make a
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/904#issuecomment-221146929
@Leemoonsoo Part of Ci are failed. Because..
[ERROR] Failed to execute goal on project zeppelin-zrinterpreter: Could not
resolve dependencies
GitHub user mwkang reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/904
[ZEPPELIN-835]Don't persist authentication info into note.json
### What is this PR for?
Don't persist authentication info into note.json
### What type of PR is i
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/904#issuecomment-221143687
Of course. I close the PR and re-open to re-trigger the ci.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user mwkang closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/904
---
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
82 matches
Mail list logo
