date:20161214

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1766
  
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 pull request #1766: [MINOR] Set default interpreter \w first intp

GitHub user AhyoungRyu opened a pull request:

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

[MINOR] Set default interpreter \w first intp

### What is this PR for?
Currently we can select default interpreter when we create a new note like 
below.
![screen shot 2016-12-15 at 4 41 15 
pm](https://cloud.githubusercontent.com/assets/10060731/21215571/56bf4078-c2e5-11e6-8455-b2cb0e126007.png)

However, even if we don't set the default interpreter(don't select any 
interpreters) in this dialog and just create new note, the default interpreter 
is automatically set as `Spark` interpreter. This can be checked in the 
interpreter binding page. It doesn't make sense I think. 
So I removed `--select--` and set the default option in the select box with 
`Spark` as the interpreter binding page does. 

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

### What is the Jira issue?
N/A

### How should this be tested?
1. Build only web \w `npm run build` under `zeppelin-web/` and run `npm run 
start` for dev mode. 
2. Create a new note and see the select box 

or just see the below screenshots :)

### Screenshots (if appropriate)
 - Before 

![before](https://cloud.githubusercontent.com/assets/10060731/21215524/1b3f9886-c2e5-11e6-9947-e9186e087b54.gif)

 - After

![default_intp](https://cloud.githubusercontent.com/assets/10060731/21215526/1cc8d258-c2e5-11e6-9a71-504f7164078b.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/AhyoungRyu/zeppelin fix-default-intp

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

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


commit 569e63bf27180c81beb0d35faabf477714fa4fec
Author: AhyoungRyu 
Date:   2016-12-15T07:32:13Z

Set default interpreter \w first intp




---
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 #1765: ZEPPELIN-1761. CI: ZeppelinSparkClusterTest .py...

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1761. CI: ZeppelinSparkClusterTest .pySpark* fail the build

### What is this PR for?
Add more logging as I could not reproduce it locally. 


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

### Todos
* [ ] - Task

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

### How should this be tested?
Outline the steps to test the PR here.

### 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/zjffdu/zeppelin ZEPPELIN-1761

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

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


commit 40bdf30185e6b63c7178e44973a3d922ab69efb1
Author: Jeff Zhang 
Date:   2016-12-15T07:29:07Z

ZEPPELIN-1761. CI: ZeppelinSparkClusterTest .pySpark* fail the build




---
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 #1750: [ZEPPELIN-1796] Add 'copy to clipboard' next to the pa...

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1750
  
CI has passed now. Ready to review 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.
---

[GitHub] zeppelin issue #1763: [MINOR] Add trivial cosmetic changes on STYLE.md

2016-12-14 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1763
  
nice!



---
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-1817) Add documentation to Zeppelin's shiro.ini to provide help users configure security

2016-12-14 Thread Vinay Shukla (JIRA)

Vinay Shukla created ZEPPELIN-1817:
--

 Summary: Add documentation to Zeppelin's shiro.ini to provide help 
users configure security
 Key: ZEPPELIN-1817
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1817
 Project: Zeppelin
  Issue Type: Task
Reporter: Vinay Shukla
Assignee: Vinay Shukla


Right now Zeppelin's shiro.ini has only limited documentation. It would be 
great to add more documentation (as comments) to shiro.ini so it is self 
documented and will help end-user configure Zeppelin for security



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] zeppelin issue #1753: [ZEPPELIN-1594] Support personalized mode

Github user cloverhearts commented on the issue:

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

CI failed.
However, this is a not relate to modifications.
```
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:2.17:test (default-test) on 
project zeppelin-pig: Execution default-test of goal 
org.apache.maven.plugins:maven-surefire-plugin:2.17:test failed: The forked VM 
terminated without properly saying goodbye. VM crash or System.exit called?
[ERROR] Command was /bin/sh -c cd /home/travis/build/apache/zeppelin/pig && 
/usr/lib/jvm/java-7-oracle/jre/bin/java -Xmx2g -Xms1g -Dfile.encoding=UTF-8 
-jar 
/home/travis/build/apache/zeppelin/pig/target/surefire/surefirebooter6134435344985761979.jar
 
/home/travis/build/apache/zeppelin/pig/target/surefire/surefire308229830701668274tmp
 
/home/travis/build/apache/zeppelin/pig/target/surefire/surefire_331449743848135082457tmp
[ERROR] -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, 
please read the following articles:
[ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/PluginExecutionException
[ERROR] 
[ERROR] After correcting the problems, you can resume the build with the 
command
[ERROR]   mvn  -rf :zeppelin-pig
```


---
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 #1753: [ZEPPELIN-1594] Support personalized mode

2016-12-14 Thread jongyoul

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1753
  
@minahlee @Leemoonsoo Please test it. @cloverhearts and I have tested but I 
want to check it 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.
---

[GitHub] zeppelin issue #1750: [ZEPPELIN-1796] Add 'copy to clipboard' next to the pa...

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1750
  
For the consistency with other menus in the dropdown menu, I removed the 
clippy icon and make paragraph id text itself as a button.
![screen shot 2016-12-15 at 2 56 43 
pm](https://cloud.githubusercontent.com/assets/10060731/21213316/d6a20032-c2d6-11e6-8c37-90ed644e153d.png)![screen
 shot 2016-12-15 at 2 56 51 
pm](https://cloud.githubusercontent.com/assets/10060731/21213329/f4392e40-c2d6-11e6-9387-04542f4f7c99.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 #1753: [ZEPPELIN-1594] Support personalized mode

Github user cloverhearts commented on the issue:

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

![123123ppp](https://cloud.githubusercontent.com/assets/10525473/21213045/44013a82-c2d5-11e6-82e6-5c7591069516.gif)

support `live mode share `
the notes about using the user-to-user mode are reflected in real time.
corrected invalid log name.
only `owner` can modify personalized 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 #1764: ZEPPELIN-1816. Implement getProgress in SparkRInterpre...

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1764
  
@Leemoonsoo @echarles Please help review. The failed test is not relevant.

```
Results :
 
Failed tests: 
  ZeppelinSparkClusterTest.zRunTest:347 expected: but was:
```
```
Caused by: org.openqa.selenium.StaleElementReferenceException: Element not 
found in the cache - perhaps the page has changed since it was looked up
For documentation on this error, please visit: 
http://seleniumhq.org/exceptions/stale_element_reference.html
Build info: version: '2.48.2', revision: 
'41bccdd10cf2c0560f637404c2d96164b67d9d67', time: '2015-10-09 13:08:06'
System info: host: 'testing-docker-44f30cc6-7d42-4680-bea6-3b04f484c151', 
ip: '172.17.0.7', os.name: 'Linux', os.arch: 'amd64', os.version: 
'4.8.12-040812-generic', java.version: '1.7.0_76'
Driver info: driver.version: unknown
at .fxdriver.cache.getElementAt(resource://fxdriver/modules/web-element-cache.js:9351)
at .Utils.getElementAt(file:///tmp/anonymous195422017500551webdriver-profile/extensions/fxdri...@googlecode.com/components/command-processor.js:8978)
at .WebElement.getElementText(file:///tmp/anonymous195422017500551webdriver-profile/extensions/fxdri...@googlecode.com/components/command-processor.js:11965)
at .DelayedCommand.prototype.executeInternal_/h(file:///tmp/anonymous195422017500551webdriver-profile/extensions/fxdri...@googlecode.com/components/command-processor.js:12534)
at .fxdriver.Timer.prototype.setTimeout/<.notify(file:///tmp/anonymous195422017500551webdriver-profile/extensions/fxdri...@googlecode.com/components/command-processor.js:621)
Results :
Tests in error: 
  ZeppelinIT.testAngularDisplay:157->AbstractZeppelinIT.waitForText:84 Â» 
StaleElementReference
```


---
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 #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1758
  
How about refer this note in Flink.md ?


---
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 #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1758
  
@AlexanderShoshin Thanks for your contribution! 
While I was just quickly looking through this example note in my Zeppelin, 
couple of things were noticed. 
 - AFAIK, OSX doesn't have `wget` by default so the OSX users might need to 
install by themselves. So I would suggest you to use `curl` instead of `wget` 
to download datasets. (`curl` is built-in command I guess)

 - I saw you set the location of dataset to `tmp/`. It can occurs sth link 
"permission denied" error for that dir in Zeppelin like below. 
```
Caused by: org.apache.flink.runtime.JobException: 
Creating the input splits caused an error: 
File /tmp/flights98.csv does not exist or the user running Flink 
('ahyoungryu') has insufficient permissions to access 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 pull request #1764: ZEPPELIN-1816. Implement getProgress in SparkRI...

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1816. Implement getProgress in SparkRInterpreter

### What is this PR for?
Since I add jobGroup in ZEPPELIN-1769, it is easy to implement getProgress 
as well. It could be delegated to `SparkInterpreter` in the same session.


### What type of PR is it?
 Improvement | Feature ]

### Todos
* [ ] - Task

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

### How should this be tested?
Tested manually, see screenshot

### Screenshots (if appropriate)

![sparkr_progress](https://cloud.githubusercontent.com/assets/164491/21212236/aeadd22e-c2c5-11e6-8a16-ba3f940aee39.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/zjffdu/zeppelin ZEPPELIN-1816

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

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


commit 20425ea3eca37be91fcdfedee127f118119ad7e7
Author: Jeff Zhang 
Date:   2016-12-15T04:53:23Z

ZEPPELIN-1816. Implement getProgress in SparkRInterpreter




---
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-1816) Implement getProgress in SparkRInterpreter

2016-12-14 Thread Jeff Zhang (JIRA)

Jeff Zhang created ZEPPELIN-1816:


 Summary: Implement getProgress in SparkRInterpreter
 Key: ZEPPELIN-1816
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1816
 Project: Zeppelin
  Issue Type: Improvement
Affects Versions: 0.7.0
Reporter: Jeff Zhang
Assignee: Jeff Zhang
 Fix For: 0.7.0


Since I add jobGroup in ZEPPELIN-1769, it is easy to implement getProgress as 
well. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] zeppelin issue #1763: [MINOR] Add trivial cosmetic changes on STYLE.md

2016-12-14 Thread HyukjinKwon

Github user HyukjinKwon commented on the issue:

https://github.com/apache/zeppelin/pull/1763
  
Thank you, @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 #1763: [MINOR] Add trivial cosmetic changes on STYLE.md

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1763
  
@HyukjinKwon Thanks for your contribution. Looks nice ð 


---
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 #1730: [ZEPPELIN-1736] Introduce trash & enable removing fold...

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1730
  
@tae-jun Appreciate for your precise explanation and picking proper 
candidates as well :) Tested again to check the time stamp,  it makes sense. 
Then how about attaching the time stamp for every cases to keep consistency? 
Currently the firstly removed one doesn't show the stamp. Surely It's just my 
opinion. 
And regarding the icon, I couldn't find the better icon that has meaning 
"Clear" either. So you can use anything that you suggested! 



---
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 #1763: [MINOR] Add trivial cosmetic changes on STYLE.md

2016-12-14 Thread HyukjinKwon

Github user HyukjinKwon commented on the issue:

https://github.com/apache/zeppelin/pull/1763
  
Hi @felixcheung, it seems the last change was by you. Could you please take 
a look? I think I am not used to this and also not sure if this change is good 
or not but just decided to submit this as it just looks a little bit prettier 
apparently.


---
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 #1763: [MINOR] Add trivial cosmetic changes on STYLE.m...

2016-12-14 Thread HyukjinKwon

GitHub user HyukjinKwon opened a pull request:

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

[MINOR] Add trivial cosmetic changes on STYLE.md

### What is this PR for?

This PR proposes a little bit of cosmetic changes on `STYLE.md`.

**Before**

```
  "src-noconflict/mode-sql.js",
  "src-noconflict/mode-markdown.js",
  "src-noconflict/keybinding-emacs.js",
 "src-noconflict/ext-language_tools.js",
+ "src-noconflict/theme-github.js"],
   "version": "1.1.8",
   "name": "ace-builds"
},
"highlightjs": {
"main": ["highlight.pack.js",
+ "styles/github.css"],
  "version": "8.4.0",
  "name": "highlightjs"
 }
```

```
-  $scope.editor.setTheme('ace/theme/github');
+  $scope.editor.setTheme('ace/theme/monokai');
```

```
- "src-noconflict/theme-github.js"],
+ "src-noconflict/theme-monokai.js"],
```

**After**

```diff
  "src-noconflict/mode-sql.js",
  "src-noconflict/mode-markdown.js",
  "src-noconflict/keybinding-emacs.js",
  "src-noconflict/ext-language_tools.js",
+ "src-noconflict/theme-github.js"],
   "version": "1.1.8",
   "name": "ace-builds"
},
"highlightjs": {
"main": ["highlight.pack.js",
+ "styles/github.css"],
  "version": "8.4.0",
  "name": "highlightjs"
 }
```

```diff
-  $scope.editor.setTheme('ace/theme/github');
+  $scope.editor.setTheme('ace/theme/monokai');
```

```diff
- "src-noconflict/theme-github.js"],
+ "src-noconflict/theme-monokai.js"],
```

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

### What is the Jira issue?
MINOR

### How should this be tested?
Manually checked the rendering 

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



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

$ git pull https://github.com/HyukjinKwon/zeppelin minor-fix

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

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


commit a6fcbd31baa92c726a54e7b917f54903160396bf
Author: hyukjinkwon 
Date:   2016-12-15T02:44:24Z

Add trivial cosmetic changes on STYLE.md




---
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 #1733: [ZEPPELIN-1753] Fix blank notename issue

Github user asfgit closed the pull request at:

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


---
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 #1737: ZEPPELIN-1769. Support cancel job in SparkRInte...

Github user asfgit closed the pull request at:

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


---
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 #1722: [ZEPPELIN-1776] substitute null check for the r...

Github user asfgit closed the pull request at:

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


---
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 #1744: Better output of JDBC Interpreter.

Github user asfgit closed the pull request at:

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


---
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 #1754: [ZEPPELIN-1800] Update scio doc - fix type $ sh...

Github user asfgit closed the pull request at:

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


---
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 #1735: [ZEPPELIN-1767] Blank paragraph should not be r...

Github user asfgit closed the pull request at:

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


---
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 #1717: [ZEPPELIN-1744] replace to movement for paragraph in j...

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1717
  
Will merge this into master if there are no more comments.
@cloverhearts Thanks for your effort to keep maintaining job management :)


---
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 #1760: Update DESCRIPTION

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1760
  
2 CI build job failed with below errors and looks irrelevant with this 
change
```
Results :

Tests in error:
  InterpreterRestApiTest.testRestartInterpreterPerNote:303 NullPointer

Tests run: 82, Failures: 0, Errors: 1, Skipped: 1

[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-surefire-plugin:2.17:test (default-test) on 
project zeppelin-server: There are test failures.
[ERROR]
[ERROR] Please refer to 
/home/travis/build/apache/zeppelin/zeppelin-server/target/surefire-reports for 
the individual test results.
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, 
please read the following articles:
[ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the 
command
[ERROR]   mvn  -rf :zeppelin-server

Results :

Tests in error:
  InterpreterIT.testShowDescriptionOnInterpreterCreate:69 Â» 
ElementNotVisible El...
  ZeppelinIT.testAngularDisplay:157->AbstractZeppelinIT.waitForText:84 Â» 
StaleElementReference

Tests run: 18, Failures: 0, Errors: 2, Skipped: 0
```

Will Merge this 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 #1762: [ZEPPELIN-1810] Removed incorrect usage of getString

2016-12-14 Thread lresende

Github user lresende commented on the issue:

https://github.com/apache/zeppelin/pull/1762
  
Please check the build failure, but it might be a flaky test.

Otherwise, LGTM. Tested on master with these changes on top of #1757 and 
all looks good with relative path trust store.


---
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: code generation: paragraph generates another paragraph

2016-12-14 Thread Ruslan Dautkhanov

Hi Moon,

I believe there are many use cases beyound our needs, when users could
access
rest API directly through 'z' variable.

Done: https://issues.apache.org/jira/browse/ZEPPELIN-1814

Thank you.



-- 
Ruslan Dautkhanov

On Wed, Dec 14, 2016 at 8:53 AM, moon  wrote:

> Hi Ruslan,
>
> Although you can always create your function that call Zeppelin's rest API
> to add paragraph,
> providing capability to add paragraph through 'z' (more precisely, from
> Interpreter) helps
> provide user more interactive usage of notebook i think.
>
> Do you mind file an issue for this feature?
>
> Thanks,
> moon
>
> On Mon, Dec 12, 2016 at 2:55 PM Ruslan Dautkhanov 
> wrote:
>
> > We'd like to have paragraph's code generated by a preceding paragraph.
> >
> > For example, one of the use cases we have
> > is when %pyspark generates Hive DDLs.
> > (can't run those in Spark in some cases)
> > Any chance an output of a paragraph can be redirected to a following
> > paragraph?
> >
> > I was thinking something like this could be used
> >
> > https://zeppelin.apache.org/docs/latest/rest-api/rest-
> notebook.html#create-a-new-paragraph
> > http://
> > [zeppelin-server]:[zeppelin-port]/api/notebook/[notebookId]/paragraph
> > But not sure if there is a easy way to call Zeppelin API directly through
> > "z" variable?
> >
> > Something like z.addParagraph(...)
> >
> > In most cases a paragraph generates a SQL code that can't be run directly
> > as Spark SQL
> > and has to be run by a different engine, for example, by Hive.
> >
> >
> > Thanks,
> > Ruslan
> >
> >
>

[jira] [Created] (ZEPPELIN-1814) Accessing Zeppelin's rest API through `z` variable

2016-12-14 Thread Ruslan Dautkhanov (JIRA)

Ruslan Dautkhanov created ZEPPELIN-1814:
---

 Summary: Accessing Zeppelin's rest API through `z` variable
 Key: ZEPPELIN-1814
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1814
 Project: Zeppelin
  Issue Type: Improvement
  Components: Core, Interpreters, rest-api, zeppelin-server, 
zeppelin-zengine
Affects Versions: 0.7.0
Reporter: Ruslan Dautkhanov


We'd like to have paragraph's code generated by a preceding paragraph.

For example, one of the use cases we have  is when %pyspark generates Hive 
DDLs. (can't run those in Spark in some cases)
Any chance an output of a paragraph can be redirected to a following paragraph?

I was thinking something like this could be used
https://zeppelin.apache.org/docs/latest/rest-api/rest-notebook.html#create-a-new-paragraph
{noformat}
http://[zeppelin-server]:[zeppelin-port]/api/notebook/[notebookId]/paragraph
{noformat}
But not sure if there is a easy way to call Zeppelin API directly through "z" 
variable?

Something like z.addParagraph(...) 

In most cases a paragraph generates a SQL code that can't be run directly as 
Spark SQL and has to be run by a different engine, for example, by Hive or by a 
JDBC backend. It's one of the use cases that we have, but I am sure there are 
many more use cases too.

Reply from [~moon] in email distribution list
{quote}
Although you can always create your function that call Zeppelin's rest API
to add paragraph, providing capability to add paragraph through 'z' (more 
precisely, from Interpreter) helps provide user more interactive usage of 
notebook i think.
{quote}





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Created] (ZEPPELIN-1813) If FQDN Hostnames are broken, Zeppelin Websockets show disconnected.

2016-12-14 Thread Julien Eid (JIRA)

Julien Eid created ZEPPELIN-1813:


 Summary: If FQDN Hostnames are broken, Zeppelin Websockets show 
disconnected.
 Key: ZEPPELIN-1813
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1813
 Project: Zeppelin
  Issue Type: Bug
Reporter: Julien Eid


Hey everyone, I've seen this issue come up a few times now. If the hostname is 
not resolvable of the box hosting Zeppelin, the /ws endpoint 500 errors, so 
Zeppelin shows as disconnected in the top right.

https://github.com/apache/zeppelin/pull/235/files#diff-0d6ccf388c8aa67062870388536ed274R36

The code on that line throws an UnknownHostException ( 
http://stackoverflow.com/a/1881967) as the hostname isn't in /etc/hosts on nor 
is the hostname resolvable by DNS. The fastest way to repro this is to pretty 
much clear /etc/hosts and only have localhost 127.0.0.1 in there, from what 
I've seen.

Is it possible to change this section to catch the exception, and if the 
hostname isn't resolvable still attempt the connection? 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Created] (ZEPPELIN-1812) Editor code is not visible after page refresh.

2016-12-14 Thread Spencer Smith (JIRA)

Spencer Smith created ZEPPELIN-1812:
---

 Summary: Editor code is not visible after page refresh.
 Key: ZEPPELIN-1812
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1812
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end, GUI
Affects Versions: 0.7.0
 Environment: OSX El Capitan. Chrome 54.0.2840.98 (64-bit)
Reporter: Spencer Smith


After a notebook has been created and the page refreshed the editor code is no 
longer visible when clicking the show/hide editor button. If you click "Show 
Editor" and then refresh the page, the editor code is shown again. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] zeppelin issue #1745: ZEPPELIN-1785. Remove ZEPPELIN_NOTEBOOK_DIR and ZEPPEL...

2016-12-14 Thread jongyoul

Github user jongyoul commented on the issue:

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

Re: code generation: paragraph generates another paragraph

2016-12-14 Thread moon

Hi Ruslan,

Although you can always create your function that call Zeppelin's rest API
to add paragraph,
providing capability to add paragraph through 'z' (more precisely, from
Interpreter) helps
provide user more interactive usage of notebook i think.

Do you mind file an issue for this feature?

Thanks,
moon

On Mon, Dec 12, 2016 at 2:55 PM Ruslan Dautkhanov 
wrote:

> We'd like to have paragraph's code generated by a preceding paragraph.
>
> For example, one of the use cases we have
> is when %pyspark generates Hive DDLs.
> (can't run those in Spark in some cases)
> Any chance an output of a paragraph can be redirected to a following
> paragraph?
>
> I was thinking something like this could be used
>
> https://zeppelin.apache.org/docs/latest/rest-api/rest-notebook.html#create-a-new-paragraph
> http://
> [zeppelin-server]:[zeppelin-port]/api/notebook/[notebookId]/paragraph
> But not sure if there is a easy way to call Zeppelin API directly through
> "z" variable?
>
> Something like z.addParagraph(...)
>
> In most cases a paragraph generates a SQL code that can't be run directly
> as Spark SQL
> and has to be run by a different engine, for example, by Hive.
>
>
> Thanks,
> Ruslan
>
>

[GitHub] zeppelin pull request #1762: [ZEPPELIN-1810] Removed incorrect usage of getS...

2016-12-14 Thread FireArrow

GitHub user FireArrow opened a pull request:

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

[ZEPPELIN-1810] Removed incorrect usage of getString

### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - 
https://zeppelin.apache.org/contribution/contributions.html


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

### Todos
* [ ] - Task

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

### How should this be tested?
Before this fix, after setting `zeppelin.ssl` to true in 
`zeppelin-site.xml` and setting `zeppelin.ssl.keystore.path` to a relative path 
to a keystore (technically it doens't have to exist at all) and setting 
`zeppelin.ssl.truststore.path` to a relative path to a truststore Zeppelin 
won't start. In the log you can read something like
```WARN [2016-12-14 13:55:45,522] ({main} 
AbstractLifeCycle.java[setFailed]:212) - FAILED 
SslContextFactory@8f2098e(/home/firearrow/zeppelin/incubator-zeppelin/conf/null,/home/firearrow/zeppelin/incubator-zeppelin/conf/null):
 java.io.FileNotFoundException: 
/home/firearrow/zeppelin/incubator-zeppelin/conf/null (No such file or 
directory)```

With this fix the correct path is resolved
### 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/FireArrow/zeppelin ZEPPELIN-1810]

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

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






---
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-1811) Enable SSL by default

2016-12-14 Thread Lee moon soo (JIRA)

Lee moon soo created ZEPPELIN-1811:
--

 Summary: Enable SSL by default
 Key: ZEPPELIN-1811
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1811
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Lee moon soo


How about make Zeppelin use SSL by default?

One possible approach is generate keystore, truststore on the first run if 
necessary command line tool (key-tools, openssl) is available on the system. 
Otherwise fallback to non-ssl with print warning message.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] zeppelin issue #1761: [ZEPPELIN-1711] Create Released Binary Zeppelin Docker...

2016-12-14 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1761
  
\cc @bzz @mfelgamal @astroshim 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 issue #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-14 Thread AlexanderShoshin

Github user AlexanderShoshin commented on the issue:

https://github.com/apache/zeppelin/pull/1758
  
Yes, but ZeppelinHub does not show all the paragraphs. I will try to find 
out the reason.


---
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: DependencyResolver with absolute path as localRepoPath

2016-12-14 Thread moon

Hi Igor Drozdov,

Taking care of absolute path and relative path correctly make sense,
Thanks for investigating the issue!

Thanks,
moon

On Wed, Dec 14, 2016 at 6:33 AM Igor Drozdov  wrote:

> Hello,
>
> I'm investigating issue
> https://issues.apache.org/jira/browse/ZEPPELIN-1797 (first case, "Cannot
> fetch dependencies for com.databricks:spark-csv_2.10:1.3.0")
>
> The reason is than
> org.apache.zeppelin.dep.Booter#newRepositorySystemSession method is unable
> to use absolute path for local repo. It creates broken path for it.
>
> Underlying exception is "Specified destination directory cannot be
> created:
> C:\dev\projects\zeppelin\zeppelin-interpreter\..\C:\Users\IGOR_D~1\AppData\Local\Temp\ZeppelinLTest_1481724731009\test-repo\com\databricks\spark-csv_2.10\1.3.0".
> As you can see, it appends absolute path to the end of ZEPPELIN_HOME.
>
> Huge amount of tests passes absolute path to local repo to
> DependencyResolver.
>
> I suggest modification to this method to handle absolute paths properly:
> When we pass absolute path it should use it for local repo
> When we pass relative path it should resolve it as relative to
> ZEPPELIN_HOME (current behavior)
>
> What do you think? Is it safe?
>
> Thank you
> Igor Drozdov
>

[GitHub] zeppelin pull request #1761: [ZEPPELIN-1711] Create Released Binary Zeppelin...

2016-12-14 Thread 1ambda

GitHub user 1ambda opened a pull request:

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

[ZEPPELIN-1711] Create Released Binary Zeppelin Docker Images

### What is this PR for?

This PR

- added Dockerfiles for released zeppelin binaries (0.6.0, 0.6.1, 0.6.2)
- refactored Dockerfiles `scripts/docker/zeppelin-base` so that we can keep 
small size base images
- added missing R, python packages to run tutorials properly
- also included `scripts/docker/create-dockerfile.sh` to create docker 
images for newly released zeppelin binaries

 For Reviewers

I have things to be discussed for this PR

1. **Support other linux (centos) or not:** Alpine linux is designed for 
light weight OS so it doesn't have graphic devices. But usually people use 
zeppelin to render graph like things. In this case Alpine is not enough
2. **Extracting script for installing required packages into 
`testing/install_external_dependencies.sh` or not:** Alpine linux has it's own 
command to install packages like `apk add`. I think this is not reusable. And 
extracting scripts from Dockerfile causes other problems. For example, we need 
to add script just for building base images for python, r to copy scripts for 
external dependencies because Docker command `COPY` and `ADD` doens't support 
relative paths. It means, we have to copy external dependency scripts before 
building into the path where Dockerfile is built.
3. **Adding Binary Zeppelin Image to 
[docker-library/official-images](https://github.com/docker-library/official-images/)**:
 I haven't experiences about this.

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

### What is the Jira issue?

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

### How should this be tested?

 Building Base Images

```bash
$ cd scripts/docker/zeppelin-base/alpine/java
$ docker build . -t zeppelin:alpine-base_java

$ cd scripts/docker/zeppelin-base/alpine/python
$ docker build . -t zeppelin:alpine-base_python

$ cd scripts/docker/zeppelin-base/alpine/r
$ docker build . -t zeppelin:alpine-base_r
```

 Building Zeppelin Images

```bash
$ scripts/docker/zeppelin-bin-all/alpine/0.6.2_java
$ docker build . -t zeppelin:alpine-0.6.2_java

$ scripts/docker/zeppelin-bin-all/alpine/0.6.2_python
$ docker build . -t zeppelin:alpine-0.6.2_python

$ scripts/docker/zeppelin-bin-all/alpine/0.6.2_r
$ docker build . -t zeppelin:alpine-0.6.2_r

# build 0.6.1, 0.6.0 images too
```

 Running Zeppelin Images

Make sure you have zeppelin docker images before running containers

```bash
$ docker images
REPOSITORY   TAG   IMAGE IDCREATED  
   SIZE
zeppelin alpine-base_java  5a24572968d65 hours 
ago 145 MB
```

Then, running containers by replacing tags (0.6.2, 0.6.1, 0.6.0)

```
$ docker run -it --name zeppelin --rm -p 8080:8080 -p 7077:7077 
zeppelin:alpine-0.6.2_java
$ docker run -it --name zeppelin --rm -p 8080:8080 -p 7077:7077 
zeppelin:alpine-0.6.2_python
$ docker run -it --name zeppelin --rm -p 8080:8080 -p 7077:7077 
zeppelin:alpine-0.6.2_r

# run 0.6.1, 0.6.0 also
```

 Testing Zeppelin Tutorials

Here are things you need to know before testing.

- `zeppelin:alpine-$TAG_java` images can run **Zeppelin Tutorial** since it 
has openjdk7
- `zeppelin:alpine-$TAG_python` images can run **Zeppelin Tutorial: Python 
- matplotlib basic** as well as **Zeppelin Tutorial** since it has python 
related packages
- `zeppelin:alpine-$TAG_r` images can run **R Tutorial** as well as 
**Zeppelin Tutorial** since it has python related packages
- Since alpine linux doesn't have graphical device, some function may not 
work (e.g plot in R). Some R examples throw invalid images like chunked.png
- For the same reason, we have to change some code of **Python - matplotlib 
basic** before testing otherwise it will throw tkinter errors in the python 
interpreter

```python
// before
%python
import numpy as np
import matplotlib.pyplot as plt

// should be
%python
import numpy as np
import matplotlib
matplotlib.use('Agg')
import matplotlib.pyplot as pet
```

Then, run each zeppelin container and run tutorial while replacing tags by 
accessing localhost:8080 in your browsers

```
# run: Zeppelin Tutorial
$ docker run -it --name zeppelin --rm -p 8080:8080 -p 7077:7077 
zeppelin:alpine-0.6.2_java

# run: Zeppelin Tutorial, Python - matplotlib basic
# and modify the code mentioned above before running paragraphs
$ docker run -it --name zeppelin --rm -p 8080:8080 -p 7077:7077 
zeppelin:alpine-0.6.2_python

[jira] [Created] (ZEPPELIN-1810) Keystore and truststore fails from config file

2016-12-14 Thread David Ersson (JIRA)

David Ersson created ZEPPELIN-1810:
--

 Summary: Keystore and truststore fails from config file
 Key: ZEPPELIN-1810
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1810
 Project: Zeppelin
  Issue Type: Bug
  Components: conf
Affects Versions: 0.7.0
Reporter: David Ersson


The changes in the fix for https://issues.apache.org/jira/browse/ZEPPELIN-1319 
introduced a bug which makes setting path to keystore and truststore impossible.
The reason seems to be an incorrect use of getString() here
https://github.com/apache/zeppelin/pull/1319/files#diff-69b17249ac9d265091d730130d973f4aR282
 
and here
https://github.com/apache/zeppelin/pull/1319/files#diff-69b17249ac9d265091d730130d973f4aR311

There should be no getString there, just "path" as it is.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] zeppelin pull request #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl...

2016-12-14 Thread cuspymd

GitHub user cuspymd reopened a pull request:

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

[ZEPPELIN-1239] Google WebFont: goog-webfont-dl Grunt task warning and 
build error

### What is this PR for?
It is to fix a zeppelin-web build error in Windows 10.
To fix it, I changed google webfont download module of grunt.

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

### Todos


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

### How should this be tested?
zeppelin-web build in Windows 10

### Screenshots (if appropriate)

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

'spawn UNKNOWN' error occurs on zeppelin-web build processing in windows 10.
It due to the function 'grunt.util.spawn()' which is called by the module 
'grunt-goog-webfont-dl'.
So I changed this module to 'grunt-google-fonts'.

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

$ git pull https://github.com/cuspymd/zeppelin google-fonts

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

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


commit 1fea68fe08b06f7c5215dc54c5601d52d9386bbf
Author: Myoungdo Park 
Date:   2016-12-06T16:12:59Z

Change google font downloader to fix a build error in win 7

Curruntly 'spawn UNKNOWN' error occurs on zeppelin-web build processing in 
windows 7.
It due to the function 'grunt.util.spawn()' which is called by the module 
'grunt-goog-webfont-dl'.
So I changed this module to 'grunt-google-fonts'.




---
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 #1756: [ZEPPELIN-1804] Added Link this paragraph to keyboard ...

2016-12-14 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1756
  
Tested and 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.
---

DependencyResolver with absolute path as localRepoPath

2016-12-14 Thread Igor Drozdov

Hello,

I'm investigating issue https://issues.apache.org/jira/browse/ZEPPELIN-1797 
(first case, "Cannot fetch dependencies for 
com.databricks:spark-csv_2.10:1.3.0")

The reason is than org.apache.zeppelin.dep.Booter#newRepositorySystemSession 
method is unable to use absolute path for local repo. It creates broken path 
for it.

Underlying exception is "Specified destination directory cannot be created: 
C:\dev\projects\zeppelin\zeppelin-interpreter\..\C:\Users\IGOR_D~1\AppData\Local\Temp\ZeppelinLTest_1481724731009\test-repo\com\databricks\spark-csv_2.10\1.3.0".
 As you can see, it appends absolute path to the end of ZEPPELIN_HOME.

Huge amount of tests passes absolute path to local repo to DependencyResolver.

I suggest modification to this method to handle absolute paths properly:
When we pass absolute path it should use it for local repo
When we pass relative path it should resolve it as relative to ZEPPELIN_HOME 
(current behavior)

What do you think? Is it safe?

Thank you
Igor Drozdov

[GitHub] zeppelin pull request #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl...

2016-12-14 Thread cuspymd

Github user cuspymd closed the pull request at:

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


---
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-1809) Paragraph title is not correctly displayed in "link this paragraph" url.

2016-12-14 Thread Lee moon soo (JIRA)

Lee moon soo created ZEPPELIN-1809:
--

 Summary: Paragraph title is not correctly displayed in "link this 
paragraph" url.
 Key: ZEPPELIN-1809
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1809
 Project: Zeppelin
  Issue Type: Bug
Reporter: Lee moon soo


paragraph title is not correctly displayed in "Link this paragraph" url. see 
attached screenshot.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Created] (ZEPPELIN-1808) Keyboard shortcut should not work in "Link this paragraph" url

2016-12-14 Thread Lee moon soo (JIRA)

Lee moon soo created ZEPPELIN-1808:
--

 Summary: Keyboard shortcut should not work in "Link this 
paragraph" url
 Key: ZEPPELIN-1808
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1808
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.6.2
Reporter: Lee moon soo


Keyboard shortcut should not work in "Link this paragraph" url.




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] zeppelin issue #1745: ZEPPELIN-1785. Remove ZEPPELIN_NOTEBOOK_DIR and ZEPPEL...

2016-12-14 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1745
  
Thanks @zjffdu for the fix!
LGTM and merge to master if there're no more 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 #1756: [ZEPPELIN-1804] Added Link this paragraph to keyboard ...

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1756
  
@tae-jun Sorry for many reopened PR. 
I had a weird CI problem like the link 
(https://travis-ci.org/apache/zeppelin/builds/183881539). But I solved with 
recommit PR. Now, CI fail again :crying_cat_face: 


---
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 #1758: [ZEPPELIN-1787] Add an example of Flink Notebook

2016-12-14 Thread tae-jun

Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1758
  
[Link for the note on 
ZeppelinHub](https://www.zeppelinhub.com/viewer/notebooks/aHR0cHM6Ly9yYXcuZ2l0aHVidXNlcmNvbnRlbnQuY29tL0FsZXhhbmRlclNob3NoaW4vemVwcGVsaW4tZXhhbXBsZS9tYXN0ZXIvbm90ZWJvb2tzL0ZsaWdodHMuanNvbg)

For convenience :)

@AlexanderShoshin Is this link 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 #1756: [ZEPPELIN-1804] Added Link this paragraph to keyboard ...

2016-12-14 Thread tae-jun

Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1756
  
OMG hope that it passes this time... :)


---
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 #1759: [ZEPPELIN-922] New interpreter registration mechanism ...

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1759
  
@DrIgor Thanks for your contribution!

Currently 1 test profile failed like below, but it looks irrelevant with 
this change/
```

Results :

Failed tests: 
  SparkParagraphIT.testDep:234 First paragraph from SparkParagraphIT of 
testDep status: 
Expected: "FINISHED"
 but: was "ERROR"

Tests in error: 
  
ZeppelinIT.testSparkInterpreterDependencyLoading:234->AbstractZeppelinIT.waitForParagraph:70->AbstractZeppelinIT.pollingWait:96
 Â» Timeout

Tests run: 18, Failures: 1, Errors: 1, Skipped: 0

```

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 #1760: Update DESCRIPTION

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1760
  
@fred777 Thanks for your contribution. Nice catch! 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 #1730: [ZEPPELIN-1736] Introduce trash & enable removing fold...

2016-12-14 Thread tae-jun

Github user tae-jun commented on the issue:

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

@AhyoungRyu Always thanks for your nice review ð

Yes, it's intended. A time stamp is generated only if a `folder` already
exists in the trash. But it's not generated when moving a `note` to the trash.

I made this feature because of a **use case** like this:
### A use case needs time stamp
1. A user moves "NoteDirA" folder to the trash
2. And the user create a folder which has the same name ("NoteDirA")
3. After a month from then, the user forgot the existence of the folder
already exists in the trash, and moves the folder to the trash which created
later
4. But it was a mistake and the user wants to restore the folder created
later

### Necessity of the time stamp for the use case above

**If there is no time stamp:**
The folder created later will be merged into old one and the folder
"NoteDirA" will be messy.

If there is a time stamp:
The folder created later will be moved to the trash with a time stamp
postfix. There will be two folders which are "NoteDirA" (old one) and "NoteDirA
removed at 2016-12-14 xx:xx:xx" (new one). In this case, the user can restore
the folder successfully.

This is why I added time stamp! :-) Please tell me if there is any other
idea.

### Remove permanently icon candidates (?)
I agree with you so I searched Font Awesome and found some candidates for
the icon. I think not only `empty trash` but also `Remove note permanently` and
`Remove folder permanently` should be replaced together.

![image](https://cloud.githubusercontent.com/assets/8201019/21180874/543a32b0-c23e-11e6-9f5f-25a0fa25bb84.png)

![image](https://cloud.githubusercontent.com/assets/8201019/21180952/c8ce421a-c23e-11e6-9297-f7df93a36ef5.png)

![image](https://cloud.githubusercontent.com/assets/8201019/21180965/e9a051ea-c23e-11e6-96b8-7df5469a87cf.png)

![image](https://cloud.githubusercontent.com/assets/8201019/21181004/179fe736-c23f-11e6-9e58-36ade60959c4.png)

The icons above are the candidates I found, but please let me know if you
have a better one!

Which icon do you prefer? ð

Thanks for the review 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-1807) DependencyResolverTest.testLoad \w spark-csv

2016-12-14 Thread Igor Drozdov (JIRA)

Igor Drozdov created ZEPPELIN-1807:
--

 Summary: DependencyResolverTest.testLoad \w spark-csv
 Key: ZEPPELIN-1807
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1807
 Project: Zeppelin
  Issue Type: Sub-task
Reporter: Igor Drozdov
Assignee: Igor Drozdov


{code}
org.apache.zeppelin.dep.DependencyResolverTest.testLoad(org.apache.zeppelin.dep.DependencyResolverTest)
  Time elapsed: 0.633 sec  <<< ERROR!
org.sonatype.aether.RepositoryException: Cannot fetch dependencies for 
com.databricks:spark-csv_2.10:1.3.0
at 
org.apache.zeppelin.dep.DependencyResolver.getArtifactsWithDep(DependencyResolver.java:181)
at 
org.apache.zeppelin.dep.DependencyResolver.loadFromMvn(DependencyResolver.java:131)
at 
org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:79)
at 
org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:96)
at 
org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:88)
at 
org.apache.zeppelin.dep.DependencyResolverTest.testLoad(DependencyResolverTest.java:77)

Results :

Tests in error: 
  DependencyResolverTest.testLoad:77 » Repository Cannot fetch dependencies for 
...
{code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] zeppelin pull request #1760: Update DESCRIPTION

2016-12-14 Thread fred777

GitHub user fred777 opened a pull request:

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

Update DESCRIPTION

### What is this PR for?
This PR just fixes a typo in R package DESCRIPTION-File for misspelled 
googleVis dependency.

### What type of PR is it?
Documentation

### 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/fred777/zeppelin patch-1

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

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


commit dc8993f3f4de1d9efbf300ad942f4b2e52dd7940
Author: fred777 
Date:   2016-12-14T11:13:28Z

Update DESCRIPTION

Fix typo for suggested googleVis dependency




---
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 #1756: [ZEPPELIN-1804] Added Link this paragraph to ke...

GitHub user soralee reopened a pull request:

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

[ZEPPELIN-1804] Added Link this paragraph to keyboard shortcut

### What is this PR for?
Each paragraph menu, it has "Link this paragraph" function which is link 
each of the paragraph but there is not shortcut key. 

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

### Todos
* None

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

### How should this be tested?
Please press Ctrl+Alt+w shortcut key and check work as expected well

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/8110458/21171777/7520eb68-c211-11e6-8498-ae22e99a547a.png)


![z1804_1](https://cloud.githubusercontent.com/assets/8110458/21171586/5b2d88d4-c210-11e6-820a-bc9d40d4a2c1.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/soralee/zeppelin ZEPPELIN-1804

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

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


commit c3c3915b785598e86e8a882879ad9e9c36922290
Author: soralee 
Date:   2016-12-14T05:45:30Z

add a shortcut key of Link this paragraph




---
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 #1756: [ZEPPELIN-1804] Added Link this paragraph to ke...

Github user soralee closed the pull request at:

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


---
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 #1756: [ZEPPELIN-1804] Added "Link this paragraph" to ...

Github user soralee closed the pull request at:

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


---
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 #1756: [ZEPPELIN-1804] Added "Link this paragraph" to ...

GitHub user soralee reopened a pull request:

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

[ZEPPELIN-1804] Added "Link this paragraph" to keyboard shortcut

### What is this PR for?
Each paragraph menu, it has "Link this paragraph" function which is link 
each of the paragraph but there is not shortcut key. 

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

### Todos
* None

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

### How should this be tested?
Please press Ctrl+Alt+w shortcut key and check work as expected well

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/8110458/21171777/7520eb68-c211-11e6-8498-ae22e99a547a.png)


![z1804_1](https://cloud.githubusercontent.com/assets/8110458/21171586/5b2d88d4-c210-11e6-820a-bc9d40d4a2c1.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/soralee/zeppelin ZEPPELIN-1804

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

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


commit c3c3915b785598e86e8a882879ad9e9c36922290
Author: soralee 
Date:   2016-12-14T05:45:30Z

add a shortcut key of Link this paragraph




---
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 #1753: [ZEPPELIN-1594] Supported personalized mode

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1753
  
Hello, @jongyoul.
What do you think on this mockup?


![pmode](https://cloud.githubusercontent.com/assets/10525473/21179014/7d37a6ec-c235-11e6-9ea4-ab10c57570b7.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 pull request #1756: [ZEPPELIN-1804] Added "Link this paragraph" to ...

Github user soralee closed the pull request at:

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


---
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 #1756: [ZEPPELIN-1804] Added "Link this paragraph" to ...

GitHub user soralee reopened a pull request:

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

[ZEPPELIN-1804] Added "Link this paragraph" to keyboard shortcut

### What is this PR for?
Each paragraph menu, it has "Link this paragraph" function which is link 
each of the paragraph but there is not shortcut key. 

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

### Todos
* None

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

### How should this be tested?
Please press Ctrl+Alt+w shortcut key and check work as expected well

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/8110458/21171777/7520eb68-c211-11e6-8498-ae22e99a547a.png)


![z1804_1](https://cloud.githubusercontent.com/assets/8110458/21171586/5b2d88d4-c210-11e6-820a-bc9d40d4a2c1.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/soralee/zeppelin ZEPPELIN-1804

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

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


commit c3c3915b785598e86e8a882879ad9e9c36922290
Author: soralee 
Date:   2016-12-14T05:45:30Z

add a shortcut key of Link this paragraph




---
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 #1716: [ ZEPPELIN-531 ] execute and stop note in job m...

Github user cloverhearts closed the pull request at:

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


---
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 #1716: [ ZEPPELIN-531 ] execute and stop note in job m...

GitHub user cloverhearts reopened a pull request:

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

[ ZEPPELIN-531 ] execute and stop note in job menu

### What is this PR for?
We can note start and stop in job menu.


### What type of PR is it?
Improvement

### Todos
- [x] - add feature for run/stop

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

### How should this be tested?
Click to a note start or stop button in job menu.

### Screenshots (if appropriate)

![executenote](https://cloud.githubusercontent.com/assets/10525473/20823821/ec8a39c8-b89a-11e6-91f2-0eddc4bcffd9.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/cloverhearts/zeppelin jobm/executeNote

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

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


commit 23abb3ae025ef7b6ba9339b9cd383969adb6e526
Author: CloverHearts 
Date:   2016-12-02T04:38:51Z

execute and stop note in job menu

commit f40b12fe3aaf0429fc9501cc2b32cf039d83a773
Author: CloverHearts 
Date:   2016-12-05T07:25:49Z

Merge branch 'master' into jobm/executeNote

commit d3bd76e6255a4c20766785ff186934cf0d601c64
Author: CloverHearts 
Date:   2016-12-07T13:40:02Z

change `if` syntanx style

commit 24d7ced861dbc770b0bc182cfa4ba0aff0b67c4d
Author: CloverHearts 
Date:   2016-12-07T13:42:33Z

Merge branch 'master' into jobm/executeNote

# Conflicts:
#   zeppelin-web/src/app/jobmanager/jobs/job.controller.js

commit b7f205171835ea025aed7c6f22ad189eff1042e1
Author: CloverHearts 
Date:   2016-12-07T14:13:45Z

remove whitespace

commit e00f9ce581716b24ccb2b76095982ff9ff71d5a7
Author: CloverHearts 
Date:   2016-12-08T03:05:46Z

apply code style

commit b6fb3819392c17ae911f7ffcc419d19997dbc091
Author: CloverHearts 
Date:   2016-12-08T03:36:36Z

change string check style

commit f00b7388cf589ab418f3a41e723f1cbfb2503e9f
Author: CloverHearts 
Date:   2016-12-14T07:06:19Z

Merge branch 'master' into jobm/executeNote




---
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 #1756: [ZEPPELIN-1804] Added "Link this paragraph" to ...

GitHub user soralee reopened a pull request:

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

[ZEPPELIN-1804] Added "Link this paragraph" to keyboard shortcut

### What is this PR for?
Each paragraph menu, it has "Link this paragraph" function which is link 
each of the paragraph but there is not shortcut key. 

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

### Todos
* None

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

### How should this be tested?
Please press Ctrl+Alt+w shortcut key and check work as expected well

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/8110458/21171777/7520eb68-c211-11e6-8498-ae22e99a547a.png)


![z1804_1](https://cloud.githubusercontent.com/assets/8110458/21171586/5b2d88d4-c210-11e6-820a-bc9d40d4a2c1.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/soralee/zeppelin ZEPPELIN-1804

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

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


commit c3c3915b785598e86e8a882879ad9e9c36922290
Author: soralee 
Date:   2016-12-14T05:45:30Z

add a shortcut key of Link this paragraph




---
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 #1756: [ZEPPELIN-1804] Added "Link this paragraph" to ...

Github user soralee closed the pull request at:

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


---
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 #1751: ZEPPELIN-1786. Refactor LivyHelper

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1751
  
@prabhjyotsingh @bzz @felixcheung Please help review.

The failed test is not relevant.

```
16/12/14 08:08:03 INFO RemoteActorRefProvider$RemotingTerminator: Remote 
daemon shut down; proceeding with flushing remote transports.
Results :
Tests in error: 
  DepInterpreterTest.testDefault:96 Â» NullPointer
```


---
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 #1759: [ZEPPELIN-922] New interpreter registration mechanism ...

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1759
  
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 pull request #1756: [ZEPPELIN-1804] Added "Link this paragraph" to ...

GitHub user soralee reopened a pull request:

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

[ZEPPELIN-1804] Added "Link this paragraph" to keyboard shortcut

### What is this PR for?
Each paragraph menu, it has "Link this paragraph" function which is link 
each of the paragraph but there is not shortcut key. 

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

### Todos
* None

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

### How should this be tested?
Please press Ctrl+Alt+w shortcut key and check work as expected well

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/8110458/21171777/7520eb68-c211-11e6-8498-ae22e99a547a.png)


![z1804_1](https://cloud.githubusercontent.com/assets/8110458/21171586/5b2d88d4-c210-11e6-820a-bc9d40d4a2c1.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/soralee/zeppelin ZEPPELIN-1804

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

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


commit c3c3915b785598e86e8a882879ad9e9c36922290
Author: soralee 
Date:   2016-12-14T05:45:30Z

add a shortcut key of Link this paragraph




---
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 #1756: [ZEPPELIN-1804] Added "Link this paragraph" to ...