[GitHub] zeppelin issue #2143: [ZEPPELIN-2266] Parse helium description as HTML

2017-03-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2143
  
@1ambda Looks good to me 👍 

It's not in the scope of this PR, but as ppl are getting have more interest 
in Helium, i think we need a guide which explains how ppl can add helium 
package related fields to `package.json`  e.g.  
 - description (we can let ppl know they can use html tag as you did)
 - icon (since ppl normally don't use `icon` field in npm's `package.json` 
ppl may not know about they need to put `icon`field)
 - license
 - ..   

We can improve it together in later. 




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2106: Rewrite PythonInterpreter.

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2106
  
Tested `%python.conda` and `%python.pandasql` and it works well.
Could you restore PythonDockerInterpreter.java and 
PythonDockerInterpreterTest.java for `%python.docker` as well?

To make it work, i think we need to let docker container mount directory 
that py4j exists using -v option, and then set PYTHONPATH env variable to load 
py4j library using -e option.

So this line
```java
setPythonCommand("docker run -i --rm " + image + " python -iu");
```

can be updated to something like

```java
String py4jDir = ZEPPELIN_HOME + "/interpreter/python";
String py4jFile = "py4j-{version}.zip";   // read directory py4jDir and get 
file starts with py4j
setPythonCommand("docker run -i --rm -v " + py4jDir + ":/zeppelin_python -e 
/zeppelin_python/" + py4jFile + " " + image + " python -iu");
```



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2142: [ZEPPELIN-2124] Missing dependencies array in i...

2017-03-15 Thread minahlee
GitHub user minahlee reopened a pull request:

https://github.com/apache/zeppelin/pull/2142

[ZEPPELIN-2124] Missing dependencies array in interpreter.json after 
upgrade from 0.6.2 to 0.7.0

### What is this PR for?
If there is no `dependencies` field specified in `interpreter.json`, 
front-end throws error because it tries to push new element to undefined 
variable. This PR fixes this issue by setting initial value of `dependencies` 
to empty array.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
[ZEPPELIN-2124](https://issues.apache.org/jira/browse/ZEPPELIN-2124)

### How should this be tested?
Remove `dependencies` field from `conf/interpreter.json` and try to add new 
dependencies in http://localhost:8080/#/interpreter page.

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/minahlee/zeppelin ZEPPELIN-2124

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2142.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2142


commit 01b27ebd78203f4befcc88a555671864c5d8d3f4
Author: Mina Lee 
Date:   2017-03-16T01:57:38Z

Assign init value for dependencies field




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2120: [ZEPPELIN-1492] fixing the issue where updating a para...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2120
  
Thanks for fixing bug. I will review and give you feedback 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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2105: [ZEPPELIN-2215] Progress bar for Spell execution

2017-03-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2105
  
Merge into master if there are no more comment on 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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-15 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue:

https://github.com/apache/zeppelin/pull/2134
  

![111](https://cloud.githubusercontent.com/assets/25951039/23982329/c4134daa-0a2e-11e7-94ce-09a5423c7821.png)



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2134
  
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-15 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue:

https://github.com/apache/zeppelin/pull/2134
  
ok. 

![111](https://cloud.githubusercontent.com/assets/25951039/23982093/f681cdea-0a2c-11e7-890e-5744e2fc5629.png)



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2105: [ZEPPELIN-2215] Progress bar for Spell execution

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2105
  
CI passed.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Setting zeppelin.livy.url

2017-03-15 Thread Jeff Zhang
Actually this is a bug, most of enviroment variable in interpreter-setting
doesn't work. So try to use property instead of enviroment variable.


Arun Khetarpal 于2017年3月16日周四 上午12:40写道:

Thanks Jeff.. In-general, IMO, there should be a better way to override
these settings atleast when i look at interpreter-settings.json (pasted
below the snippet)

It seems i should have, somehow been able to define the environment
variable, ZEPPELIN_LIVY_HOST_URL somewhere to override default.


Thoughts?

(Snippet of interpreter-settings.json )

{
"group": "livy",
"name": "spark",
"className": "org.apache.zeppelin.livy.LivySparkInterpreter",
"defaultInterpreter": true,
"properties": {
"zeppelin.livy.url": {
"envName": "ZEPPELIN_LIVY_HOST_URL",
"propertyName": "zeppelin.livy.url",
"defaultValue": "http://localhost:8998;,
"description": "The URL for Livy Server."
}, .

On 15 March 2017 at 20:21, Jeff Zhang  wrote:

> I have an improved version of solution b.
>
> You can copy interpreter-setting.json (with modified zeppelin.livy.url) to
> ZEPPELIN_HOME/interpreter/livy/ to override the default one.
>
>
>
> Arun Khetarpal 于2017年3月15日周三 下午10:43写道:
>
> > Hi Jeff -
> >
> > Thanks for taking out the time to help me out.
> >
> > Actually i have a case of building a managed service and would like any
> > user who spins up a cluster with zeppelin installed, to have it work
with
> > livy out of the box - since this use case also requires enterprise
> security
> > - following this doc
> > <
> > https://community.hortonworks.com/articles/80059/how-to-
> configure-zeppelin-livy-interpreter-for-sec.html
> > >
> > -
> > i would like to have zeppelin.livy.url to be set to a specific livy
> cluster
> > by default and would not like the user to go and change it later.
> >
> > Now before the zeppelin server is even up, i tried setting the value of
> > zeppelin.livy.url in zeppelin-site.xml which as you said, did not work.
> So
> > my question is, how do i override the default value without me even
going
> > via UI.
> >
> > a. Go to the UI and change the interpreter setting? - Not an option for
> my
> > use case - an external user would expect a working livy and zeppelin,
and
> > would not like to go and change the setting after i spin up a zeppelin
> > instance for him
> >
> > b. Before i spin up the zeppelin server for a specific user - I go and
> > change interpreter-settings.json by unjar'ing livy-interpreter.jar and
> > changing the default value of zeppelin.livy.url? - Obviously a hack! and
> > something i will be hanged for :(
> >
> > c. Override zeppelin.livy.url in some xml or env.sh - which would
> override
> > the default value of interpreter - *Need help on how*.
> >
> > Regards,
> > Arun
> >
> > On 15 March 2017 at 13:15, Jeff Zhang  wrote:
> >
> > > Still not clear to me, do you have multiple instances of livy server ?
> If
> > > there is only one livy server, then you just need to change it one
> time.
> > >
> > > If you have multiple livy instances, you always need to update
> > > zeppelin.livy.url no matter through interpreter setting ui or
> > > programmatically.
> > >
> > >
> > >
> > >
> > > Arun Khetarpal 于2017年3月15日周三 下午3:35写道:
> > >
> > > > The usecase we have is to create a service to spawn up the zepplin.
> Now
> > > > since i cannot override zeppelin.livy.url  and the default value is
> set
> > > to
> > > > "
> > > > https://localhost:8998; - users which are using my service need to
> > > > manually
> > > > go and change the interpreter url.
> > > >
> > > > I don't know if the scenario is clear, but i can explain more.
> > > >
> > > > Regards,
> > > > Arun
> > > >
> > > > On 15 March 2017 at 12:58, Jeff Zhang  wrote:
> > > >
> > > > > Why is it inconvenient for you ? If you have multiple livy
servers,
> > you
> > > > can
> > > > > create multiple livy interpreters for each livy server in
> interpreter
> > > > > setting page.
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > Arun Khetarpal 于2017年3月15日周三 下午1:25写道:
> > > > >
> > > > > > Is there a way to set it programmatically? I find it very
> > > inconvenient
> > > > > for
> > > > > > users to go and modify the setting in the page - is there a
> better
> > > way?
> > > > > >
> > > > > > Regards,
> > > > > > Arun
> > > > > >
> > > > > > On 15 March 2017 at 04:36, Jeff Zhang  wrote:
> > > > > >
> > > > > > > zeppelin-site.xml is for zeppelin server, not for interpreter.
> > You
> > > > > should
> > > > > > > set zeppelin.livy.url in interpreter setting page.
> > > > > > >
> > > > > > > https://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/
> > > > > > > quickstart/explorezeppelinui.html#settings
> > > > > > >
> > > > > > > Arun Khetarpal 于2017年3月15日周三 上午3:16写道:
> > > > > > >
> > > > > > > > Hi,
> > > > > > > >
> > > > > > > > I am trying to setup the livy interpreter in zeppelin.
> > > > > > > >
> > > 

[jira] [Created] (ZEPPELIN-2270) Kill livy session when creating session timeout

2017-03-15 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-2270:


 Summary: Kill livy session when creating session timeout
 Key: ZEPPELIN-2270
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2270
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.7.0
Reporter: Jeff Zhang
Assignee: Jeff Zhang






--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin issue #2100: [ZEPPELIN-2060] Make dynamic select form turn on or of...

2017-03-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2100
  
@Leemoonsoo I made `runOnSelectionChange` can be persisted under 
`note.json`'s `config` field like below.
https://cloud.githubusercontent.com/assets/10060731/23981188/fe8a5fbe-0a47-11e7-8516-ca0be81ed4dd.png;>

 If user tries to `check`/`uncheck` the option in dropdown menu, this value 
will be updated. So reloading / Zeppelin restart won't be affect to this value 
change. 
What do you think? 

And regarding 
>Do you mind improve unittest to not only verify button is checked, but 
also verify actual dynamic form behavior, in this PR or later in the other PR?

If it's okay, I'll add the test case for this changed behaviour in another 
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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2119
  
Merge into branch-0.7 and master if there are no more 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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2142: [ZEPPELIN-2124] Missing dependencies array in i...

2017-03-15 Thread minahlee
GitHub user minahlee opened a pull request:

https://github.com/apache/zeppelin/pull/2142

[ZEPPELIN-2124] Missing dependencies array in interpreter.json after 
upgrade from 0.6.2 to 0.7.0

### What is this PR for?
If there is no `dependencies` field specified in `interpreter.json`, 
front-end throws error because it tries to push new element to undefined 
variable. This PR fixes this issue by setting initial value of `dependencies` 
to empty array.

### What type of PR is it?
Bug Fix

### What is the Jira issue?
[ZEPPELIN-2124](https://issues.apache.org/jira/browse/ZEPPELIN-2124)

### How should this be tested?
Remove `dependencies` field from `conf/interpreter.json` and try to add new 
dependencies in http://localhost:8080/#/interpreter page.

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/minahlee/zeppelin ZEPPELIN-2124

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2142.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2142


commit 01b27ebd78203f4befcc88a555671864c5d8d3f4
Author: Mina Lee 
Date:   2017-03-16T01:57:38Z

Assign init value for dependencies field




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2140: [HOTFIX][ZEPPELIN-2037][ZEPPELIN-1832] "Restart" butto...

2017-03-15 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/2140
  
@cloverhearts Thanks for testing it.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2116: [ZEPPELIN-2179] "clear output" paragraph doesn't work ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2116
  
Regarding to the same fix for master branch, I will open another PR. 
Thanks for reviewing this PR @Leemoonsoo @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 so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2105: [ZEPPELIN-2215] Progress bar for Spell executio...

2017-03-15 Thread 1ambda
Github user 1ambda closed the pull request at:

https://github.com/apache/zeppelin/pull/2105


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2105: [ZEPPELIN-2215] Progress bar for Spell executio...

2017-03-15 Thread 1ambda
GitHub user 1ambda reopened a pull request:

https://github.com/apache/zeppelin/pull/2105

[ZEPPELIN-2215] Progress bar for Spell execution

### What is this PR for?

Add progress bar for spell execution.

- Used `RUNNING` state to display progress bar
- Decided not to display `{number}%` for spell, since we can't calculate in 
advance.

### What type of PR is it?
[Improvement]

### Todos

NONE

### What is the Jira issue?

[ZEPPELIN-2215](https://issues.apache.org/jira/browse/ZEPPELIN-2215)

### How should this be tested?

- Install any spell what you want. However, I suggest 
[zeppelin-echo-spell@1.0.6](https://www.npmjs.com/package/zeppelin-echo-spell) 
because it allow you to [get delayed 
result](https://github.com/1ambda/zeppelin-echo-spell/blob/master/index.js#L23-#L25)
 (default delay is 1000 millisecond.)
- Execute the spell

### Screenshots (if appropriate)


![2215](https://cloud.githubusercontent.com/assets/4968473/23655669/4f514424-0379-11e7-8557-8b8d24521a7e.gif)

### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1ambda/zeppelin 
ZEPPELIN-2215/progress-bar-for-spell

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2105.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2105


commit 79e63d44bf8ecdba583b160463678ab5602e568e
Author: 1ambda <1am...@gmail.com>
Date:   2017-03-07T11:53:28Z

feat: Don't display '%' for spell'

commit e913eff304a1fef18786b9f97912c73f10de85df
Author: 1ambda <1am...@gmail.com>
Date:   2017-03-16T01:33:19Z

fix: Remove duplicated line




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-2269) Create zeppelin-web build profile for fake production mode

2017-03-15 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2269:
---

 Summary: Create zeppelin-web build profile for fake production mode
 Key: ZEPPELIN-2269
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2269
 Project: Zeppelin
  Issue Type: Improvement
  Components: build, front-end
Affects Versions: 0.7.0
Reporter: Hoon Park
 Fix For: 0.8.0


Actually, some build process which is done in {{zeppelin-web}} for production 
is not necessary in some cases. 

For example

- we don't need to minify for CI build. 

The problem is, building zeppelin-web takes so much time considering total 
build time. we can reduce it by introducing new profile which doesn't do 
minify, ...





--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Created] (ZEPPELIN-2268) Helium package should be able to import png, jpg

2017-03-15 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2268:
---

 Summary: Helium package should be able to import png, jpg
 Key: ZEPPELIN-2268
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2268
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Hoon Park


Helium package should be able to import png, jpg for 3rd party modules. 

For example, {{amchart}} imports png and jpg. 

- https://github.com/amcharts/amcharts3



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Created] (ZEPPELIN-2266) Treat helium package description as HTML

2017-03-15 Thread Hoon Park (JIRA)
Hoon Park created ZEPPELIN-2266:
---

 Summary: Treat helium package description as HTML
 Key: ZEPPELIN-2266
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2266
 Project: Zeppelin
  Issue Type: Improvement
  Components: front-end, helium
Reporter: Hoon Park


Treat helium package description as HTML



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin issue #2141: [ZEPPELIN-1720] Adding tests to verify behaviour of dy...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2141
  
Hi @randerzander. Thanks for contribution.

- Zeppelin build system relies on travis. So please setup your travis 
account and travis project for Zeppelin. 
- 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
- The repository name must be `zeppelin` (not `incubator-zeppelin`, ...)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2119
  
CI is green :) 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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2116: [ZEPPELIN-2179] "clear output" paragraph doesn't work ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2116
  
CI passed. @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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2119: [ZEPPELIN-2243] Use uppercase for shortcut desc...

2017-03-15 Thread 1ambda
Github user 1ambda closed the pull request at:

https://github.com/apache/zeppelin/pull/2119


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2119: [ZEPPELIN-2243] Use uppercase for shortcut desc...

2017-03-15 Thread 1ambda
GitHub user 1ambda reopened a pull request:

https://github.com/apache/zeppelin/pull/2119

[ZEPPELIN-2243] Use uppercase for shortcut description

### What is this PR for?

Use uppercase for keyboard shortcut description so that user can recognize 
shortcuts easily.
it's hard to understand for some shortcuts since we used lowercases. For 
example, `Ctrl+Option+l` (actually this `L`, not `I`)

I attached screenshots.

### What type of PR is it?
[Improvement]

### Todos

NONE

### What is the Jira issue?

[ZEPPELIN-2243](https://issues.apache.org/jira/browse/ZEPPELIN-2243)

### How should this be tested?

1. Open the paragraph shortcut panel.
2. Open the note shortcut dialog.

### Screenshots (if appropriate)

 Before

https://cloud.githubusercontent.com/assets/4968473/23786191/fb3dafe8-05ae-11e7-8e1e-36b5f151070f.png;>

 After

https://cloud.githubusercontent.com/assets/4968473/23786194/fe59b1e0-05ae-11e7-9741-371f8e3f77de.png;>

### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/1ambda/zeppelin 
ZEPPELIN-2243/use-uppercase-for-keyboard-shortcut-description

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2119.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2119


commit 42021da53093edcc699b69f1ad73355c9a215d2c
Author: 1ambda <1am...@gmail.com>
Date:   2017-03-10T07:24:12Z

fix: Use uppercase for shortcut desc

commit dc933cc47e3b733727f2c72b73da05af39f52baa
Author: 1ambda <1am...@gmail.com>
Date:   2017-03-10T08:47:03Z

fix: ParagraphActionsIT

commit 1cdb36a1f70a55bdb34f12ade9b83306d139d275
Author: 1ambda <1am...@gmail.com>
Date:   2017-03-15T20:00:20Z

fix: Remove white spaces




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2100: [ZEPPELIN-2060] Make dynamic select form turn on or of...

2017-03-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2100
  
@Leemoonsoo Didn't realize that. Thanks for letting me know.  Let me update 
again.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-2265) Zeppelin notebook output adding commas for thousand separator

2017-03-15 Thread Andrei Stankevich (JIRA)
Andrei Stankevich created ZEPPELIN-2265:
---

 Summary: Zeppelin notebook output adding commas for thousand 
separator
 Key: ZEPPELIN-2265
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2265
 Project: Zeppelin
  Issue Type: Wish
  Components: front-end
Affects Versions: 0.6.2
Reporter: Andrei Stankevich
Priority: Minor


Sorry in advance if it is not a right place to ask this question.
When i am trying to show dataframe on Zeppelin UI and i am doing 
z.show(dataframe) 
if my dataframe has column which has type long, Zeppelin adds comma separator 
after each thousand. 
Example 
if my value is 1232424
Zeppelin shows it like this 1,232,424

Could you please point me to right place in code where Zeppelin adds these 
commas? I need to remove them but didn't find in source code were Zeppelin adds 
them.
Thank you so much.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin issue #2141: [ZEPPELIN-1720] Adding tests to verify behaviour of dy...

2017-03-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2141
  
Cool!


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2126: [gh-pages][ZEPPELIN-2209, 2110] Show 'INTERPRETER' typ...

2017-03-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2126
  
Thanks again @1ambda. 
Will merge into gh-pages if there are no more 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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2114: [ZEPPELIN-2234][BUG] Can't display the same chart agai...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2114
  
@1ambda Thanks! LGTM and merge to branch-0.7 if no further discussions.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2137
  
FYI, @NohSeho


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2114: [ZEPPELIN-2234][BUG] Can't display the same chart agai...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2114
  
Rebased and fixed using the implementation discussed in #2110 

- i didn't bring refactoring commits because 0.7.0-branch is not actively 
developed 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 project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-2264) DevInterpreter settings references JDBC driver.

2017-03-15 Thread Simeon H.K. Fitch (JIRA)
Simeon H.K. Fitch created ZEPPELIN-2264:
---

 Summary: DevInterpreter settings references JDBC driver.
 Key: ZEPPELIN-2264
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2264
 Project: Zeppelin
  Issue Type: Task
  Components: helium
Affects Versions: 0.7.0
Reporter: Simeon H.K. Fitch
Priority: Minor


In the code for the dev interpreter, [this 
file](https://github.com/apache/zeppelin/blob/master/helium-dev/src/main/resources/interpreter-setting.json)
 references a JDBC driver under the `port` property. There doesn't seem to be 
any JDBC code in the dev interpreter. Is this a copy/paste accident? It's 
certainly confusing when trying to use the DevInterpreter for the first time.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin issue #2125: [ZEPPELIN-2222] Add Network Visualization

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2125
  
I will test and give you feedback 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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2105: [ZEPPELIN-2215] Progress bar for Spell execution

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2105
  
I'v just rebased to resolve conflict.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2126: [gh-pages][ZEPPELIN-2209, 2110] Show 'INTERPRETER' typ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2126
  
Looks nice. @AhyoungRyu


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2115: [ZEPPELIN-2202] Disable personalized mode btn when not...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2115
  
@Leemoonsoo 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 so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2115: [ZEPPELIN-2202] Disable personalized mode btn w...

2017-03-15 Thread 1ambda
Github user 1ambda closed the pull request at:

https://github.com/apache/zeppelin/pull/2115


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2116: [ZEPPELIN-2179] "clear output" paragraph doesn't work ...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2116
  
I'v just rebased, let me comment after fixing 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 if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2119: [ZEPPELIN-2243] Use uppercase for shortcut description

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2119
  
Rebased to resolve @felixcheung's the last 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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2137
  
You can set `ZEPPELIN_HELIUM_REGISTRY` environment variable in 
`conf/zeppelin-env.sh` or `zeppelin.helium.registry` property in 
`conf/zeppelin-site.xml`.

[Default 
value](https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java#L629)
 is `helium,https://s3.amazonaws.com/helium-package/helium.json`, (a local dir 
and a remote registry).
If you change this value to `helium`, then Zeppelin will check only the 
local directory.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-15 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2134
  
Sorry for late reply. 

- regarding to design, it would be great if we use bootstrap-alert. 
(`warning` level would be ok)

For example,


![image](https://cloud.githubusercontent.com/assets/4968473/23967914/1d37a49c-0a04-11e7-8cbf-a11ac00aa368.png)



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2140: [HOTFIX][ZEPPELIN-2037][ZEPPELIN-1832] "Restart" butto...

2017-03-15 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2140
  
@jongyoul Tested.

Spark and Markdown and Misc interpreter (without shiro)
```
restart interpreter on web ui > restart feature well work.
zeppelin shutdown (daemon stop) --->  well working, I can't reproduce with 
zombie process. :)
```
Thank you for good fix :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread FRosner
Github user FRosner commented on the issue:

https://github.com/apache/zeppelin/pull/2137
  
Ok @Leemoonsoo. So what does this PR actually do if it still waits 2 
minutes now at a different stage? Or can I somehow prevent Zeppelin from 
getting the available helium package list?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Setting zeppelin.livy.url

2017-03-15 Thread Arun Khetarpal
Thanks Jeff.. In-general, IMO, there should be a better way to override
these settings atleast when i look at interpreter-settings.json (pasted
below the snippet)

It seems i should have, somehow been able to define the environment
variable, ZEPPELIN_LIVY_HOST_URL somewhere to override default.


Thoughts?

(Snippet of interpreter-settings.json )

{
"group": "livy",
"name": "spark",
"className": "org.apache.zeppelin.livy.LivySparkInterpreter",
"defaultInterpreter": true,
"properties": {
"zeppelin.livy.url": {
"envName": "ZEPPELIN_LIVY_HOST_URL",
"propertyName": "zeppelin.livy.url",
"defaultValue": "http://localhost:8998;,
"description": "The URL for Livy Server."
}, .

On 15 March 2017 at 20:21, Jeff Zhang  wrote:

> I have an improved version of solution b.
>
> You can copy interpreter-setting.json (with modified zeppelin.livy.url) to
> ZEPPELIN_HOME/interpreter/livy/ to override the default one.
>
>
>
> Arun Khetarpal 于2017年3月15日周三 下午10:43写道:
>
> > Hi Jeff -
> >
> > Thanks for taking out the time to help me out.
> >
> > Actually i have a case of building a managed service and would like any
> > user who spins up a cluster with zeppelin installed, to have it work with
> > livy out of the box - since this use case also requires enterprise
> security
> > - following this doc
> > <
> > https://community.hortonworks.com/articles/80059/how-to-
> configure-zeppelin-livy-interpreter-for-sec.html
> > >
> > -
> > i would like to have zeppelin.livy.url to be set to a specific livy
> cluster
> > by default and would not like the user to go and change it later.
> >
> > Now before the zeppelin server is even up, i tried setting the value of
> > zeppelin.livy.url in zeppelin-site.xml which as you said, did not work.
> So
> > my question is, how do i override the default value without me even going
> > via UI.
> >
> > a. Go to the UI and change the interpreter setting? - Not an option for
> my
> > use case - an external user would expect a working livy and zeppelin, and
> > would not like to go and change the setting after i spin up a zeppelin
> > instance for him
> >
> > b. Before i spin up the zeppelin server for a specific user - I go and
> > change interpreter-settings.json by unjar'ing livy-interpreter.jar and
> > changing the default value of zeppelin.livy.url? - Obviously a hack! and
> > something i will be hanged for :(
> >
> > c. Override zeppelin.livy.url in some xml or env.sh - which would
> override
> > the default value of interpreter - *Need help on how*.
> >
> > Regards,
> > Arun
> >
> > On 15 March 2017 at 13:15, Jeff Zhang  wrote:
> >
> > > Still not clear to me, do you have multiple instances of livy server ?
> If
> > > there is only one livy server, then you just need to change it one
> time.
> > >
> > > If you have multiple livy instances, you always need to update
> > > zeppelin.livy.url no matter through interpreter setting ui or
> > > programmatically.
> > >
> > >
> > >
> > >
> > > Arun Khetarpal 于2017年3月15日周三 下午3:35写道:
> > >
> > > > The usecase we have is to create a service to spawn up the zepplin.
> Now
> > > > since i cannot override zeppelin.livy.url  and the default value is
> set
> > > to
> > > > "
> > > > https://localhost:8998; - users which are using my service need to
> > > > manually
> > > > go and change the interpreter url.
> > > >
> > > > I don't know if the scenario is clear, but i can explain more.
> > > >
> > > > Regards,
> > > > Arun
> > > >
> > > > On 15 March 2017 at 12:58, Jeff Zhang  wrote:
> > > >
> > > > > Why is it inconvenient for you ? If you have multiple livy servers,
> > you
> > > > can
> > > > > create multiple livy interpreters for each livy server in
> interpreter
> > > > > setting page.
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > Arun Khetarpal 于2017年3月15日周三 下午1:25写道:
> > > > >
> > > > > > Is there a way to set it programmatically? I find it very
> > > inconvenient
> > > > > for
> > > > > > users to go and modify the setting in the page - is there a
> better
> > > way?
> > > > > >
> > > > > > Regards,
> > > > > > Arun
> > > > > >
> > > > > > On 15 March 2017 at 04:36, Jeff Zhang  wrote:
> > > > > >
> > > > > > > zeppelin-site.xml is for zeppelin server, not for interpreter.
> > You
> > > > > should
> > > > > > > set zeppelin.livy.url in interpreter setting page.
> > > > > > >
> > > > > > > https://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/
> > > > > > > quickstart/explorezeppelinui.html#settings
> > > > > > >
> > > > > > > Arun Khetarpal 于2017年3月15日周三 上午3:16写道:
> > > > > > >
> > > > > > > > Hi,
> > > > > > > >
> > > > > > > > I am trying to setup the livy interpreter in zeppelin.
> > > > > > > >
> > > > > > > > For this, i have set zeppelin.livy.url property in
> > > > zeppelin-site.xml
> > > > > -
> > > > > > > but
> > > > > > > > in Zeppelin ui, the property of interpreter is still set to
> > > > 

[GitHub] zeppelin pull request #2141: [ZEPPELIN-1720] Adding tests to verify behaviou...

2017-03-15 Thread guicaro
GitHub user guicaro opened a pull request:

https://github.com/apache/zeppelin/pull/2141

[ZEPPELIN-1720] Adding tests to verify behaviour of dynamic forms

### What is this PR for?
Adding Selenium tests to ensure proper behaviour of dynamic forms. 

### What type of PR is it?
Test

### Todos
N/A

### What is the Jira issue?
[ZEPPELIN-1720](https://issues.apache.org/jira/browse/ZEPPELIN-1720)

### How should this be tested?

1. Once should first get Firefox v. 41 as it is the latest version that 
works with the current version of Selenium in Apache Zeppelin.
2. You can then run the tests with following command:

`TEST_SELENIUM="true" mvn package -DfailIfNoTests=false -pl 
'zeppelin-interpreter,zeppelin-zengine,zeppelin-server' 
-Dtest=ParagraphActionsIT
`
### Screenshots (if appropriate)
N/A

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/guicaro/zeppelin 
ZEPPELIN-1720-AddingTestsForDynamicForms

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2141.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2141


commit 17ca94208f5b4553e3cc2a93682b6baa0f626457
Author: Guillermo Cabrera 
Date:   2017-03-13T16:27:22Z

Added method stubbs for new UI tests checking correct behaviour of dynamic 
forms

commit e6bcd78edf9197b13c0ec12e7e03712d895bc4c9
Author: Guillermo Cabrera 
Date:   2017-03-14T12:50:01Z

Completed and verified corrct behaviour of testSingleDynamicFormTextInput

commit 5507be53894622329208cac8511103f2031fc6a8
Author: Guillermo Cabrera 
Date:   2017-03-14T16:50:26Z

Added tests that cover behaviour of dynamic forms

commit ce0823db0d3189569131208ab16a0ff8751d0762
Author: Guillermo Cabrera 
Date:   2017-03-15T15:23:57Z

Removed unused import, alignment and removed unnecesary condition in 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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2100: [ZEPPELIN-2060] Make dynamic select form turn on or of...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2100
  
Ah i realized that "Run on selection change" is not persisted, so 
refreshing browser resets the option. @AhyoungRyu Could you take 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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-2263) Behaviour of checkbox form inconsistent with select or text input form

2017-03-15 Thread Guillermo Cabrera (JIRA)
Guillermo Cabrera created ZEPPELIN-2263:
---

 Summary: Behaviour of checkbox form inconsistent with select or 
text input form
 Key: ZEPPELIN-2263
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2263
 Project: Zeppelin
  Issue Type: Improvement
  Components: GUI
Affects Versions: 0.8.0
 Environment: Ubuntu 16.04 + Firefox v. 41 (Needed for compatibility w/ 
Selenium)
Reporter: Guillermo Cabrera
Priority: Trivial


When one creates a check box programmatically the default behavior is that all 
check boxes created are checked [1]. This is not consistent with the other 
dynamic forms (ex. text input and select) in which nothing is selected when 
they are created.

If one creates a check box via a template then one can specify which check 
boxes should be checked by default. As a fix maybe we can just set the check 
boxes to be unselected as a default behavior (when created programmatically)?

[1] 
https://github.com/apache/zeppelin/blob/master/spark/src/main/java/org/apache/zeppelin/spark/ZeppelinContext.java#L151



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2134
  
Merge to master and branch-0.7 if no further discussions.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2132: [ZEPPELIN-2253] Piechart won't render when colu...

2017-03-15 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2132


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2128: [WIP] [Discuss] Make use of all grouped data to...

2017-03-15 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2128


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2100: [ZEPPELIN-2060] Make dynamic select form turn on or of...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2100
  
LGTM.

@AhyoungRyu Do you mind improve unittest to not only verify button is 
checked, but also verify actual dynamic form behavior, in this PR or later in 
the other 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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2117: ZEPPELIN-2241: JDBC interpreter throws npe on c...

2017-03-15 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2117


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2137
  
Thanks @FRosner reviewing the PR.

> ```  INFO [2017-03-15 10:41:22,547] ({main} Helium.java[loadConf]:114) - 
Add helium local registry /usr/local/zeppelin/helium
>  INFO [2017-03-15 10:41:22,548] ({main} Helium.java[loadConf]:111) - Add 
helium online registry https://s3.amazonaws.com/helium-package/helium.json
> WARN [2017-03-15 10:41:22,552] ({main} Helium.java[loadConf]:122) - 
/usr/local/zeppelin/conf/helium.json does not exists
> ERROR [2017-03-15 10:43:30,270] ({main} 
HeliumOnlineRegistry.java[getAll]:73) - Connect to s3.amazonaws.com:443 
[s3.amazonaws.com/52.216.82.43] failed: Connection timed out (Connection timed 
out)
> ```

Zeppelin tries to download available helium package list (which is filtered 
list of npm and maven registry). The new error is related with this process. 
This is a bit different issue and let me try solve it in different 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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2106: [WIP] Rewrite PythonInterpreter.

2017-03-15 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/2106
  
@Leemoonsoo fixed the issues you commented and added PandasSql. 
and also added `conda`, `pandasql` interpreter test on PR description.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Setting zeppelin.livy.url

2017-03-15 Thread Jeff Zhang
I have an improved version of solution b.

You can copy interpreter-setting.json (with modified zeppelin.livy.url) to
ZEPPELIN_HOME/interpreter/livy/ to override the default one.



Arun Khetarpal 于2017年3月15日周三 下午10:43写道:

> Hi Jeff -
>
> Thanks for taking out the time to help me out.
>
> Actually i have a case of building a managed service and would like any
> user who spins up a cluster with zeppelin installed, to have it work with
> livy out of the box - since this use case also requires enterprise security
> - following this doc
> <
> https://community.hortonworks.com/articles/80059/how-to-configure-zeppelin-livy-interpreter-for-sec.html
> >
> -
> i would like to have zeppelin.livy.url to be set to a specific livy cluster
> by default and would not like the user to go and change it later.
>
> Now before the zeppelin server is even up, i tried setting the value of
> zeppelin.livy.url in zeppelin-site.xml which as you said, did not work. So
> my question is, how do i override the default value without me even going
> via UI.
>
> a. Go to the UI and change the interpreter setting? - Not an option for my
> use case - an external user would expect a working livy and zeppelin, and
> would not like to go and change the setting after i spin up a zeppelin
> instance for him
>
> b. Before i spin up the zeppelin server for a specific user - I go and
> change interpreter-settings.json by unjar'ing livy-interpreter.jar and
> changing the default value of zeppelin.livy.url? - Obviously a hack! and
> something i will be hanged for :(
>
> c. Override zeppelin.livy.url in some xml or env.sh - which would override
> the default value of interpreter - *Need help on how*.
>
> Regards,
> Arun
>
> On 15 March 2017 at 13:15, Jeff Zhang  wrote:
>
> > Still not clear to me, do you have multiple instances of livy server ? If
> > there is only one livy server, then you just need to change it one time.
> >
> > If you have multiple livy instances, you always need to update
> > zeppelin.livy.url no matter through interpreter setting ui or
> > programmatically.
> >
> >
> >
> >
> > Arun Khetarpal 于2017年3月15日周三 下午3:35写道:
> >
> > > The usecase we have is to create a service to spawn up the zepplin. Now
> > > since i cannot override zeppelin.livy.url  and the default value is set
> > to
> > > "
> > > https://localhost:8998; - users which are using my service need to
> > > manually
> > > go and change the interpreter url.
> > >
> > > I don't know if the scenario is clear, but i can explain more.
> > >
> > > Regards,
> > > Arun
> > >
> > > On 15 March 2017 at 12:58, Jeff Zhang  wrote:
> > >
> > > > Why is it inconvenient for you ? If you have multiple livy servers,
> you
> > > can
> > > > create multiple livy interpreters for each livy server in interpreter
> > > > setting page.
> > > >
> > > >
> > > >
> > > >
> > > > Arun Khetarpal 于2017年3月15日周三 下午1:25写道:
> > > >
> > > > > Is there a way to set it programmatically? I find it very
> > inconvenient
> > > > for
> > > > > users to go and modify the setting in the page - is there a better
> > way?
> > > > >
> > > > > Regards,
> > > > > Arun
> > > > >
> > > > > On 15 March 2017 at 04:36, Jeff Zhang  wrote:
> > > > >
> > > > > > zeppelin-site.xml is for zeppelin server, not for interpreter.
> You
> > > > should
> > > > > > set zeppelin.livy.url in interpreter setting page.
> > > > > >
> > > > > > https://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/
> > > > > > quickstart/explorezeppelinui.html#settings
> > > > > >
> > > > > > Arun Khetarpal 于2017年3月15日周三 上午3:16写道:
> > > > > >
> > > > > > > Hi,
> > > > > > >
> > > > > > > I am trying to setup the livy interpreter in zeppelin.
> > > > > > >
> > > > > > > For this, i have set zeppelin.livy.url property in
> > > zeppelin-site.xml
> > > > -
> > > > > > but
> > > > > > > in Zeppelin ui, the property of interpreter is still set to
> > > default.
> > > > > > >
> > > > > > > Alternatively, i tried also setting it zeppelin-env.sh as
> export
> > > > result
> > > > > > > ZEPPELIN_LIVY_HOST_URL  but the same result.
> > > > > > >
> > > > > > > Any help?
> > > > > > >
> > > > > > > Regards,
> > > > > > > Arun
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>


Re: Setting zeppelin.livy.url

2017-03-15 Thread Arun Khetarpal
Hi Jeff -

Thanks for taking out the time to help me out.

Actually i have a case of building a managed service and would like any
user who spins up a cluster with zeppelin installed, to have it work with
livy out of the box - since this use case also requires enterprise security
- following this doc

-
i would like to have zeppelin.livy.url to be set to a specific livy cluster
by default and would not like the user to go and change it later.

Now before the zeppelin server is even up, i tried setting the value of
zeppelin.livy.url in zeppelin-site.xml which as you said, did not work. So
my question is, how do i override the default value without me even going
via UI.

a. Go to the UI and change the interpreter setting? - Not an option for my
use case - an external user would expect a working livy and zeppelin, and
would not like to go and change the setting after i spin up a zeppelin
instance for him

b. Before i spin up the zeppelin server for a specific user - I go and
change interpreter-settings.json by unjar'ing livy-interpreter.jar and
changing the default value of zeppelin.livy.url? - Obviously a hack! and
something i will be hanged for :(

c. Override zeppelin.livy.url in some xml or env.sh - which would override
the default value of interpreter - *Need help on how*.

Regards,
Arun

On 15 March 2017 at 13:15, Jeff Zhang  wrote:

> Still not clear to me, do you have multiple instances of livy server ? If
> there is only one livy server, then you just need to change it one time.
>
> If you have multiple livy instances, you always need to update
> zeppelin.livy.url no matter through interpreter setting ui or
> programmatically.
>
>
>
>
> Arun Khetarpal 于2017年3月15日周三 下午3:35写道:
>
> > The usecase we have is to create a service to spawn up the zepplin. Now
> > since i cannot override zeppelin.livy.url  and the default value is set
> to
> > "
> > https://localhost:8998; - users which are using my service need to
> > manually
> > go and change the interpreter url.
> >
> > I don't know if the scenario is clear, but i can explain more.
> >
> > Regards,
> > Arun
> >
> > On 15 March 2017 at 12:58, Jeff Zhang  wrote:
> >
> > > Why is it inconvenient for you ? If you have multiple livy servers, you
> > can
> > > create multiple livy interpreters for each livy server in interpreter
> > > setting page.
> > >
> > >
> > >
> > >
> > > Arun Khetarpal 于2017年3月15日周三 下午1:25写道:
> > >
> > > > Is there a way to set it programmatically? I find it very
> inconvenient
> > > for
> > > > users to go and modify the setting in the page - is there a better
> way?
> > > >
> > > > Regards,
> > > > Arun
> > > >
> > > > On 15 March 2017 at 04:36, Jeff Zhang  wrote:
> > > >
> > > > > zeppelin-site.xml is for zeppelin server, not for interpreter. You
> > > should
> > > > > set zeppelin.livy.url in interpreter setting page.
> > > > >
> > > > > https://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/
> > > > > quickstart/explorezeppelinui.html#settings
> > > > >
> > > > > Arun Khetarpal 于2017年3月15日周三 上午3:16写道:
> > > > >
> > > > > > Hi,
> > > > > >
> > > > > > I am trying to setup the livy interpreter in zeppelin.
> > > > > >
> > > > > > For this, i have set zeppelin.livy.url property in
> > zeppelin-site.xml
> > > -
> > > > > but
> > > > > > in Zeppelin ui, the property of interpreter is still set to
> > default.
> > > > > >
> > > > > > Alternatively, i tried also setting it zeppelin-env.sh as export
> > > result
> > > > > > ZEPPELIN_LIVY_HOST_URL  but the same result.
> > > > > >
> > > > > > Any help?
> > > > > >
> > > > > > Regards,
> > > > > > Arun
> > > > > >
> > > > >
> > > >
> > >
> >
>


[jira] [Created] (ZEPPELIN-2262) REST Api for configuring 'Interpreter for note' as Isolated, Scoped or Shared

2017-03-15 Thread David Barnett (JIRA)
David Barnett created ZEPPELIN-2262:
---

 Summary: REST Api for configuring 'Interpreter for note' as 
Isolated, Scoped or Shared
 Key: ZEPPELIN-2262
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2262
 Project: Zeppelin
  Issue Type: New Feature
Reporter: David Barnett


As a team frequently collaborating on spark using Zeppelin we would like to 
programmatically start a cluster to be used by the entire team. We would like 
to be able to programmatically set the spark interpreter to run in scoped mode. 
As far as I have been able to tell the current REST api doesn't support this as 
a configuration option.

Thank you for your consideration.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin pull request #2140: [HOTFIX][ZEPPELIN-2037][ZEPPELIN-1832] "Restart...

2017-03-15 Thread jongyoul
GitHub user jongyoul opened a pull request:

https://github.com/apache/zeppelin/pull/2140

[HOTFIX][ZEPPELIN-2037][ZEPPELIN-1832] "Restart" button does not work

### What is this PR for?
Fixing restarting interpreters work correctly. All restart buttons runs 
restarting only user's interpreter instance including "scoped" and "isolated". 
If you shutdown the server, Zeppelin terminates all interpreters' processes

### What type of PR is it?
[Bug Fix | Hot Fix]

### Todos
* [ ] - Make "Restart" button work properly 

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-2037
* https://issues.apache.org/jira/browse/ZEPPELIN-1832

### How should this be tested?
1. Enable shiro
1. Login with "admin"
1. Set "Per user" to "scoped"
1. Run "sc.version" in note1 with "admin"
1. Login with "user1"
1. Run "sc.version" in note1 with "user1"
1. Click the "restart" button in note1 page with "admin"
1. Check the process with 'ps aux | grep RemoteInterpreterServer'. Will 
find one process
1. Click the "restart" button in note1 page with "user1"
1. Check the process with 'ps aux | grep RemoteInterpreterServer'. Won't 
find any process

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jongyoul/zeppelin ZEPPELIN-2037

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2140.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2140


commit 49dbf65864351b142305a3845899e5f5bbab853e
Author: Jongyoul Lee 
Date:   2017-03-12T13:55:26Z

WIP
Added unit test for all modes
Fixed dereference bug

commit bb4019d96c2cce90f7c8b10db4aac9a02c3b3cff
Author: Jongyoul Lee 
Date:   2017-03-15T13:43:48Z

Changed the logic of closing interpreter
Changed closing logic of lazyinterpreter to synchronous execution to 
guarantee the order of execution




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread FRosner
Github user FRosner commented on the issue:

https://github.com/apache/zeppelin/pull/2137
  
Hi @Leemoonsoo,

I built your branch 
(https://travis-ci.org/FRosner/docker-zeppelin/builds/211271444) and tried it. 
However, it still waits for 2 minutes, now with a different 'error':

```
 INFO [2017-03-15 10:41:22,547] ({main} Helium.java[loadConf]:114) - Add 
helium local registry /usr/local/zeppelin/helium
 INFO [2017-03-15 10:41:22,548] ({main} Helium.java[loadConf]:111) - Add 
helium online registry https://s3.amazonaws.com/helium-package/helium.json
 WARN [2017-03-15 10:41:22,552] ({main} Helium.java[loadConf]:122) - 
/usr/local/zeppelin/conf/helium.json does not exists
ERROR [2017-03-15 10:43:30,270] ({main} 
HeliumOnlineRegistry.java[getAll]:73) - Connect to s3.amazonaws.com:443 
[s3.amazonaws.com/52.216.82.43] failed: Connection timed out (Connection timed 
out)
 ```

For your convenience, the previous 'error' was:

```
INFO [2017-03-07 12:52:34,068] ({main} NPMInstaller.java[installNpm]:117) - 
Installing npm version 3.10.8
 INFO [2017-03-07 12:52:34,070] ({main} 
NPMInstaller.java[downloadFile]:198) - Downloading 
http://registry.npmjs.org/npm/-/npm-3.10.8.tgz to 
/usr/local/zeppelin/local-repo/vis/cache/npm-3.10.8.tar.gz
 INFO [2017-03-07 12:52:34,074] ({main} 
ProxyConfig.java[getProxyForUrl]:27) - No proxies configured
 INFO [2017-03-07 12:52:34,074] ({main} FileDownloader.java[execute]:91) - 
No proxy was configured, downloading directly
ERROR [2017-03-07 12:54:41,887] ({main} 
HeliumVisualizationFactory.java[installNodeAndNpm]:92) - Could not download npm
com.github.eirslett.maven.plugins.frontend.lib.InstallationException: Could 
not download npm
```

What is the way to "not select a helium package" so it does not try to 
download? What am I doing wrong?

Thanks
Frank



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2100: [ZEPPELIN-2060] Make dynamic select form turn on or of...

2017-03-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2100
  
CI is also green now :D


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2099: [ZEPPELIN-2106] providing paragraph config in create n...

2017-03-15 Thread Remilito
Github user Remilito commented on the issue:

https://github.com/apache/zeppelin/pull/2099
  
@Leemoonsoo : the most common usages would be to set visual paramaters, 
i.e. graph options, paragraph width and paragraph title show, so the idea was 
to provide a more direct way to configure those rather than having to provide 
the whole config:{results:[{}  to do so. Anyway, this is open to discussions


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2085: [ZEPPELIN-1999] get interpreter property with replaced...

2017-03-15 Thread dwhsys
Github user dwhsys commented on the issue:

https://github.com/apache/zeppelin/pull/2085
  
I think that this PR is a great step forward to enterprise version of 
Zeppelin. Using this feature there will be an ability to implement native 
integration between different interpreters based on single schema per user / 
note in core analytical DBMS. Another feature is the possibility of data access 
control without using credentials.

@felixcheung, @Leemoonsoo, what do you think?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2137: [ZEPPELIN-2260] Skip node,npm install and bundle when ...

2017-03-15 Thread FRosner
Github user FRosner commented on the issue:

https://github.com/apache/zeppelin/pull/2137
  
LGTM. I'm gonna build it today and give it a shot. I'll let you know how it 
went :)

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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Setting zeppelin.livy.url

2017-03-15 Thread Jeff Zhang
Still not clear to me, do you have multiple instances of livy server ? If
there is only one livy server, then you just need to change it one time.

If you have multiple livy instances, you always need to update
zeppelin.livy.url no matter through interpreter setting ui or
programmatically.




Arun Khetarpal 于2017年3月15日周三 下午3:35写道:

> The usecase we have is to create a service to spawn up the zepplin. Now
> since i cannot override zeppelin.livy.url  and the default value is set to
> "
> https://localhost:8998; - users which are using my service need to
> manually
> go and change the interpreter url.
>
> I don't know if the scenario is clear, but i can explain more.
>
> Regards,
> Arun
>
> On 15 March 2017 at 12:58, Jeff Zhang  wrote:
>
> > Why is it inconvenient for you ? If you have multiple livy servers, you
> can
> > create multiple livy interpreters for each livy server in interpreter
> > setting page.
> >
> >
> >
> >
> > Arun Khetarpal 于2017年3月15日周三 下午1:25写道:
> >
> > > Is there a way to set it programmatically? I find it very inconvenient
> > for
> > > users to go and modify the setting in the page - is there a better way?
> > >
> > > Regards,
> > > Arun
> > >
> > > On 15 March 2017 at 04:36, Jeff Zhang  wrote:
> > >
> > > > zeppelin-site.xml is for zeppelin server, not for interpreter. You
> > should
> > > > set zeppelin.livy.url in interpreter setting page.
> > > >
> > > > https://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/
> > > > quickstart/explorezeppelinui.html#settings
> > > >
> > > > Arun Khetarpal 于2017年3月15日周三 上午3:16写道:
> > > >
> > > > > Hi,
> > > > >
> > > > > I am trying to setup the livy interpreter in zeppelin.
> > > > >
> > > > > For this, i have set zeppelin.livy.url property in
> zeppelin-site.xml
> > -
> > > > but
> > > > > in Zeppelin ui, the property of interpreter is still set to
> default.
> > > > >
> > > > > Alternatively, i tried also setting it zeppelin-env.sh as export
> > result
> > > > > ZEPPELIN_LIVY_HOST_URL  but the same result.
> > > > >
> > > > > Any help?
> > > > >
> > > > > Regards,
> > > > > Arun
> > > > >
> > > >
> > >
> >
>


[GitHub] zeppelin issue #2133: [ZEPPELIN-1357][Umbrella] Text overlap in the MultiBar...

2017-03-15 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2133
  
I think the following JIRA issues are similar with this PR
* [ZEPPELIN-1892; Display label vertically or horizontally 
smartly](https://issues.apache.org/jira/browse/ZEPPELIN/ZEPPELIN-1892)
* [ZEPPELIN-1370; Label overlaps - in default visualization example 
barchart](https://issues.apache.org/jira/browse/ZEPPELIN-1370)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Setting zeppelin.livy.url

2017-03-15 Thread Arun Khetarpal
The usecase we have is to create a service to spawn up the zepplin. Now
since i cannot override zeppelin.livy.url  and the default value is set to "
https://localhost:8998; - users which are using my service need to manually
go and change the interpreter url.

I don't know if the scenario is clear, but i can explain more.

Regards,
Arun

On 15 March 2017 at 12:58, Jeff Zhang  wrote:

> Why is it inconvenient for you ? If you have multiple livy servers, you can
> create multiple livy interpreters for each livy server in interpreter
> setting page.
>
>
>
>
> Arun Khetarpal 于2017年3月15日周三 下午1:25写道:
>
> > Is there a way to set it programmatically? I find it very inconvenient
> for
> > users to go and modify the setting in the page - is there a better way?
> >
> > Regards,
> > Arun
> >
> > On 15 March 2017 at 04:36, Jeff Zhang  wrote:
> >
> > > zeppelin-site.xml is for zeppelin server, not for interpreter. You
> should
> > > set zeppelin.livy.url in interpreter setting page.
> > >
> > > https://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/
> > > quickstart/explorezeppelinui.html#settings
> > >
> > > Arun Khetarpal 于2017年3月15日周三 上午3:16写道:
> > >
> > > > Hi,
> > > >
> > > > I am trying to setup the livy interpreter in zeppelin.
> > > >
> > > > For this, i have set zeppelin.livy.url property in zeppelin-site.xml
> -
> > > but
> > > > in Zeppelin ui, the property of interpreter is still set to default.
> > > >
> > > > Alternatively, i tried also setting it zeppelin-env.sh as export
> result
> > > > ZEPPELIN_LIVY_HOST_URL  but the same result.
> > > >
> > > > Any help?
> > > >
> > > > Regards,
> > > > Arun
> > > >
> > >
> >
>


Re: Setting zeppelin.livy.url

2017-03-15 Thread Jeff Zhang
Why is it inconvenient for you ? If you have multiple livy servers, you can
create multiple livy interpreters for each livy server in interpreter
setting page.




Arun Khetarpal 于2017年3月15日周三 下午1:25写道:

> Is there a way to set it programmatically? I find it very inconvenient for
> users to go and modify the setting in the page - is there a better way?
>
> Regards,
> Arun
>
> On 15 March 2017 at 04:36, Jeff Zhang  wrote:
>
> > zeppelin-site.xml is for zeppelin server, not for interpreter. You should
> > set zeppelin.livy.url in interpreter setting page.
> >
> > https://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/
> > quickstart/explorezeppelinui.html#settings
> >
> > Arun Khetarpal 于2017年3月15日周三 上午3:16写道:
> >
> > > Hi,
> > >
> > > I am trying to setup the livy interpreter in zeppelin.
> > >
> > > For this, i have set zeppelin.livy.url property in zeppelin-site.xml -
> > but
> > > in Zeppelin ui, the property of interpreter is still set to default.
> > >
> > > Alternatively, i tried also setting it zeppelin-env.sh as export result
> > > ZEPPELIN_LIVY_HOST_URL  but the same result.
> > >
> > > Any help?
> > >
> > > Regards,
> > > Arun
> > >
> >
>


[GitHub] zeppelin issue #2105: [ZEPPELIN-2215] Progress bar for Spell execution

2017-03-15 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2105
  
@1ambda could you rebase this because of conflicts?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #2139: ZEPPELIN-2261. Support to connect with livy thr...

2017-03-15 Thread zjffdu
GitHub user zjffdu opened a pull request:

https://github.com/apache/zeppelin/pull/2139

ZEPPELIN-2261. Support to connect with livy through https

### What is this PR for?
Livy server support https, but the currently livy interpreter doesn't 
support it. This PR is for for the supporting to connect with livy through https


### What type of PR is it?
[Improvement]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-2261

### How should this be tested?
Tested manually on livy server with ssl enabled. 

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-2261

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2139.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2139


commit 5ab205091603462beed8415af263551ee569c63b
Author: Jeff Zhang 
Date:   2017-03-15T07:06:21Z

ZEPPELIN-2261. Support to connect with livy through https




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-2261) Support to connect with livy through https

2017-03-15 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-2261:


 Summary: Support to connect with livy through https
 Key: ZEPPELIN-2261
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2261
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.7.0
Reporter: Jeff Zhang
Assignee: Jeff Zhang






--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] zeppelin issue #2099: [ZEPPELIN-2106] providing paragraph config in create n...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2099
  
Thanks @Remilito for the contribution!

Could you elaborate why do you support two different forms of message?

```
{
  ...
  "graph": {
"mode": "pieChart"
  }
}
```

and
```
{
  ...
  "config": {
...
"results": [
  {
"graph": {
  "mode": "scatterChart",
  "optionOpen": true
}
  }
]
  }
}
```


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2116: [ZEPPELIN-2179] "clear output" paragraph doesn't work ...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2116
  
CI failure on `branch-0.7` is fixed. @1ambda Could you rebase this PR to 
`branch-0.7` and see if CI passes?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2124: [ZEPPELIN-2154] Support zeppelin.server.port in dev mo...

2017-03-15 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2124
  
Sure! Thanks @Leemoonsoo for inform it


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2115: [ZEPPELIN-2202] Disable personalized mode btn when not...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2115
  
This PR has been merged but @asfgit does not close this PR automatically. 
@1ambda Could you close 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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2124: [ZEPPELIN-2154] Support zeppelin.server.port in dev mo...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2124
  
CI failure on `branch-0.7` has fixed by 
https://github.com/apache/zeppelin/pull/2103.
@soralee could you rebase this PR to `branch-0.7` and see if CI passes?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2131: ZEPPELIN-2256. poll job progress is called twice

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2131
  
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #2135: [WIP] Groovy Interpreter for Apazhe Zeppelin [ZEPPELIN...

2017-03-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2135
  
The error 

```
[ERROR] error An unexpected error occurred: 
"https://registry.yarnpkg.com/binary-extensions/-/binary-extensions-1.8.0.tgz: 
Request failed \"502 Bad Gateway\"".
[ERROR] Failed to execute goal 
com.github.eirslett:frontend-maven-plugin:1.3:yarn (yarn install) on project 
zeppelin-web: Failed to run task: 'yarn install --no-lockfile' failed. (error 
code 1) -> [Help 1]
[ERROR] 
```
looks like temporary network problem. You can always restart [the failed 
Job](https://travis-ci.org/dlukyanov/zeppelin/jobs/211224371) and see if it 
passing.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---