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
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
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/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
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/1411
@bzz Thank you for review and sorry for making you confused.
I'll fix version to 0.7.0.
---
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/1396
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 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
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/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 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1385
Please fix the CI-fails.
```
[INFO] There are 5 checkstyle errors.
[ERROR] InterpreterFactory.java[302] (sizes) LineLength: Line is longer
than 100 characters (found 106
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?
![image](https://cloud.githubusercontent.com/assets
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 email
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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1358
@mikhaildubkov `dev` doesn't exists on 0.6.x so you should use `master`
branch for this.
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/1302
@corneadoug I just re-based 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/1396
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/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 'Connection
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1396
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1358
Sorry maybe it's my mistake to set jira version.
---
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/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
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
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
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 `%html
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
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 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
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/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
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
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 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 'Connection
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 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
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
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/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/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 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
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/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 email
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
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.
![st](https
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1411
change to create local interpreter when option is false.
### What is this PR for?
This PR fixes
[ZEPPELIN-1417](https://issues.apache.org/jira/browse/ZEPPELIN-1417
).
### What
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1318
@zjffdu You're right, usually users don't make their production using
docker.
but We have been received many question about running Zeppelin on their
production environments so I think
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
\cc @Leemoonsoo I fixed UI like following screenshot and process. please
review.
![o](https://cloud.githubusercontent.com/assets/3348133/18028272/33d20810-6cb5-11e6-9026-b51ddf8eb972.gif
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1366
LGTM. :+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 and wishes so
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
Thank you very much about reviewing this PR. :smile:
---
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/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 have
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
GitHub user astroshim reopened a pull request:
https://github.com/apache/zeppelin/pull/1358
[ZEPPELIN-1365] Error of Zeppelin Application in development mode.
### What is this PR for?
This PR fixes the bug of running application in development mode.
### What type
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1358
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/1358
---
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/1358
[ZEPPELIN-1365] Error of Zeppelin Application in development mode.
### What is this PR for?
This PR fixes the bug of running application in development mode.
### What type
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1302
@Leemoonsoo Your suggestion about UI is definitely better!
and also makes sense about restarting action on interpreter binding page.
I will fix them.
---
If your project is set up
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/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 `completion
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
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 condition
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
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/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 does
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
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 reopened 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
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1465
---
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/1465
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 commented on the issue:
https://github.com/apache/zeppelin/pull/1465
please 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 astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1433
I did rebase and 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
GitHub user astroshim reopened 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
CI is green now. 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/1617
In order to call `commit()` in mysql, `auto commit` should be `false`
otherwise it throws `SQLException` like following.
```
java.sql.SQLException: Can't call commit when autocommit=true
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
Thank you @Leemoonsoo and @nathanscully to review.
Let me address issues.
---
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 closed the pull request at:
https://github.com/apache/zeppelin/pull/1465
---
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/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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1465
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
Github user astroshim closed the pull request at:
https://github.com/apache/zeppelin/pull/1465
---
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/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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
@nathanscully I fixed that you issued. Could you review again?
@Leemoonsoo What `Entity` name did you set? `Entity` should be set as
`jdbc`.`InterpreterName`.
---
If your project is set
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, you
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 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/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
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/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 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 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 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 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
Github user astroshim commented on the issue:
https://github.com/apache/zeppelin/pull/1539
@1ambda Thank you for the testing and great suggestion about the `Entity`
name.
Let me fix it.
---
If your project is set up for it, you can reply to this email and have your
reply appear
201 - 300 of 435 matches
Mail list logo