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

2016-12-06 Thread conker84
Github user conker84 commented on the issue:

https://github.com/apache/zeppelin/pull/1582
  
Sure, i'm working on it!
Thanks!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. 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-1762) How to reconfigure zeppelin to use SSL port 636 for ldap instead of port 398

2016-12-06 Thread Bandhu Gupta (JIRA)
Bandhu Gupta created ZEPPELIN-1762:
--

 Summary: How to reconfigure zeppelin to use SSL port 636 for ldap 
instead of port 398
 Key: ZEPPELIN-1762
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1762
 Project: Zeppelin
  Issue Type: Bug
Reporter: Bandhu Gupta






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


[GitHub] zeppelin pull request #1714: [ZEPPELIN-1743] Use explicit arguments in funct...

2016-12-06 Thread felizbear
Github user felizbear closed the pull request at:

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


---
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 #1714: [ZEPPELIN-1743] Use explicit arguments in funct...

2016-12-06 Thread felizbear
GitHub user felizbear reopened a pull request:

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

[ZEPPELIN-1743] Use explicit arguments in functions in 
paragraph.controller.js

### What is this PR for?
This is `paragraph.controller.js` maintenance PR. It refactors most 
functions in paragraph controller to accept explicit parameters instead of 
getting data from `$scope`. It is a first - simple and safe - step in making 
paragraph controller more maintainable.

### What type of PR is it?
Refactoring

### Todos
* [x] - Call functions explicitly with parameters when makes sense
* [x] - Do minor refactoring when makes sense

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

### How should this be tested?
Make sure that paragraph works as expected by interacting with it;

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

$ git pull https://github.com/felizbear/zeppelin fe-explicit-arguments

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

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


commit 13d314071de5373daa699e7eb0c50b4846f8eeba
Author: felizbear 
Date:   2016-12-01T08:04:31Z

make arguments explicit in initializeDefault

commit d25060ba1dfc299b2053f946d5f48be7405402d7
Author: felizbear 
Date:   2016-12-01T08:08:06Z

make arguments explicit in isRunning

commit abe56ed8c9f849acaf18287bcca44a95df25ca4e
Author: felizbear 
Date:   2016-12-01T08:09:42Z

make arguments explicit in cancelParagraph

commit c2939d0fd21ab95e670fbfe8908deaf2b58e471b
Author: felizbear 
Date:   2016-12-01T08:18:03Z

make arguments explicit in saveParagraph

commit 20d66e565ed1effbe063b00fe5d58a2610e6c461
Author: felizbear 
Date:   2016-12-01T08:20:32Z

make arguments explicit in toggleEnableDisable

commit 99d23cf680d8bd71e777986e554d57da36d579bf
Author: felizbear 
Date:   2016-12-01T08:29:42Z

make arguments explicit in run

commit e9e584e44fb648688ebddb7a1de5b9becf5b5670
Author: felizbear 
Date:   2016-12-01T08:33:29Z

make arguments explicit for moveUp and moveDown

commit 7bbb7dc7e1801ea334ba8b2c24dc610751e88b1f
Author: felizbear 
Date:   2016-12-01T08:43:40Z

make arguments explicit in insertNew

commit d807e6ee760f4cc3890877ca3b471eaa46ff68ee
Author: felizbear 
Date:   2016-12-01T08:45:47Z

make arguments explicit in removeParagraph

commit c383a2f92a1c097d66a8d0902b16f18d60f7be2f
Author: felizbear 
Date:   2016-12-01T08:46:59Z

make arguments explicit in clearParagraphOutput

commit 414fd9e7839ff2375354960109897940b43b1b22
Author: felizbear 
Date:   2016-12-01T08:50:38Z

make arguments explicit in toggleEditor

commit 62cb81a11e702347fff55b490139a267daa90a88
Author: felizbear 
Date:   2016-12-01T08:52:02Z

make arguments explicit in open / close table

commit 83f3b0f6a35d4e687725dfa95193e16f53396867
Author: felizbear 
Date:   2016-12-01T08:54:37Z

make arguments explicit in various open / close editor / table nonsense
functions

commit 58dcf3ab352bff5ac93a95dbcc45d997837618d8
Author: felizbear 
Date:   2016-12-01T09:09:54Z

make arguments explicit in show / hide / set title

commit b44972acc57381e9b67a8e5cee2cbb9b5eb452f7
Author: felizbear 
Date:   2016-12-01T09:11:29Z

make arguments explicit in show / hide lineNumbers

commit 41772acd5e71df4a1952ecc1626bc33082273021
Author: felizbear 
Date:   2016-12-01T09:22:31Z

make arguments explicit in changeColWidth

commit 569368c144245ae853f101c58fc796d9a0da2a90
Author: felizbear 
Date:   2016-12-01T09:23:17Z

remove unused function toggleGraphOption

commit ba9a429f0a669242470f972801c4217f73d381db
Author: felizbear 
Date:   2016-12-01T09:24:32Z

make arguments explicit in toggleOutput

commit 3315b3c99268081ff277d27b59295e5b3cef5e92
Author: felizbear 
Date:   2016-12-01T09:28:11Z

make arguments explicit in aceChanged

commit 40ca7d18a9a7c2873f3a169ff8182835a59aff5d
Author: felizbear 
Date:   2016-12-01T09:31:24Z

make arguments explicit in getEditorSetting

commit 9af71a087aa44ff6a7bef3edb46d855d82c99eb4
Author: felizbear 
Date:   2016-12-01T09:35:58Z

make arguments explicit in autoAdjustEditorHeight

commit a9ddba6f8c3ed2843abc876b7eb0439f42530608
Author: felizbear 
Date:   2016-12-01T09:44:16Z

refactor getProgress

commit 620a6e0260d1430c524469fff81fd7416aa0a88d
Author: felizbear 
Date:   

[GitHub] zeppelin pull request #1719: [ZEPPELIN-965] missing lastExecuteTime function

2016-12-06 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-06 Thread cuspymd
Github user cuspymd commented on the issue:

https://github.com/apache/zeppelin/pull/1730
  
@tae-jun Yes. Good Idea~!!


---
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 #1612: ZEPPELIN-1607. Refactor Livy Interpreter to ada...

2016-12-06 Thread zjffdu
Github user zjffdu closed the pull request at:

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


---
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 #1612: ZEPPELIN-1607. Refactor Livy Interpreter to ada...

2016-12-06 Thread zjffdu
GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1607. Refactor Livy Interpreter to adapt scope mode

### What is this PR for?
ZEPPELIN-1606 will make scoped mode as the default interpreter mode of 
livy. This PR is to refactor livy interpreter to adapt the scoped mode. 
Besides, it also fix several bugs of livy interpreter.  like ZEPPELIN-1516. 
Besides that I also enable livy integration test in travis to make sure the 
refactoring correct. 


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

### Todos
* Move session creation to open()
* Other refactoring in livy interpreter, especially LivyHelper.java

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

### How should this be tested?
Travis is also enabled for livy integration test. 

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

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

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


commit 798de1bb55eb161132d4a9a92679374d6de6f031
Author: Jeff Zhang 
Date:   2016-11-03T06:21:33Z

ZEPPELIN-1607. Refactor Livy Interpreter to adapt scope mode

commit 995ddf36273aace625e45215d453052111772fdc
Author: Jeff Zhang 
Date:   2016-11-08T08:29:17Z

enable travis for livy integration test

commit 2ce15327e35e9dbf8a5a976a313656868efe647d
Author: Jeff Zhang 
Date:   2016-11-08T09:11:10Z

add licence header

commit e99be8d45147704d662dced31c6f4b8ec745e892
Author: Jeff Zhang 
Date:   2016-11-08T09:12:25Z

update .travis.yml

commit 6c9795f3d9cca2a6a39eff1c4b20dffdfe8e31f4
Author: Jeff Zhang 
Date:   2016-11-08T09:44:30Z

update test

commit f7e131f66170d68637ee9d8b366358dcf4ecb08a
Author: Jeff Zhang 
Date:   2016-11-29T00:12:31Z

change version from 2.3 to 2.6 in .travis.yml




---
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 #1727: [ZEPPELIN-1695] Use shared versions in test lib...

2016-12-06 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl Grunt ...

2016-12-06 Thread cuspymd
Github user cuspymd commented on the issue:

https://github.com/apache/zeppelin/pull/1731
  
@bzz Thanks for your comment. The file 'google-fonts.css' is created during 
zeppelin-web build. 
You need to run `npm run 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 #1538: [ZEPPELIN-1386] Docker images for running Apache Zeppe...

2016-12-06 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1538
  
@mfelgamal could you please explain, why do you think one more script is 
needed - `start-zeppelin.sh` and why `CMD["/bin/zeppelin.sh"]` can not be used 
as entrypoint for the image?

@1ambda has the point, and we should try to reduce the number of shell 
scripts that need to be supported later on, as well as possible issues with 
setting up classpath, etc.



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

2016-12-06 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1716
  
@cloverhearts thank you for contribution! In my experience, if PR author or 
one of reviewers post the reason for CI failure - that speeds up the 
review\merge process dramatically.

CI is failing with flaky pySpark tests, not relevant to the PR

```
Results :

Failed tests: 
  ZeppelinSparkClusterTest.pySparkDepLoaderTest:430 expected: but 
was:
  ZeppelinSparkClusterTest.pySparkAutoConvertOptionTest:312 
expected: but was:

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

It will be handled under 
[ZEPPELIN-1761](https://issues.apache.org/jira/browse/ZEPPELIN-1761).

\cc @felizbear to review the frontend part.


---
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 #1731: [ZEPPELIN-1239] Google WebFont: goog-webfont-dl Grunt ...

2016-12-06 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1731
  
👍  Idea sounds great to me. It looks like 
https://github.com/optimalisatie/grunt-goog-webfont-dl is not maintained but 
https://github.com/Mika-/grunt-google-fonts had changes this year.

I tried locally in dev mode (\wo maven) `npm i; npm run start` and the 
fonts were not loaded - could you fix that please?
![screen shot 2016-12-07 at 08 54 
27](https://cloud.githubusercontent.com/assets/5582506/20949157/499f94c8-bc5b-11e6-979f-6c45586e3974.png)

Same on current master works as expected (fonts are loaded).



CI is failing on single profile \w flaky test 
`ZeppelinSparkClusterTest.pySparkDepLoaderTest`
It will be handled separatly under 
[ZEPPELIN-1761](https://issues.apache.org/jira/browse/ZEPPELIN-1761).




---
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-1761) CI: pySparkDepLoaderTest fails the build

2016-12-06 Thread Alexander Bezzubov (JIRA)
Alexander Bezzubov created ZEPPELIN-1761:


 Summary: CI: pySparkDepLoaderTest fails the build
 Key: ZEPPELIN-1761
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1761
 Project: Zeppelin
  Issue Type: Bug
Reporter: Alexander Bezzubov


Test in {{zeppelin-server}} fails

{code}
Results :

Failed tests: 
  ZeppelinSparkClusterTest.pySparkDepLoaderTest:430 expected: but 
was:

Tests run: 82, Failures: 1, Errors: 0, 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]
{code}

Failures:
 - PR [#1731|https://github.com/apache/zeppelin/pull/1731], 
[logs|https://api.travis-ci.org/jobs/181725932/log.txt?deansi=true]



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


[GitHub] zeppelin issue #1727: [ZEPPELIN-1695] Use shared versions in test libraries ...

2016-12-06 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1727
  
Looks great to me, thank you @1ambda for valuable build improvements!

Merging to master if there is no further discussion.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project 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...

2016-12-06 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1582
  
I reviewed these changes again and didn't see why they would cause that 
error.

Could you rebase this PR and check our my comment (which should simplify 
the diff).

These changes would be very good to have in the coming release; let's try 
this again - if test fails again you could close and reopen this PR to trigger 
CI runs manually.


---
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-1760) Option to disable auto-formatting and type detection in table displays

2016-12-06 Thread Everett Anderson (JIRA)
Everett Anderson created ZEPPELIN-1760:
--

 Summary: Option to disable auto-formatting and type detection in 
table displays
 Key: ZEPPELIN-1760
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1760
 Project: Zeppelin
  Issue Type: Wish
  Components: front-end, GUI
Affects Versions: 0.6.2
Reporter: Everett Anderson


Support disabling auto-format/type detection of table displays.

ZEPPELIN-1371 added auto-format/type detection table displays. For example, 
when Zeppelin believes a given column contains numbers (even if it were, say, a 
Spark DataFrame string column) it displays these in the UI with commas.

However, this can be misleading in some workflows. For example, when using 
Zeppelin to evaluate a table produced by an extraction pipeline for a 
fixed-width or CSV file, it can be important to know how many characters were 
in the field. Maybe more commonly, sometimes columns are not numbers but codes.

It'd be great to have the ability to configure column type detection and 
display format.

Disabling automatic type detection/format globally or per interpreter in 
configuration would be a first step. At the moment, I don't believe there's a 
way to see the "true" column values in a table.



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


[GitHub] zeppelin issue #1730: [ZEPPELIN-1736] Introduce trash & enable removing fold...

2016-12-06 Thread tae-jun
Github user tae-jun commented on the issue:

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

![image](https://cloud.githubusercontent.com/assets/8201019/20929471/8897948a-bc0d-11e6-941b-4123bc09c8a7.png)
It failed only at Selenium test twice. I guess it's because I **changed the 
action of remove note button** of notebook view to moving to trash.

But I have no idea where to fix! 😢 Please help!


---
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-06 Thread tae-jun
Github user tae-jun commented on the issue:

https://github.com/apache/zeppelin/pull/1730
  
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 #1689: [ZEPPELIN-1709] Update checkstyle rule file, plugin

2016-12-06 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1689
  
@bzz Let me know if there is anything to do before merging this :) 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 #1730: [ZEPPELIN-1736] Introduce trash & enable removing fold...

2016-12-06 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/1730
  
Let me review and give some feedbacks :) Thanks for implementing the useful 
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 #1727: [ZEPPELIN-1695] Use shared versions in test libraries ...

2016-12-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1727
  
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 #1692: [ZEPPELIN-1710 - WIP] Ace Editor upgrade

2016-12-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1692
  
LGTM and merge to master if there're no more discussions.
Great work @cloverhearts !


---
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 #1723: [ZEPPELIN-1747] Fix Korean notename input probl...

2016-12-06 Thread asfgit
Github user asfgit closed the pull request at:

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


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

2016-12-06 Thread tae-jun
GitHub user tae-jun reopened a pull request:

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

[ZEPPELIN-1736] Introduce trash & enable removing folder

### What is this PR for?

![image](https://cloud.githubusercontent.com/assets/8201019/20925378/03a722f8-bbfa-11e6-9afc-c62cfe6b0e2a.png)
This PR introduces the trash! (which is essential, I think ^^;)

I added basic operations of the trash:
 Added operations of trash
* `Move` a folder to the trash
* `Move` a note to the trash
* `Restore` a folder
* `Restore` a note
* `Remove` a folder in the trash for good
* `Remove` a note in the trash for good
* `Empty` the trash
* `Restore all` in the trash
* `Move` a note to the trash from the notebook view
* `Remove` a note to the trash for good from the notebook view
* If there is a folder that has the same name, suffix current data rather 
than just merging(please see screenshot below!)


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

### Future work
Maybe it would be better if notebook view notices that a note is in the 
trash.

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

### How should this be tested?
* Build this PR
* Move some folders or notes to the trash
* Restore them
* Empty the trash
* Restore all notes in the trash
* Remove folders or notes in the trash
* Remove a folder which has the same folder name in the trash

### Screenshots (if appropriate)
## Move, restore, remove a note from the main page

![note_from_main](https://cloud.githubusercontent.com/assets/8201019/20925760/13b0d91c-bbfc-11e6-94c5-2211c9d9a282.gif)

## Move, restore, remove a folder from the main page

![folder_test](https://cloud.githubusercontent.com/assets/8201019/20925774/2884abde-bbfc-11e6-9ba9-7b1f8a935a41.gif)

## Move, remove a note from the notebook view

![noteview_test](https://cloud.githubusercontent.com/assets/8201019/20925837/83564162-bbfc-11e6-9a92-c6bc1984b30d.gif)

## Restore, empty the trash

![restore_empty_trash](https://cloud.githubusercontent.com/assets/8201019/20925841/8f6d78d0-bbfc-11e6-90a9-6f67c95371a5.gif)

## Suffix current date if the folder exists in the trash

![same_folder_name_test](https://cloud.githubusercontent.com/assets/8201019/20926085/e8cb6954-bbfd-11e6-9cdf-acf9b785d833.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/tae-jun/zeppelin ZEPPELIN-1736

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

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


commit 665e30227a903b067b9b3d4e887f603bedf61698
Author: Jun Kim 
Date:   2016-12-06T11:21:38Z

Introduce trash & removing folder

commit 8d53a1347028b09588027fa8715ea99d23abb65b
Author: Jun Kim 
Date:   2016-12-06T12:54:54Z

Satisfy style check

commit 7caebdaee036a3276d6c116aaa0307466fe3c1d0
Author: Jun Kim 
Date:   2016-12-06T12:55:19Z

Remove if note is in the trash from the notebook view




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

2016-12-06 Thread tae-jun
Github user tae-jun closed the pull request at:

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


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

2016-12-06 Thread tae-jun
GitHub user tae-jun opened a pull request:

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

[ZEPPELIN-1736] Introduce trash & enable removing folder

### What is this PR for?

![image](https://cloud.githubusercontent.com/assets/8201019/20925378/03a722f8-bbfa-11e6-9afc-c62cfe6b0e2a.png)
This PR introduces the trash! (which is essential, I think ^^;)

I added basic operations of the trash:
 Added operations of trash
* `Move` a folder to the trash
* `Move` a note to the trash
* `Restore` a folder
* `Restore` a note
* `Remove` a folder in the trash for good
* `Remove` a note in the trash for good
* `Empty` the trash
* `Restore all` in the trash
* `Move` a note to the trash from the notebook view
* `Remove` a note to the trash for good from the notebook view
* If there is a folder that has the same name, suffix current data rather 
than just merging(please see screenshot below!)


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

### Future work
Maybe it would be better if notebook view notices that a note is in the 
trash.

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

### How should this be tested?
* Build this PR
* Move some folders or notes to the trash
* Restore them
* Empty the trash
* Restore all notes in the trash
* Remove folders or notes in the trash
* Remove a folder which has the same folder name in the trash

### Screenshots (if appropriate)
## Move, restore, remove a note from the main page

![note_from_main](https://cloud.githubusercontent.com/assets/8201019/20925760/13b0d91c-bbfc-11e6-94c5-2211c9d9a282.gif)

## Move, restore, remove a folder from the main page

![folder_test](https://cloud.githubusercontent.com/assets/8201019/20925774/2884abde-bbfc-11e6-9ba9-7b1f8a935a41.gif)

## Move, remove a note from the notebook view

![noteview_test](https://cloud.githubusercontent.com/assets/8201019/20925837/83564162-bbfc-11e6-9a92-c6bc1984b30d.gif)

## Restore, empty the trash

![restore_empty_trash](https://cloud.githubusercontent.com/assets/8201019/20925841/8f6d78d0-bbfc-11e6-90a9-6f67c95371a5.gif)

## Suffix current date if the folder exists in the trash

![same_folder_name_test](https://cloud.githubusercontent.com/assets/8201019/20926085/e8cb6954-bbfd-11e6-9cdf-acf9b785d833.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/tae-jun/zeppelin ZEPPELIN-1736

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

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


commit 665e30227a903b067b9b3d4e887f603bedf61698
Author: Jun Kim 
Date:   2016-12-06T11:21:38Z

Introduce trash & removing folder




---
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-1759) Zeppelin NoteBook Error logs are truncated

2016-12-06 Thread Prabhu Joseph (JIRA)
Prabhu Joseph created ZEPPELIN-1759:
---

 Summary: Zeppelin NoteBook Error logs are truncated
 Key: ZEPPELIN-1759
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1759
 Project: Zeppelin
  Issue Type: Bug
  Components: Core
Affects Versions: 0.6.2
Reporter: Prabhu Joseph
 Attachments: TruncatedErrorMessage.png, TruncatedErrorMessage_1.png

Error thrown from Spark Sql interpreter is truncated on Zeppelin NoteBook. For 
example: Below Spark program which tries to access people1.txt from hdfs /tmp 
which is not present will fail and error message will be thrown. Top of the 
error message is truncated.

{code}
%spark


val people = sc.textFile("hdfs:///tmp/people1.txt")
val schemaString = "name age"
import org.apache.spark.sql.Row;
import org.apache.spark.sql.types.{StructType,StructField,StringType};
val schema =StructType(schemaString.split(" ").map(fieldName => 
StructField(fieldName, StringType, true)))
val rowRDD = people.map(_.split(",")).map(p => Row(p(0), p(1).trim))
val peopleDataFrame = sqlContext.createDataFrame(rowRDD, schema)
peopleDataFrame.registerTempTable("people")
val results = sqlContext.sql("SELECT name FROM people")
results.map(t => "Name: " + t(0)).collect().foreach(println)
{code}

Truncated Error Message is attached. On removing some blank lines from the 
paragraph, the error message is shown like the attached image full error 
message.









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


[GitHub] zeppelin issue #1721: [ZEPPELIN-969] order by note name in job menu

2016-12-06 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1721
  
@soralee Thank you!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have 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 #1721: [ZEPPELIN-969] order by note name in job menu

2016-12-06 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1721
  
@cloverhearts It works well as expected. 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.
---