[GitHub] zeppelin issue #1582: [ZEPPELIN-1604] Add Neo4j interpreter and Network visu...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1582
  
Sorry for late response. 
Sounds like a plan!


---
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 #2076: ZEPPELIN-2173 Duplicate user names populated in the No...

[soralee]

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2076
  
Tested and It works well as that remove duplication name. 

But, In my personal opinion, I'd like to suggest that It would be better to 
show it in its original state. For example, if user who is not permission fail 
to edit permission, the status of permission is displayed to values before the 
modification .

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 #2078: [ZEPPELIN-1988] add property "precode" to JDBCInterpre...

[sotnich]

Github user sotnich commented on the issue:

https://github.com/apache/zeppelin/pull/2078
  
Looks Good To Me


---
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 #2078: [ZEPPELIN-1988] add property "precode" to JDBCInterpre...

[andrykrp]

Github user andrykrp commented on the issue:

https://github.com/apache/zeppelin/pull/2078
  
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 #2055: [ZEPPELIN-1999] get interpreter property with replaced...

[andrykrp]

Github user andrykrp commented on the issue:

https://github.com/apache/zeppelin/pull/2055
  
Looks good


---
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 #1887: [ZEPPELIN-1940] most of eslint rules are NOT applied a...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1887
  
@1ambda Sure. No worries! 


---
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 #2079: ZEPPELIN-2195. Use PYSPARK_PYTHON and PYSPARK_D...

[zjffdu]

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-2195. Use PYSPARK_PYTHON and PYSPARK_DRIVER_PYTHON over 
zeppelin.pyspark.python

### What is this PR for?
`zeppelin.pyspark.python` is zeppelin configuration for the python exec on 
driver side, it won't affect executor side. It would be better to use 
`PYSPARK_PYTHON` and `PYSPARK_DRIVER_PYTHON`  which is what spark use 
officially.  So that user can define their own python exec in interpreter 
setting for different version of python rather than defining them 
`zeppelin-env.sh` which is shared globally. 


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

### Todos
* [ ] - Task

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

### How should this be tested?
Tested it manually. 


### 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-2195

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

https://github.com/apache/zeppelin/pull/2079.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 #2079


commit 7395c93ece18d61c5ab39844261dae6bcc23e591
Author: Jeff Zhang 
Date:   2017-02-28T05:57:53Z

ZEPPELIN-2195. Use PYSPARK_PYTHON and PYSPARK_DRIVER_PYTHON over 
zeppelin.pyspark.python




---
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-2195) Use PYSPARK_PYTHON and PYSPARK_DRIVER_PYTHON over zeppelin.pyspark.python

[Jeff Zhang (JIRA)]

Jeff Zhang created ZEPPELIN-2195:


 Summary: Use PYSPARK_PYTHON and PYSPARK_DRIVER_PYTHON over 
zeppelin.pyspark.python
 Key: ZEPPELIN-2195
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2195
 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 #1495: ZEPPELIN-1325: WIP - Python, advanced interpreter impl

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1495
  
ping @bzz Any updates ?


---
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 #2076: ZEPPELIN-2173 Duplicate user names populated in the No...

[soralee]

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2076
  
@r-kamath Thanks for fixing this issue!
Let me check and give a 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.
---

[jira] [Created] (ZEPPELIN-2194) precode execution in Python Interpreter

[Maks (JIRA)]

Maks created ZEPPELIN-2194:
--

 Summary: precode execution in Python Interpreter
 Key: ZEPPELIN-2194
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2194
 Project: Zeppelin
  Issue Type: Improvement
  Components: Interpreters
Reporter: Maks
Priority: Minor






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

[GitHub] zeppelin issue #2078: [ZEPPELIN-1988] add property "precode" to JDBCInterpre...

[tinkoff-dwh]

Github user tinkoff-dwh commented on the issue:

https://github.com/apache/zeppelin/pull/2078
  
CI failed, errors in unrelated tests
https://travis-ci.org/tinkoff-dwh/zeppelin/builds/205735485


---
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 #1887: [ZEPPELIN-1940] most of eslint rules are NOT applied a...

[1ambda]

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1887
  
@AhyoungRyu Thanks for caring this! 

- It would be easier to start from scratch i think. 😭 
- I will work on after #1982 is handed!


---
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 #2078: [ZEPPELIN-1988] add property "precode" to JDBCI...

[tinkoff-dwh]

GitHub user tinkoff-dwh opened a pull request:

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

[ZEPPELIN-1988] add property "precode" to JDBCInterpreter

### What is this PR for?
Adds property "precode". Value of property contains SQL which executes 
while opening connection.


### What type of PR is it?
Improvement

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

### How should this be tested?
1) Set property zeppelin.interpreter.precode =` set search_path='test, 
public' `
2) Execute `%jdbc show search_path`

### 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/tinkoff-dwh/zeppelin ZEPPELIN-1988

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

https://github.com/apache/zeppelin/pull/2078.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 #2078


commit ba3477a6afb2fd29df5e88124599a37453ab480c
Author: Tinkoff DWH 
Date:   2017-02-27T09:41:58Z

[ZEPPELIN-1988] add property "precode" to JDBCInterpreter




---
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 #2074: ZEPPELIN-2148: On creation of Bar graph zeppelin UI sh...

[1ambda]

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2074
  
Sure. 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 #2054: [ZEPPELIN-2113] Remove border and margin of Link this ...

[soralee]

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2054
  
@prabhjyotsingh Thanks for review!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled 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 #2054: [ZEPPELIN-2113] Remove border and margin of Link this ...

[prabhjyotsingh]

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2054
  
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 #2042: [ZEPPELIN-1588]: bumping nvd3 to 1.8.5

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2042
  
Tested and it works well. Definitely better than before 👍  


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this 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-2193) Helium can't bundle which has large svg icon text

[Hoon Park (JIRA)]

Hoon Park created ZEPPELIN-2193:
---

 Summary: Helium can't bundle which has large svg icon text
 Key: ZEPPELIN-2193
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2193
 Project: Zeppelin
  Issue Type: Bug
  Components: GUI, helium
Reporter: Hoon Park


Helium can't bundle which has large svg icon text

- To reproduce this, create {{package.json}} which include the icon below.

{code}
  "helium": {
"icon": "http://www.w3.org/2000/svg\; 
xmlns:xlink=\"http://www.w3.org/1999/xlink\; x=\"0px\" y=\"0px\" viewBox=\"0 0 
303.188 303.188\" style=\"enable-background:new 0 0 303.188 303.188;\" 
xml:space=\"preserve\">   
 
  ",
"spell": {
  "magic": "%csv",
  "usage": "%csv "
}
  }
{code}

And error message throwed by npm is confusing 

{code}
npm ERR! node v6.9.2
npm ERR! npm  v3.10.9
npm ERR! code E404

npm ERR! 404 Registry returned 404 for GET on 
https://registry.npmjs.org/zeppelin-csv-spell
npm ERR! 404
npm ERR! 404  'zeppelin-csv-spell' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404 It was specified as a dependency of 'zeppelin-helium-bundle'
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! Please include the following file with any support request:
npm ERR! 
/Users/1ambda/github/apache/apache-zeppelin/zeppelin-feature/local-repo/helium-bundle/npm-debug.log
{code}



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

[GitHub] zeppelin issue #2054: [ZEPPELIN-2113] Remove border and margin of Link this ...

[soralee]

Github user soralee commented on the issue:

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

[GitHub] zeppelin pull request #2004: [MINOR] add pig wiki page to pig doc

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #2067: [DOCS][ZEPPELIN-2140] Add docs for notebookRepo...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1980: [gh-pages][ZEPPELIN-2061] Add Medium blog secti...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #2000: [gh-pages] update contribution/webapplication.html

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2000
  
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 #2073: [ZEPPELIN-2130][Doc]Do not use web development port

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2073
  
Merge into master and branch-0.7 if there are no 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 #1985: [ZEPPELIN-2075] Can't stop infinite `while` statement ...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1985
  
> User zeppelin have permission to execute paswordless sudo to local users 
allowed to log into Zeppelin
> Interpreters will with this run as the user starting the process, and the 
zeppelin user will not have permission to send any signal to them.

I think @FireArrow 's concern is that zeppelin user can kill the process. 
Actually it is not the user zeppelin send the signal, it is the interpreter 
process owner. So I think don't think there's any permission issue here. 
@FireArrow Please help confirm. 



---
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 #1985: [ZEPPELIN-2075] Can't stop infinite `while` statement ...

[astroshim]

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1985
  
@zjffdu I tested impersonation mode followed 
http://zeppelin.apache.org/docs/0.7.0/manual/userimpersonation.html and worked 
well.
BTW for now anybody can interrupt to interpreter process. and that is 
@FireArrow's worrying. Am i right?


---
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 #1985: [ZEPPELIN-2075] Can't stop infinite `while` statement ...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1985
  
Even in the impersonation mode, it is the process owner to interrupt the 
process. Here it is the interpreter process owner interrupt the process instead 
of the zeppelin server owner.  But I can not verify it in impersonation mode. 
It seems the impersonation mode is broken. I got the following error, can 
anyone else verify the impersonation mode ?

```
ERROR [2017-02-28 09:20:18,882] ({pool-2-thread-2} Job.java[run]:194) - Job 
failed
org.apache.zeppelin.interpreter.InterpreterException: 
org.apache.hadoop.security.authorize.AuthorizationException: User: user1 is not 
allowed to impersonate user1
```


---
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 #2077: [ZEPPELIN-2191] Helium package should be able to impor...

[1ambda]

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2077
  
CI failed, but irrelevant.

- build profile: https://travis-ci.org/1ambda/zeppelin/builds/206002571
- failed log1: 
https://s3.amazonaws.com/archive.travis-ci.org/jobs/206002573/log.txt
- failed log2: 
https://s3.amazonaws.com/archive.travis-ci.org/jobs/206002574/log.txt


---
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 #1985: [ZEPPELIN-2075] Can't stop infinite `while` statement ...

[astroshim]

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1985
  
@Leemoonsoo @zjffdu @felixcheung @karuppayya  Could you guys help to review 
about the impersonation mode?


---
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 #2077: [ZEPPELIN-2191] Helium package should be able t...

[1ambda]

GitHub user 1ambda opened a pull request:

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

[ZEPPELIN-2191] Helium package should be able to import css

### What is this PR for?

Helium package should be able to import css. For example,

- Someone want to create spell using jsoneditor 
(https://github.com/josdejong/jsoneditor/blob/master/docs/usage.md). In this 
case, he/she should import like this

```js
import 'jsoneditor/dist/jsoneditor.min.css'
import JSONEditor from 'jsoneditor/dist/jsoneditor.min.js'
```



**This PR is written based on #2071, so duplicated commit will be dropped. 
Please review the commit 505c7db only.**

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

### Todos
* [x] - denote required packages in `package.json`
* [x] - setup `webpack.config.js`

### What is the Jira issue?

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

### How should this be tested?

Should be able to bundle 
[zeppelin-echo-spell](https://github.com/1ambda/zeppelin-json-spell)

### Screenshots (if appropriate)

NONE

### 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-2191/import-css-in-helium-pkg

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

https://github.com/apache/zeppelin/pull/2077.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 #2077


commit 00a0afe12250db36290acb11d7e8f94aa30822bd
Author: 1ambda <1am...@gmail.com>
Date:   2017-02-27T02:35:38Z

fix: Update babel configuration

commit 505c7dbb51a80253704c9fc0b29d2c2f154be378
Author: 1ambda <1am...@gmail.com>
Date:   2017-02-27T23:45:19Z

fix: Import css globally in helium package




---
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 #2062: ZEPPELIN-2161 Nested Group Support in LdapRealm for AD

[weand]

Github user weand commented on the issue:

https://github.com/apache/zeppelin/pull/2062
  
@prabhjyotsingh There isn't any documentation for 
org.apache.zeppelin.realm.LdapRealm yet.

@Leemoonsoo Any special reason why LdapRealm wasn't documented in 0.7.0 
yet? 

Can anyone help me why the build is failing. I don't see, why these tests 
now fail, they have no obvious reference to changes of this PR:
  NotebookTest.testPublicPrivateNewNote:1173 expected:<2C[CJ4Y6RZ]> but 
was:<2C[ADXZJ8N]>
  NotebookTest.testAuthorizationRoles:759 expected: but was:


---
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 #1985: [ZEPPELIN-2075] Can't stop infinite `while` statement ...

[FireArrow]

Github user FireArrow commented on the issue:

https://github.com/apache/zeppelin/pull/1985
  
I have basically only set up zeppelin in isolated mode with impersonation, 
so I'm not sure what is the "expected" behavior is outside of that. Someone 
else will have to pitch in here.


---
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-2192) Helium should provide a way to configure each package's webpack config

[Hoon Park (JIRA)]

Hoon Park created ZEPPELIN-2192:
---

 Summary: Helium should provide a way to configure each package's 
webpack config
 Key: ZEPPELIN-2192
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2192
 Project: Zeppelin
  Issue Type: Improvement
  Components: front-end, helium
Reporter: Hoon Park
 Fix For: 0.8.0


Helium should provide a way to configure each package's webpack config. 

Should be handed after https://issues.apache.org/jira/browse/ZEPPELIN-2088



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

[GitHub] zeppelin issue #1985: [ZEPPELIN-2075] Can't stop infinite `while` statement ...

[astroshim]

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1985
  
@FireArrow I think it seems not can be possible to know if impersonation 
mode or not in Interpreter side(I couldn't find it) and checking `cancel` 
paragraph for impersonation is also not good idea (Interpreter sever can be 
launched itself not the same system with ZeppelinServer)
So I think that the canceling job can run only by user who launched 
interpreter process can be one option. and In this way, the other user can't 
stop the job in shared mode.
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.
---

[jira] [Created] (ZEPPELIN-2191) Helium package should be able to import css

[Hoon Park (JIRA)]

Hoon Park created ZEPPELIN-2191:
---

 Summary: Helium package should be able to import css
 Key: ZEPPELIN-2191
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2191
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end, helium
Affects Versions: 0.7.0
Reporter: Hoon Park
 Fix For: 0.8.0


Helium package should be able to import css. For example,

- Someone want to create spell using jsoneditor 
(https://github.com/josdejong/jsoneditor/blob/master/docs/usage.md). He / She 
will import like this

{code}
import 'jsoneditor/dist/jsoneditor.min.css'
import JSONEditor from 'jsoneditor/dist/jsoneditor.min.js'
{code}



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

[GitHub] zeppelin pull request #2076: ZEPPELIN-2173 Duplicate user names populated in...

[r-kamath]

GitHub user r-kamath opened a pull request:

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

ZEPPELIN-2173 Duplicate user names populated in the Note permission box

### What is this PR for?
Duplicate user names populated in the Note permission box

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

### Todos
* [ ] - Task

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

### How should this be tested?
See JIRA description for the steps to reproduce the issue.

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/r-kamath/zeppelin ZEPPELIN-2173

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

https://github.com/apache/zeppelin/pull/2076.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 #2076


commit 14296b4e59fa81066b4d96e2b439adfaa5dcd851
Author: Renjith Kamath 
Date:   2017-02-27T13:53:11Z

ZEPPELIN-2173 Duplicate user names populated in the Note permission box




---
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 #2073: [ZEPPELIN-2130][Doc]Do not use web development port

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2073
  
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 #2073: [ZEPPELIN-2130][Doc]Do not use web development port

[soralee]

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2073
  
Thanks for review @AhyoungRyu @1ambda @felixcheung!
And I updated to remove `.`  :smile: 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does 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 #2037: [ZEPPELIN-2109][2110] Sortable Helium pkgs & Introduce...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2037
  
Thanks @1ambda! 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 #2037: [ZEPPELIN-2109][2110] Sortable Helium pkgs & Introduce...

[1ambda]

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2037
  
I will review and comment soon. Great Feature!


---
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 #1980: [gh-pages][ZEPPELIN-2061] Add Medium blog section to t...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1980
  
I added [Medium Story Channel for 
Zeppelin](https://cwiki.apache.org/confluence/display/ZEPPELIN/Medium+Story+Channel+for+Zeppelin)
 under Zeppelin wiki page as @Leemoonsoo  suggested. If the contents need to be 
added more, please let me know. 
I'll merge this into `gh-pages` branch 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 #2075: [ZEPPELIN-791] Build infra: move all RAT to roo...

[DmytroShkvyra]

GitHub user DmytroShkvyra opened a pull request:

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

[ZEPPELIN-791] Build infra: move all RAT to root pom.xml

What is this PR for?

It is better have a single place where we manage project-wise RAT 
exclusions for a contributions under licenses different from Apache, then let 
maven sub-modules have them as we do now (makes things harder to track)
What type of PR is it?

[Improvement]
Todos

N/A
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-791
How should this be tested?

mvn verify -DskipTests
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/DmytroShkvyra/zeppelin ZEPPELIN-791-1

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

https://github.com/apache/zeppelin/pull/2075.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 #2075


commit fb888e8562c6c8720361d1a27f853ca1f4dbd3b2
Author: DmytroShkvyra 
Date:   2017-02-27T11:53:00Z

[ZEPPELIN-791] Build infra: move all RAT to root pom.xml




---
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 #2004: [MINOR] add pig wiki page to pig doc

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2004
  
Merge into master and branch-0.7 if there are no more comments 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 pull request #2043: [ZEPPELIN-2114] adding reload endpoint

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #2065: [ZEPPELIN-2168] Fix typo on Elasticsearch inter...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #2032: [ZEPPELIN-791] Build infra: move all RAT to roo...

[DmytroShkvyra]

Github user DmytroShkvyra closed the pull request at:

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


---
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 #2071: [ZEPPELIN-2166] HeliumBundleFactoty can't transfile im...

[1ambda]

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2071
  
CI failed, but irrelevant. 

- profile: https://travis-ci.org/1ambda/zeppelin/builds/205664772
- failed log1: 
https://s3.amazonaws.com/archive.travis-ci.org/jobs/205664774/log.txt
- failed log2: 
https://s3.amazonaws.com/archive.travis-ci.org/jobs/205664775/log.txt


---
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 #2072: [ZEPPELIN-2060] Show tooltip msg after dynamic form va...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2072
  
@Leemoonsoo I checked now, the behaviour is same (the tooltip is shown 
right after sth changed in dynamic form) like default mode. I think this should 
be fixed. Maybe I need to hide this tooltip in report mode. Or maybe another 
tooltip msg can be shown sth like "You can't run the paragraph in report mode". 
Not sure which one is better. 

BTW don't we need to block any changes in dynamic form then? Currently we 
can't run the paragraph since the run button is not shown and `Shift+Enter` 
doesn't work.

@Leemoonsoo @felixcheung Any suggestion will be appreciated :)


---
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 #2051: [ZEPPELIN-1968] Added property to disable hive user im...

[prabhjyotsingh]

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2051
  
Yes, 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 #2051: [ZEPPELIN-1968] Added property to disable hive user im...

[gfalcone]

Github user gfalcone commented on the issue:

https://github.com/apache/zeppelin/pull/2051
  
@prabhjyotsingh is everything good for you :) ?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have 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 #2072: [ZEPPELIN-2060] Show tooltip msg after dynamic form va...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2072
  
Great. How does it work in report mode?


---
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 #2068: Groovy Interpreter for Apazhe Zeppelin [ZEPPELIN-2176]

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/2068
  
Please see 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
 for setting up 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 #1582: [ZEPPELIN-1604] Add Neo4j interpreter and Network visu...

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1582
  
Let's go with (c) & (b) and then (a)?


---
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 #1982: [ZEPPELIN-2069] Helium Package Configuration

[1ambda]

Github user 1ambda commented on the issue:

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

## Newly added commits

- Remove `local` values in helium package jsons: 
https://github.com/apache/zeppelin/pull/1982/commits/ce5f8c0c0f88de79bd9c4172a5d74066c9345a50
- DON't refresh `helium.json` when fetching spell config to reduce 
execution time: 
https://github.com/apache/zeppelin/pull/1982/commits/dbc4f10fd3ee556d5e38cb4f6e391eaf69a9

@Leemoonsoo Could you help review? Thanks 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.
---