Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2594
Ready to merge! ð
---
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/2594
[ZEPPELIN-2944] Cannot launch Spark interpreter for non-local mode
### What is this PR for?
When I used Spark interpreter for non-local mode, I got
```
java.lang.RuntimeException
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/2594
---
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2594
Oops! I was looking at wrong place ð¢
Thanks @felixcheung !
I restarted failed jobs.
After that, should I re-open this PR?
---
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2594
Hi! How can I make Jenkins green?
All tests passed on my Travis:
https://travis-ci.org/tae-jun/zeppelin/builds/277762975
---
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/2594
[ZEPPELIN-2944] Cannot launch Spark interpreter for non-local mode
### What is this PR for?
When I used Spark interpreter for non-local mode, I got
```
java.lang.RuntimeException
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/2594
---
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/2594
[ZEPPELIN-2944] Cannot launch Spark interpreter for non-local mode
### What is this PR for?
When I used Spark interpreter for non-local mode, I got
```
java.lang.RuntimeException
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/2594
---
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2594
@zjffdu Thanks for the review! ð
---
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/2594
[ZEPPELIN-2944] Cannot launch Spark interpreter for non-local mode
### What is this PR for?
When I used Spark interpreter for non-local mode, I got
```
java.lang.RuntimeException
Tae Jun Kim created ZEPPELIN-2944:
-
Summary: Cannot launch Spark interpreter
Key: ZEPPELIN-2944
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2944
Project: Zeppelin
Issue Type: Bug
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2547
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/2543
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/2543
@byung-u
Oh, I didn't see the code and commented above...ð
But I found you missed something!
The original code has an error because it always gets `lastRunningUnixTime`
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2543
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2451
Now it works perfectly ð
LGTM!
---
If your project is set up for
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/2432
---
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/2432
@zjffdu Ok. 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 wishes
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2432
@zjffdu Thanks for the review! I see.
Then would it better to close this PR and wait for it?
What do you think? :)
---
If your project is set up for it, you can reply to this email
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2432
@tinkoff-dwh @soralee Thanks for the reviews ð
---
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/2432
@zjffdu Hi, could you please review this? I think this is related to #2245.
I am not sure that I did the right thing.
Is `Input` class is an abstract class?
Thanks ð
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/2432
[ZEPPELIN-2652] Can't open spark tutorial note in 0.8.0-SNAPSHOT
### What is this PR for?
Fix the bug which cannot open a note containing a dynamic form.
It is not a probl
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2317
@1ambda Thanks for the review! ð
I didn't know that you asked me to add a test ð¢
Was it added after the editing?
But, since you said it is not necessary, I will
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/2316
---
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/2316
OK. Thanks for the review @1ambda!
I'm closing myself ð
---
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2316
Oh sorry I didn't see the last comment
[here](https://issues.apache.org/jira/browse/ZEPPELIN-2405?focusedCommentId=15973784&page=com.atlassian.jira.plugin.system.issuetabpanels:comment
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/2317
[ZEPPELIN-2493] Visualization class should throw an error if an abstract
method is not implemented
### What is this PR for?
`Visualization` class should throw an error if an abstract method
Tae Jun Kim created ZEPPELIN-2493:
-
Summary: Visualization class should throw an error if an abstract
method is not implemented
Key: ZEPPELIN-2493
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2493
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/2316
[DOC] Update documentation: JDK 1.7 -> 1.8
### What is this PR for?
Zeppelin is built with JDK 1.8 since 0.7.1 as far as I know. But it is
still 1.7 on [installation
documentation](ht
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2274
Cool! Significantly better than before ð
---
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/2228
Cool feature! 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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2236
@AhyoungRyu 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/2236
@AhyoungRyu Thanks for the review!
I changed the sentence, and now it looks like this:
> You can add any dependencies in package.json, but you **must include two
dependenc
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/2236
[DOC] Improve documents related to Helium
### What is this PR for?
What I did for the documents:
* Highlight codes
* Follow JSON syntax
* Remove white spaces
And in my
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/2125
@conker84 FYI, you can simply just take a GIF using software called
**LICEcap**, and upload it directly here with drag and drop :)
---
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/2067
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/2065
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/1826
@Leemoonsoo 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/1826
Ci failed on this PR, but it succeeded on my own CI!
https://travis-ci.org/tae-jun/zeppelin/builds/200842368
---
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/1826
[ZEPPELIN-1859] Add MongoNotebookRepo
### What is this PR for?
This PR adds Mongo notebook storage.
The reason that I made this feature is for HA(High Availability).
S3 and Git
Github user tae-jun closed the pull request at:
https://github.com/apache/zeppelin/pull/1826
---
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/1826
Sorry for my late response!
I added a documentation for MongoDB notebook storage and rebased.
And now it's ready for a review ð
Please test that it operates as I desc
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1903
@AhyoungRyu @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
GitHub user tae-jun opened a pull request:
https://github.com/apache/zeppelin/pull/1903
[DOCS] Reflect changed default storage to doc
### What is this PR for?
Reflect effects caused by changing the default notebook storage
VFSNotebookRepo to GitNotebookRepo
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1730
@AhyoungRyu Finally, it's green!!! Thanks for your care ð
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1730
[ZEPPELIN-1736] Introduce trash & enable removing folder
### What is this PR for?
.
S3 and
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1730
[ZEPPELIN-1736] Introduce trash & enable removing folder
### What is this PR for?
 Merry Christmas~
---
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/1730
Thanks @1ambda ð
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1730
[ZEPPELIN-1736] Introduce trash & enable removing folder
### What is this PR for?
Now, only `remove permanently` and `restore` buttons are shown.
https
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1730
@AhyoungRyu Always thanks for the review :) Also thanks for your
understanding.
@1ambda Thanks for the review! I appreciate it and sorry for my late
response.
That was a really nice
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1730
[ZEPPELIN-1736] Introduce trash & enable removing folder
### What is this PR for?
---
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/1760
ð
---
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/1730
@AhyoungRyu Always thanks for your nice review ð
Yes, it's intended. A time stamp is generated only if a `folder` already
exists in the trash. But it's not generated whe
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1730
Thanks @Leemoonsoo ð
---
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/1730
### CI is green! ð
I **changed Selenium test codes** affected by my PR and it works now.
Failed reason
`remove note` button changed to `move to trash` but test code still
GitHub user tae-jun reopened a pull request:
https://github.com/apache/zeppelin/pull/1730
[ZEPPELIN-1736] Introduce trash & enable removing folder
### What is this PR for?
It failed only at Selenium test twice. I guess it's because I **change
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1730
Thanks, @1ambda ð
---
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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1730
[ZEPPELIN-1736] Introduce trash & enable removing folder
### What is this PR for?
---
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/1723
I tested on Firefox and the error is gone. LGTM ð
And I guess
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/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/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
1 - 100 of 206 matches
Mail list logo
