[GitHub] zeppelin issue #2309: [ZEPPELIN-2482] added font size for paragraph

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2309 @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 wishes

[GitHub] zeppelin pull request #2268: [ZEPPELIN-2403] interpreter property widgets

2017-06-22 Thread tinkoff-dwh
GitHub user tinkoff-dwh reopened a pull request: https://github.com/apache/zeppelin/pull/2268 [ZEPPELIN-2403] interpreter property widgets ### What is this PR for? I spoiled the previous PR #2251 Added widgets (string, text, url, password, url, checkbox) to properties

[GitHub] zeppelin pull request #2268: [ZEPPELIN-2403] interpreter property widgets

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh closed the pull request at: https://github.com/apache/zeppelin/pull/2268 --- 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

Re: New committer: Park Hoon

2017-06-22 Thread Khalid Huseynov
Congrats! On Fri, Jun 23, 2017 at 1:50 AM, DuyHai Doan wrote: > Congratulation and welcome to Park Hoon! > > On Thu, Jun 22, 2017 at 6:22 PM, Felix Cheung > wrote: > > > Congrats! > > > > > > From:

[GitHub] zeppelin issue #2431: ZEPPELIN-2615. Upgrade pig to 0.17.0 to support spark ...

2017-06-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2431 @felixcheung Please help review. The failed test is not relevant. --- 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] zeppelin issue #2422: [WIP] [ZEPPELIN-2627] Interpreter refactor

2017-06-22 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2422 @jongyoul Thanks for the quick review. I didn't intended to make such large change at the beginning, but found many things are coupled together that I have to make such large change. Several

[GitHub] zeppelin issue #2430: [Website] [MINOR] Link install guide around download b...

2017-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2430 Thanks @khalidhuseynov for taking a look. Merge and deploy if no further comment. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Created] (ZEPPELIN-2679) JDBC. Precode for session

2017-06-22 Thread Tinkoff DWH (JIRA)
Tinkoff DWH created ZEPPELIN-2679: - Summary: JDBC. Precode for session Key: ZEPPELIN-2679 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2679 Project: Zeppelin Issue Type: New Feature

[GitHub] zeppelin issue #2426: Remove ticket after logout

2017-06-22 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/2426 Merging this to master and branch-0.7 if no more discussion. --- 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] zeppelin pull request #2431: ZEPPELIN-2615. Upgrade pig to 0.17.0 to support...

2017-06-22 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2431 ZEPPELIN-2615. Upgrade pig to 0.17.0 to support spark engine ### What is this PR for? Pig 0.17.0 has just been released. This PR is to upgrade pig to 0.17.0 and support spark engine

[GitHub] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2428 https://travis-ci.org/herval/zeppelin/builds/245889847 try restart 3 and 5 job --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin pull request #2430: [Website] [MINOR] Link install guide around dow...

2017-06-22 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2430 [Website] [MINOR] Link install guide around download button ### What is this PR for? This PR adds link to installation guide doc around download button ### Screenshots (if

[GitHub] zeppelin pull request #2397: [ZEPPELIN-2592] Ensure open stream is closed

2017-06-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2397 --- 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] zeppelin issue #2419: [ZEPPELIN-2666] docs: Add docker usage to download pag...

2017-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2419 Reopened [ZEPPELIN-1229](https://issues.apache.org/jira/browse/ZEPPELIN-1229) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] zeppelin pull request #2419: [ZEPPELIN-2666] docs: Add docker usage to downl...

2017-06-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2419 --- 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] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2428 It seems apache/zeppelin master (which is what this branch is based on) is failing the build - should I rebase it to 0.7 instead, or wait for master to be fixed? Please let me know what should I

[GitHub] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2428 yep, getMaxResult is correct. i thought the condition <= ```while (displayRowCount < getMaxResult() && resultSet.next())``` and last next() will request the next part (fetchSize) ---

[GitHub] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2428 Why +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, or if the

[GitHub] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2428 I think it's impossible to test. --- 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] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2428 it would be better getMaxResult() + 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] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2428 I'm also not exactly sure how to write a test for this, as the JDBCInterpreterTest doesn't use anything in the line of mocks (and the outside behavior - limiting the number of returned rows - is

[GitHub] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread herval
Github user herval commented on the issue: https://github.com/apache/zeppelin/pull/2428 @tinkoff-dwh you're right, picked the wrong method - using setFetchSize now --- 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

Re: Service 'sparkDriver' could not bind on port with dockerized zeppelin

2017-06-22 Thread Иван Шаповалов
Solved this. It appears that 'spark.driver.bindAddress' should point to docker container IP while 'spark.driver.host' to outer host IP Thanks anyway 2017-06-22 19:16 GMT+03:00 Иван Шаповалов : > Help needed. > > 1. got zeppelin running in a docker container > 2. got

[GitHub] zeppelin issue #2426: Remove ticket after logout

2017-06-22 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2426 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] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2428 user will not know that the table has more rows and the result is not complete. Maybe set fetchSize? --- If your project is set up for it, you can reply to this email and have your reply

Service 'sparkDriver' could not bind on port with dockerized zeppelin

2017-06-22 Thread Иван Шаповалов
Help needed. 1. got zeppelin running in a docker container 2. got remote spark standalone cluster I want to run paragraphs against I have: - created a setting with -- master - $MASTER_IP/7077 -- 'spark.driver.host' - ip of the docker container 172.18.0.2 -- 'spark.driver.port' - free port

Re: Service 'sparkDriver' could not bind on port with dockerized zeppelin

2017-06-22 Thread Иван Шаповалов
zeppelin version is 0.7.2 2017-06-22 19:16 GMT+03:00 Иван Шаповалов : > Help needed. > > 1. got zeppelin running in a docker container > 2. got remote spark standalone cluster I want to run paragraphs against > > > I have: > - created a setting with > -- master -

Re: New committer: Park Hoon

2017-06-22 Thread DuyHai Doan
Congratulation and welcome to Park Hoon! On Thu, Jun 22, 2017 at 6:22 PM, Felix Cheung wrote: > Congrats! > > > From: astros...@gmail.com on behalf of Hyung Sung > Shim > Sent: Thursday, June

Re: New committer: Park Hoon

2017-06-22 Thread Felix Cheung
Congrats! From: astros...@gmail.com on behalf of Hyung Sung Shim Sent: Thursday, June 22, 2017 9:10:12 AM To: dev Subject: Re: New committer: Park Hoon Congratulations!!! 2017-06-22 20:56 GMT+09:00 Jeff Zhang

Re: New committer: Park Hoon

2017-06-22 Thread Hyung Sung Shim
Congratulations!!! 2017-06-22 20:56 GMT+09:00 Jeff Zhang : > Congratulations > > > CHALLA 于2017年6月22日周四 下午7:44写道: > > > Welcome Park Hoon > > > > On Wed, Jun 21, 2017 at 11:17 PM, moon soo Lee wrote: > > > > > The Project Management

[jira] [Created] (ZEPPELIN-2678) Pyspark cell fails to execute, but normal spark code in scala executes fine

2017-06-22 Thread Nassir (JIRA)
Nassir created ZEPPELIN-2678: Summary: Pyspark cell fails to execute, but normal spark code in scala executes fine Key: ZEPPELIN-2678 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2678 Project:

[GitHub] zeppelin pull request #:

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the pull request: https://github.com/apache/zeppelin/commit/c90a3c2e51f1f1853c179b42ce4b241c518d31e3#commitcomment-22703746 maybe the problem is that the source files are taken as archive from github --- If your project is set up for it, you can

[GitHub] zeppelin pull request #:

2017-06-22 Thread necosta
Github user necosta commented on the pull request: https://github.com/apache/zeppelin/commit/c90a3c2e51f1f1853c179b42ce4b241c518d31e3#commitcomment-22702764 @tinkoff-dwh , I don't know what you mean by "broken build from source" Can you inform: 1) OS: Win vs Linux vs Other

[GitHub] zeppelin pull request #2429: [ZEPPELIN-2675] Distributing Jars when using an...

2017-06-22 Thread marcotagliabue
GitHub user marcotagliabue opened a pull request: https://github.com/apache/zeppelin/pull/2429 [ZEPPELIN-2675] Distributing Jars when using an external flink cluster ### What is this PR for? This PR intends to make Flink interpreter able to distribute external dependencies

[GitHub] zeppelin issue #2415: [ZEPPELIN-2661] [WIP] Test: personalized mode action

2017-06-22 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2415 Hi, there. I just added two test cases as the @1ambda comment and have written PR description as well. --- If your project is set up for it, you can reply to this email and have your reply

Re: New committer: Park Hoon

2017-06-22 Thread Jeff Zhang
Congratulations CHALLA 于2017年6月22日周四 下午7:44写道: > Welcome Park Hoon > > On Wed, Jun 21, 2017 at 11:17 PM, moon soo Lee wrote: > > > The Project Management Committee (PMC) for Apache Zeppelin has invited > Park > > Hoon (1ambda) to become a committer and we

Re: New committer: Park Hoon

2017-06-22 Thread CHALLA
Welcome Park Hoon On Wed, Jun 21, 2017 at 11:17 PM, moon soo Lee wrote: > The Project Management Committee (PMC) for Apache Zeppelin has invited Park > Hoon (1ambda) to become a committer and we are pleased to announce that has > accepted. > > Please join us to welcome. > > The

[jira] [Created] (ZEPPELIN-2677) Zeppelin on windows does throws error after installation and trying to run a cell

2017-06-22 Thread Nassir (JIRA)
Nassir created ZEPPELIN-2677: Summary: Zeppelin on windows does throws error after installation and trying to run a cell Key: ZEPPELIN-2677 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2677

[GitHub] zeppelin pull request #:

2017-06-22 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the pull request: https://github.com/apache/zeppelin/commit/c90a3c2e51f1f1853c179b42ce4b241c518d31e3#commitcomment-22700500 @necosta @Leemoonsoo broken build from source. git.properties not generated and

[jira] [Created] (ZEPPELIN-2676) Recent notes

2017-06-22 Thread Tinkoff DWH (JIRA)
Tinkoff DWH created ZEPPELIN-2676: - Summary: Recent notes Key: ZEPPELIN-2676 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2676 Project: Zeppelin Issue Type: New Feature

[jira] [Created] (ZEPPELIN-2675) Distributing Jars when using an external flink cluster

2017-06-22 Thread Marco Tagliabue (JIRA)
Marco Tagliabue created ZEPPELIN-2675: - Summary: Distributing Jars when using an external flink cluster Key: ZEPPELIN-2675 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2675 Project:

[GitHub] zeppelin pull request #2423: [ZEPPELIN-2672] Fix interpreter repos url toolt...

2017-06-22 Thread necosta
GitHub user necosta reopened a pull request: https://github.com/apache/zeppelin/pull/2423 [ZEPPELIN-2672] Fix interpreter repos url tooltip ### What is this PR for? Fixing regression on showing interpreter repos tooltip ### What type of PR is it? Bug Fix

[GitHub] zeppelin pull request #2423: [ZEPPELIN-2672] Fix interpreter repos url toolt...

2017-06-22 Thread necosta
Github user necosta closed the pull request at: https://github.com/apache/zeppelin/pull/2423 --- 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] zeppelin issue #2398: [ZEPPELIN-2622] Add Zeppelin Git info REST API

2017-06-22 Thread necosta
Github user necosta commented on the issue: https://github.com/apache/zeppelin/pull/2398 Thanks guys! --- 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] zeppelin issue #2371: [ZEPPELIN-2596] Improving documentation page

2017-06-22 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2371 @khalidhuseynov merged and checked that newly added thing included in this PR.

[GitHub] zeppelin pull request #2424: [ZEPPELIN-2670] fix: DON'T reset all helium con...

2017-06-22 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/2424 --- 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] zeppelin pull request #2424: [ZEPPELIN-2670] fix: DON'T reset all helium con...

2017-06-22 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/2424 [ZEPPELIN-2670] fix: DON'T reset all helium config when saving display order ### What is this PR for? Reordering vis package using the `save` button resets all helium configuration.