Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1564
---
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/1564
rebuild-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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1564
@minahlee I fixed it.
Thank you for proper comments.
---
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 opened a pull request:
https://github.com/apache/zeppelin/pull/1564
[ZEPPELIN-1585] Testcase for PySparkInterpreter.
### What is this PR for?
This PR adds PySparkInterpreter testcase.
### What type of PR is it?
Improvement
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1538
@mfelgamal I checked point 1 and 3. Thank you fix properly.
Could you also fix to run the `R Tutorial` and `Python Tutorial` too?
I think all tutorial should be ran without error
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
@khalidhuseynov Thank you for sharing your good idea about the dropdown
list.
It might gives convenience users.
and I agree with @Leemoonsoo about define convention for `entity` name
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1538
@mfelgamal Thank you for contributing this.
I have several questions.
1. I think it needs `-it` option for running docker.
so The command to run zeppelin docker should be
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@jongyoul I totally agree with you adding multi-tenancy but how about
separating that to another PR because we also already have restarting
interpreter function in the `Interpreter Setting page
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
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 closed the pull request at:
https://github.com/apache/zeppelin/pull/1302
---
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/1302
[ZEPPELIN-1306] Interpreter restarts on a note.
### What is this PR for?
This PR is for usability of restarting interpreter.
### What type of PR is it?
Improvement
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
@Leemoonsoo Thank you for review this.
I was curious about the `Entity` name should be same with interpreter's.
Could you tell me your idea?
---
If your project is set up for it
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
please review @jongyoul
---
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/zeppelin/pull/1539
[ZEPPELIN-1567] Let JDBC interpreter use user credential information.
### What is this PR for?
This PR is for the multi-tenant of JDBC Interpreter.
User can create a user/password
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1322
I got following checkstyle error while building source.
```
[INFO] There are 1 checkstyle errors.
[ERROR] NotebookServer.java[1381] (sizes) LineLength: Line is longer than
100
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1518
It seems looks great to me.
What do you think @jongyoul ?
---
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/1507
[ZEPPELIN-1534] Does not load dependency library when creating new
interpreter.
### What is this PR for?
This PR fixes dependency library loading bug when creating new interpreter
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1475
@minahlee 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@corneadoug, @Leemoonsoo I re-based and added testcase. 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/1433
@Leemoonsoo 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 reopened a pull request:
https://github.com/apache/zeppelin/pull/1475
[ZEPPELIN-1509] Bug of testcase path in .travis.yml
### What is this PR for?
This PR fixes path of spark testcase.
### What type of PR is it?
Bug Fix
### What
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1475
---
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/1475
@minahlee I just rebased. Thank you for your great efforts.
---
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/1483
@minahlee Thank you for your effort!! :+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 issue:
https://github.com/apache/zeppelin/pull/1475
Thank you @minahlee Please help me to solve PR problem.
---
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/1433
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/1433
[ZEPPELIN-1451] Bug fix of Embedding %html within %table.
### What is this PR for?
This PR fixes the bug of Embedding %html within %table.
It doesn't print properly when the `
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1433
---
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/1475
[ZEPPELIN-1509] Bug of testcase path in .travis.yml
### What is this PR for?
This PR fixes path of spark testcase.
### What type of PR is it?
Bug Fix
### What is
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1433
@Leemoonsoo CI has passed, 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/1465
@fanlu That error seems like occurs before launching pyspark interpreter.
Could you please try completion after launching pyspark interpreter if
works or not like following?
![o
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1451
@AhyoungRyu Thank you so much 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 does not
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1465
@felixcheung you mean 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 not have this feature
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1465
@felixcheung Thank you for quick review.
It's because we should get result message even if `pythonScriptInitialized`
is `true` and I thought `pythonScriptInitialized` on the `compl
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/530
@fanlu As @felixcheung mentioned, you can report issue to
dev@zeppelin.apache.org.
and Could you test https://github.com/apache/zeppelin/pull/1465 for solving
your problem?
---
If your
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1465
[ZEPPELIN-1500] pyspark completion doesn't work.
### What is this PR for?
@fanlu reported bug on https://github.com/apache/zeppelin/pull/530.
This PR fixes bug of checking conditi
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1136
@fanlu Thank you for share your problem.
How did you build master? Did you add `-Ppyspark` to use pyspark on your
build command?
---
If your project is set up for it, you can reply to
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@corneadoug I'm going to add test case soon. :laughing:
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1451
@AhyoungRyu Thank you for your suggestion. I think creating new page for
`Zeppelin on CDH` is better, what do you think?
---
If your project is set up for it, you can reply to this email and
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1448
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/1396
CI has passed. 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 feature
enabled
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1396
---
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/1396
[ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.
### What is this PR for?
This PR is for refactoring code for JDBCInterpreter.
There is no putting 'Connectio
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1451
[ZEPPELIN-1279] Zeppelin with CDH5.x docker document.
### What is this PR for?
This PR is for the documentation of running zeppelin with CDH docker
environment.
and This PR is the part
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1431
---
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/1431
re-build 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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1431
[ZEPPELIN-1446] Fix broken layout of `Create new interpreter` UI.
### What is this PR for?
This PR fixes broken layout of `Create new interpreter` UI.
### What type of PR is
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1431
re-build 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 astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1431
[ZEPPELIN-1446] Fix broken layout of `Create new interpreter` UI.
### What is this PR for?
This PR fixes broken layout of `Create new interpreter` UI.
### What type of PR is
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1431
---
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/1431
I rebased but build failed.
---
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/1433
@Leemoonsoo Thank you for pointing out the problem. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1433
@corneadoug Thank you for your review. I just re-based.
---
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/1431
Thank you @AhyoungRyu and @corneadoug .
I just fixed space problems.
---
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/1431
@AhyoungRyu Thank you for your kind review. I just fix the space.
---
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/1433
[ZEPPELIN-1451] Bug fix of Embedding %html within %table.
### What is this PR for?
This PR fixes the bug of Embedding %html within %table.
It doesn't print properly when the `%htm
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai Could you take a look at the following screenshot?
After the `alluxio` and `angular` dependency error occurs, I just saved
`alluxio` interpreter setting several times but the
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1431
[ZEPPELIN-1446] Fix broken layout of `Create new interpreter` UI.
### What is this PR for?
This PR fixes broken layout of `Create new interpreter` UI.
### What type of PR is
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@Leemoonsoo I agree with your opinion. Let me fix them.
---
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/1369
@doanduyhai I know that number means but as you can see the screenshot i
attached, the number seems doesn't match with count.
---
If your project is set up for it, you can reply to this
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai Sorry to keep you waiting.
It works like a charm now but the error count number at the beginning of
message appears a little strange like following screen shot.
.
### What
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1396
Thanks @jongyoul and there was no problem running mysql.
---
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/1396
I tested `mysql`, `postgresql` and `hive`.
---
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/1302
@corneadoug I fixed it. please review. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1396
\cc @jongyoul I changed some codes to use `Connection Pool` that is
`org.apache.commons.dbcp2`, please review.
---
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/1369
@doanduyhai Could you check the infinitely downloading library problem(like
following screen shot)?
Strangely this problem happens all interpreter except spark. Is this
problem happen only
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1396
I will do it in this PR and change the title 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1396
@jongyoul I will refactor the code to use database pool.
---
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/1396
\cc @jongyoul Please review this PR if i misunderstand.
---
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/zeppelin/pull/1396
[ZEPPELIN-1405] Remove propertyKeyUnusedConnectionListMap variable in
JDBCInterpreter
### What is this PR for?
This PR is for refactoring code for JDBCInterpreter.
### What
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@corneadoug Actually it's very short time to restart interpreter job
because it's just kill interpreter process or thread.
How about alerting message when request is success?
-
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
Proxy repo tested and it works properly. :+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 issue:
https://github.com/apache/zeppelin/pull/1389
@AhyoungRyu Thank you!! :+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai BTW how come only spark interpreter shows `error message bar`?
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
@doanduyhai Good. I am going to check first commit(proxy) soon.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1358
Thank you @corneadoug
---
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/1302
@AhyoungRyu I think `deactivated` button is better so I updated it. 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
Wow very quick response!!:
I think set `combineDuplications` true is better but I don't have much
talent in UI.:)
---
If your project is set up for it, you can reply to this emai
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1369
About the second commits, Is it intended behavior that the error message
bar is stacked whenever save the other interpreter setting?
 LineLength: Line is longer
than 100 characters (found 106
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1389
@AhyoungRyu Thank you for quick review. I fixed what you pointed out.
---
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/1389
[ZEPPELIN-1279] Spark on Mesos Docker.
### What is this PR for?
This PR is for the documentation of running zeppelin on production
environments especially spark on mesos via Docker
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
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, or if the
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1358
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, or if the
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1358
@AhyoungRyu Thank you for reviewing!!
Did you clear your browser cache?
---
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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
I just changed to 'icon'.
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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@AhyoungRyu I agree with you!
I changed position and restart icon to button type, is it okay with button
or just icon is better?
