[GitHub] zeppelin issue #1408: [ZEPPELIN-808] Changed Permissions on Notebooks do not...

2016-09-13 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1408
  
@kavinkumarks this looks better and tested locally, works as expected. few 
things: 
* would be nice to update screenshot above
* CI build fails, could you take a look into that or re-trigger it? 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-13 Thread kavinkumarks
Github user kavinkumarks commented on the issue:

https://github.com/apache/zeppelin/pull/1393
  
Thanks @Leemoonsoo for reviewing this! Could you please merge this?

- Kavin
MailTo: kavin.ku...@imaginea.com


---
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 #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...

2016-09-13 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1330
  
@Leemoonsoo issues are created under 
[ZEPPELIN-1437](https://issues.apache.org/jira/browse/ZEPPELIN-1437) and 
[ZEPPELIN-1438](https://issues.apache.org/jira/browse/ZEPPELIN-1438) 
correspondingly 


---
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-1438) Per user note list broadcast on note permission change

2016-09-13 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-1438:
-

 Summary: Per user note list broadcast on note permission change
 Key: ZEPPELIN-1438
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1438
 Project: Zeppelin
  Issue Type: Bug
Reporter: Khalid Huseynov
 Fix For: 0.7.0


after https://github.com/apache/zeppelin/pull/1330 is merged, users will be 
able to receive their own list of notes, however that list is currently loaded 
automatically only once when starting Zeppelin (also possible by clicking 
reload notes).

Thus, we need to address updating other users list of notes (workbench) on the 
fly when some note permissions are changed. For example if user1 gave read 
permissions to noteA for user2, noteA should appear immediately in user2's 
working directory of notes. 



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


[jira] [Created] (ZEPPELIN-1437) Note list broadcast is not aware of user session

2016-09-13 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-1437:
-

 Summary: Note list broadcast is not aware of user session
 Key: ZEPPELIN-1437
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1437
 Project: Zeppelin
  Issue Type: Bug
Reporter: Khalid Huseynov
 Fix For: 0.7.0


after https://github.com/apache/zeppelin/pull/1330 each user will receive his 
specific list, but if user refreshes his list to reload from storage, that list 
will be broadcasted to everyone since broadcast messages are still not aware of 
user sessions.



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


[GitHub] zeppelin issue #1423: ZEPPELIN-1425. sparkr.zip is not distributed to execut...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1423
  
Thanks for the explanation.
LGTM and merge 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 #1411: [ZEPPELIN-1417] Bugfix of "Connect to existing process...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1411
  
LGTM and merge 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 #1416: Rewrite of "Quick Start" page.

2016-09-13 Thread rfilmyer
Github user rfilmyer commented on the issue:

https://github.com/apache/zeppelin/pull/1416
  
Alright. I'm probably going to make some more changes on this PR. (Probably 
2 or 3 more pages).


---
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 #1418: [gh-pages][ZEPPELIN-990] Add header anchor to gh-pages...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1418
  
Merge if there're no further discussions.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1426
  
I think displaying result with additional information is quick and simple 
approach. However, i think there're some downsides like
  - can not leverage output streaming 
  - when user output want to use different zeppelin display system, like 
%table, %html, etc, it may be conflict with the feature this PR trying to 
provide.

I think better approach would be [support multiple interpreter result]( 
https://issues.apache.org/jira/browse/ZEPPELIN-212) and display additional 
informations as a one of separate interpreter result with it's own result type.

Although it'll going to be very large work, i think resolving ZEPPELIN-212 
and build this feature on top of it, is right approach.






---
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 #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1384
  
Thanks @1ambda. Looks great to me.
Merge if there're no further discussions.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1423: ZEPPELIN-1425. sparkr.zip is not distributed to execut...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1423
  
@zjffdu Do you think 
[rpkg.zip](https://github.com/apache/spark/blob/v1.6.2/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala#L394)
 is the something also need to be taken care?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1425: Improving Matplotlib Integration with Zeppelin

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1425
  
Thanks @IshmeetKaur. I think i'll try to merge #1422 first because #1422 
arrived earlier.



---
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 #1422: Allow matplotlib plot display in pyspark context

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1422
  
Could you try indent code with 2 spaces like other part of codes do?



---
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 #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1330
  
@mwkang @khalidhuseynov Could you handle or at least create issues for 
expected problems after merge this PR? I can at least think following two 
issues,

- Note list broadcasting message is not aware user session, so notebook 
list can display notes that supposed to not see when other user refresh the 
notebook list.
- When a user changes the permission of a note, note list need to be 
broadcasted immediately so all connected users can have updated list of 
notebook. Otherwise, there's still some chances that a user able to see some 
note on notebook list that they can not access.



---
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 #1393: [ZEPPELIN-1142] Zeppelin allows two users to simultane...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1393
  
Thanks @kavinkumarks for the contribution.
This changes not only resolving race condition described in ZEPPELIN-1142 
but also changes user experience. i.e.

Before, owner is not defined when creating note.
After, owner is set to current user when creating note. (when 
authentication is enabled)

Especially with code changes such as 
https://github.com/apache/zeppelin/pull/1330, user will experience bigger 
behavior changes.

But i think having current user as a owner when creating note is natural 
behavior that most people can think, even though it brings some UX changes.

LGTM and merge if there're no further discussions.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1413: [MINOR] Format Fix for InterpreterResultTest.ja...

2016-09-13 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1404: ZEPPELIN-1411. UDF with pyspark not working - object h...

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1404
  
Thanks @zjffdu. I think second ci test profile failure looks relevant.

```
ests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 120.231 sec 
<<< FAILURE! - in org.apache.zeppelin.rest.ZeppelinSparkClusterTest
pySparkTest(org.apache.zeppelin.rest.ZeppelinSparkClusterTest)  Time 
elapsed: 3.842 sec  <<< FAILURE!
java.lang.AssertionError: expected: but was:
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:743)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at 
org.apache.zeppelin.rest.ZeppelinSparkClusterTest.pySparkTest(ZeppelinSparkClusterTest.java:150)
```

Could you check? 


---
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 #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-13 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
@astroshim @corneadoug, Shell we have some unittests for `public void 
restart(String settingId, String noteId)` before we merge? Restarting only a 
interpreter instance/process for a note is new ability. So i think it make 
sense to have a unittest.

Also, shell we have 
https://github.com/apache/zeppelin/blob/master/docs/rest-api/rest-interpreter.md#restart-an-interpreter
 updated, too?


---
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 #1427: [ZEPPELIN-1435] Move zeppelin-server-*.jar unde...

2016-09-13 Thread minahlee
GitHub user minahlee opened a pull request:

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

[ZEPPELIN-1435] Move zeppelin-server-*.jar under lib in distribution binary 
package

### What is this PR for?
Move zeppelin-server-*.jar package under `lib` directory like other 
zeppelin modules such as `zeppelin-engine`, `zeppelin-interpreter`

### What type of PR is it?
Refactoring

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

### How should this be tested?
Build with:
```
mvn clean package -DskipTests -Pbuild-distr -pl 
'!alluxio,!angular,!cassandra,!elasticsearch,!file,!flink,!hbase,!ignite,!jdbc,!kylin,!lens,!livy,!postgresql,!python,!shell,!bigquery'
```

and check if `zeppelin-server-0.7.0-SNAPSHOT.jar` file is placed under 
`zeppelin-distribution/target/zeppelin-0.7.0-SNAPSHOT/zeppelin-0.7.0-SNAPSHOT/lib`
 not 
`zeppelin-distribution/target/zeppelin-0.7.0-SNAPSHOT/zeppelin-0.7.0-SNAPSHOT`

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



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

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

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

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


commit 03a018f7dbb6f43ab72a71735e0e1203e716f73a
Author: Mina Lee 
Date:   2016-09-13T13:42:03Z

Move zeppelin-server-*.jar under lib in distribution binary package




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (ZEPPELIN-1435) Move zeppelin-server-*.jar under lib directory in distribution package

2016-09-13 Thread Mina Lee (JIRA)
Mina Lee created ZEPPELIN-1435:
--

 Summary: Move zeppelin-server-*.jar under lib directory in 
distribution package
 Key: ZEPPELIN-1435
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1435
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Mina Lee
Assignee: Mina Lee
 Fix For: 0.7.0


This issue has brought up by [~jonathak] in comment on 
https://issues.apache.org/jira/browse/ZEPPELIN-1250.

Every zeppelin-*.jar packages are placed under lib except 
zeppelin-server-*.jar. It makes more sense to keep all zeppelin-*.jar packages 
in same place.



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


[GitHub] zeppelin pull request #1425: Improving Matplotlib Integration with Zeppelin

2016-09-13 Thread IshmeetKaur
Github user IshmeetKaur closed the pull request at:

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


---
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 #1425: Improving Matplotlib Integration with Zeppelin

2016-09-13 Thread IshmeetKaur
GitHub user IshmeetKaur reopened a pull request:

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

Improving Matplotlib Integration with Zeppelin

### What is this PR for?
This PR improves the matplotlib integration with zeppelin.

### What type of PR is it?
Improvement

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

### How should this be tested?
show() functionality in the PyZeppelinContext class should work with 
matplotlib.

### 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/IshmeetKaur/zeppelin master

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

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


commit 1e3a5088011c0cca4844404d5edc87d18f34b517
Author: Ishmeet Kaur 
Date:   2016-09-06T14:44:16Z

Show function modification pyspark

commit 561860a2321409c596397e2eb3366ff7c1820e51
Author: Ishmeet Kaur 
Date:   2016-09-07T15:52:35Z

show function modification pyspark

commit 6491ff2839e2e1a94fbbbd30727bf821cdb69805
Author: Ishmeet Kaur 
Date:   2016-09-07T19:52:44Z

show function modification

commit 645e6af18dd754a0ebcbe5ff040fd0a9d2346de7
Author: Ishmeet Kaur 
Date:   2016-09-08T14:04:01Z

Merge branch 'master' of https://github.com/apache/zeppelin into modify_show

commit 207d6dd0dbe79f66b34088ff96db1fa8835ddb61
Author: Ishmeet Kaur 
Date:   2016-09-08T14:31:59Z

Revert "show function modification pyspark"

This reverts commit 561860a2321409c596397e2eb3366ff7c1820e51.

commit d42d49b0b820aa24024677b7910b40e7025efebd
Author: Ishmeet Kaur 
Date:   2016-09-08T14:42:58Z

show function modification

commit 3224184c51886ea286bd41dc9c9c34b09d9bffe9
Author: Ishmeet Kaur 
Date:   2016-09-08T15:45:59Z

combining show and show_plot

commit 0faf2aa2b3b979f352a7a1f8f449d47a24651a89
Author: Ishmeet Kaur 
Date:   2016-09-09T08:06:55Z

dataframe pandas dependency

commit 1d3583a72918c905a783d32145b4bea3826a637b
Author: Ishmeet Kaur 
Date:   2016-09-10T09:22:10Z

pandas dataframe dependency resolution

commit cd3e111383b261b1def068e8ec9fbacf73132fe9
Author: Ishmeet Kaur 
Date:   2016-09-10T15:50:13Z

Dataframe pandas module resolution

commit 3ee2555885ed5ea872cd6ef52d43ab44d0107418
Author: ishmeetkaur 
Date:   2016-09-11T22:04:39Z

dataframe function removed

commit 4ff14c1fe089c76f64ad3092ead819bebb80
Author: ishmeetkaur 
Date:   2016-09-11T22:14:37Z

Merge remote-tracking branch 'upstream/master' into modify_show

commit 3e152fc675b31c00b957974d50c2710134bea490
Author: ishmeetkaur 
Date:   2016-09-12T13:10:47Z

Merge remote-tracking branch 'origin/master' into modify_show

commit 9dc5dc01662e5c89b02a93cb5b4483c236e374d9
Author: ishmeetkaur 
Date:   2016-09-12T13:24:35Z

removing whitespace

commit d991fd511c69134ffaa1251f802aababa42d66f5
Author: IshmeetKaur 
Date:   2016-09-12T14:54:23Z

Merge pull request #1 from IshmeetKaur/modify_show

Modify show




---
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-1434) Long numbers rounding in Zeppelin-Web table

2016-09-13 Thread David Tzoor (JIRA)
David Tzoor created ZEPPELIN-1434:
-

 Summary: Long numbers rounding in Zeppelin-Web table
 Key: ZEPPELIN-1434
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1434
 Project: Zeppelin
  Issue Type: Bug
Reporter: David Tzoor
Priority: Critical


zeppelin-web is using {{Handsontable}} package to generate the table view 
within the notebooks' paragraphs.
In the [paragraph 
controller|https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/notebook/paragraph/paragraph.controller.js],
 within the Handsontable object instantiation, the {{cells}} function checks if 
the given value is a number (using {{!isNan(value)}} method). If the value is 
indeed a number, it will render as follows:
{code:javascript}
else if (!isNaN(value)) {
cellProperties.format = '0,0.[0]';
td.style.textAlign = 'left';
Handsontable.renderers.NumericRenderer.apply(this, arguments);
}
{code}
Since javascript supports numbers between {{-(2^53^ - 1)}} and {{(2^53^ - 1)}}, 
any number not within this range, will be rounded (up or down) and wrong 
numbers will be shown. I suggest to add another condition to check whether the 
number is a safe number:
{code:javascript}
Number.isSafeInteger(parseInt(value))
{code}
In that case, just render the value as a string (no formatting).

The second issue is with very small floats. The current rendering will only 
show up to 5 decimal places. There should be another condition to check if the 
value is a float, and if so use parseFloat or Number(value) to render it with 
the e notation: 
{code:javascript}
function isFloat(a) {
  return a.indexOf('.') !== -1;
}
...
if (isFloat(value)) {
parseFloat(value);
}
{code}






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


[GitHub] zeppelin issue #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1398
  
@zjffdu It's okay :) 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.
---


[jira] [Created] (ZEPPELIN-1433) What dependency to add to project (sbt or maven) get hold of ZeppelinContext?

2016-09-13 Thread sam (JIRA)
sam created ZEPPELIN-1433:
-

 Summary: What dependency to add to project (sbt or maven) get hold 
of ZeppelinContext?
 Key: ZEPPELIN-1433
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1433
 Project: Zeppelin
  Issue Type: Improvement
Reporter: sam


Zeppelin has an object `ZeppelinContext`, which can then be used to share state 
between languages and bind variables to angular and thus create cool user 
interfaces inside Zeppelin notebooks.

We have written numerous convenience methods to create things like drop down 
menus, buttons, UI stuff, from scala.  These methods call `ZeppelinContext`. We 
wish to add these methods to an sbt project, so that we can package them in a 
jar, but it seems the Zeppelin project provides no artifact that contains 
`ZeppelinContext` (we have tried many).

Rather there only seems to exist two work arounds:

1. Build all of Zeppelin and add the resulting jar as an unmanaged jar (not 
nice).
2. Use duck typing (also really not nice).

Is there a lesser known resolver / artifact id to get hold of this type??

If there is no such publically accessible module please could such a module be 
made available?



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


[GitHub] zeppelin pull request #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread zjffdu
GitHub user zjffdu reopened a pull request:

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

[MINOR] Doc fix for spark interpreter

### What is this PR for?
Several changes on doc of spark interpreter. 

* %spark, %sql, %pyspark only works when spark is the default interpreter 
group of note. So I update the doc to use the full interpreter name.
* Add SparkSession for 2.0
* Also add comments inline with other changes to explain the reason. 


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

### Todos
* [ ] - Task

### What is the Jira issue?
* No jira created. 

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

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

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


commit 40d4b11e29130fd8256af5c7848a1c7e7013c793
Author: Jeff Zhang 
Date:   2016-09-02T09:30:43Z

[MINOR] Doc fix for spark interpreter




---
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 #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1398
  
Sorry @AhyoungRyu  I didn't notice you PR is based on my branch and thought 
you will merge to master directly. 


---
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 #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1398
  
@zjffdu Did you close this PR on purpose? 


---
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 #1398: [MINOR] Doc fix for spark interpreter

2016-09-13 Thread zjffdu
Github user zjffdu closed the pull request at:

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


---
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 #1404: ZEPPELIN-1411. UDF with pyspark not working - object h...

2016-09-13 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1404
  
@Leemoonsoo , I updated the unit test. And also made a little change 
`AbstractTestRestApi`, the standalone way doesn't work for me. So I allow user 
to export `SPAKP_MASTER` to run it in other modes. 


---
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 #1412: [Zeppelin 1415] - Adding Zeppelin-Web Good Prac...

2016-09-13 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-13 Thread corneadoug
Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
Merging if there is 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 #1426: ZEPPELIN-1430. Display appId and webui link in LivyInt...

2016-09-13 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1426
  
Here is before and after of this PR that I observe, and I see these 2 
changes

 - now it does not stream output, i.e. it waits for the entire paragraph 
execution to be finished, and returns result.
 - and I see empty line between "1" and "2" where I wrote "Thread.sleep()", 
not a big deal, but it wasn't earlier.

Before:

![master](https://cloud.githubusercontent.com/assets/674497/18464749/8cb27b7e-79b0-11e6-91d8-cfbdf8a8af1a.gif)
After:

![with-1430](https://cloud.githubusercontent.com/assets/674497/18464750/8cb63746-79b0-11e6-8e41-3fd58586c541.gif)



---
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 #1424: [ZEPPELIN-1426] User aware storage sync

2016-09-13 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1424
  
Looks good to me.

Will merge 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 #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...

2016-09-13 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1330
  
Thank you @mwkang @khalidhuseynov !

Will merge 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 pull request #1426: ZEPPELIN-1430. Display appId and webui link in ...

2016-09-13 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1430. Display appId and webui link in LivyInterpreter's output

### What is this PR for?
For now, it is hard to figure out what the yarn application of the livy 
session represent, it would be better to display the appId and webui link in 
the output of LivyInterpreter for diagnosing purpose. It can also be applied to 
the native SparkInterpreter, but could be done in another ticket. 

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

### Todos
* [ ] - Task

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

### How should this be tested?
Tested manually

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/1846/e4eab580-79bb-11e6-8c8d-393ab6805638.png)

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



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

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

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

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


commit 6fe78b0dd1222d5d1b87b8a2b192aa94df989a90
Author: Jeff Zhang 
Date:   2016-09-13T06:07:02Z

ZEPPELIN-1430. Display appId and webui link in LivyInterpreter's output




---
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 #1330: [ZEPPELIN-1144]Zeppelin home page should only list not...

2016-09-13 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1330
  
I think it doesn't solve all problems, but definitely in the roadmap. 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.
---