[GitHub] zeppelin issue #1376: [ZEPPELIN-449] Enhance log messages when interpreter i...

[lresende]

Github user lresende commented on the issue:

https://github.com/apache/zeppelin/pull/1376
  
@bzz Are we merging this ? Or should i close 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 #1432: [HOT FIX][ZEPPELIN-1144] Fix compilation errors in Not...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1432
  
Please ignore my previous comments, just realize this is for fix the 
compilation issue. 


---
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 #1432: [HOT FIX][ZEPPELIN-1144] Fix compilation errors in Not...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1432
  
@prabhjyotsingh @AhyoungRyu This make the compilation fails, which affect 
other PRs. Could you help do a quick fix on it ?
https://travis-ci.org/apache/zeppelin/jobs/160614161


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

[zjffdu]

Github user zjffdu closed the pull request at:

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


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

[zjffdu]

GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1411. UDF with pyspark not working - object has no attribute 
'parseDataType'

### What is this PR for?
The root cause is that SQLContext's signature changes in spark 2.0. 
Spark 1.6
```
def __init__(self, sparkContext, sqlContext=None):
```
Spark 2.0
```
def __init__(self, sparkContext, sparkSession=None, jsqlContext=None):
```
So we need to create SQLContext using named parameters, otherwise it would 
take intp.getSQLContext() as sparkSession which cause the issue.

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

### Todos
* [ ] - Task

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

### How should this be tested?
Tested using the example code in ZEPPELIN-1411. 

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18260139/9bd702c0-741d-11e6-8b23-946c38a794c3.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-1411

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

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


commit 296c63fdf44edda96cbe12928213d8a39eabd33c
Author: Jeff Zhang 
Date:   2016-09-06T02:26:05Z

ZEPPELIN-1411. UDF with pyspark not working - object has no attribute 
'parseDataType'

commit 4fe033d3e700ff9571da78761f21c6e88689ae55
Author: Jeff Zhang 
Date:   2016-09-09T06:21:37Z

add unit test

commit 4922de1d46f625cbec97a4529ac2c6ca78c75d07
Author: Jeff Zhang 
Date:   2016-09-14T06:24:36Z

log more logging for travis CI diangnose

commit a142d4589ec8238270ea107b356a6864ffa66e7f
Author: Jeff Zhang 
Date:   2016-09-14T07:42:39Z

retry




---
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 #1432: [HOT FIX][ZEPPELIN-1144] Fix compilation errors in Not...

[prabhjyotsingh]

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1432
  
Tested on local, 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 #1432: [HOT FIX][ZEPPELIN-1144] Fix compilation errors in Not...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1432
  
Only Selenium test failed now. I'm merging this as a hotfix.


---
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 #1432: [HOT FIX][ZEPPELIN-1144] Fix compilation errors in Not...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1432
  
@khalidhuseynov @Leemoonsoo Thanks for the quick verification. Then will 
merge this one as a hotfix after CI checking! 


---
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 #1432: [HOT FIX][ZEPPELIN-1144] Fix compilation errors in Not...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1432
  
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 #1432: [HOT FIX][ZEPPELIN-1144] Fix compilation errors...

[AhyoungRyu]

GitHub user AhyoungRyu opened a pull request:

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

[HOT FIX][ZEPPELIN-1144] Fix compilation errors in Notebook.java

### What is this PR for?
After #1330 merged, the latest master build failed with below compilation 
errors.

```
[ERROR] COMPILATION ERROR :
[INFO] -
[ERROR] 
/Users/ahyoungryu/Dev/zeppelin/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Notebook.java:[553,31]
 cannot find symbol
  symbol:   method id()
  location: variable note1 of type org.apache.zeppelin.notebook.Note
[ERROR] 
/Users/ahyoungryu/Dev/zeppelin/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Notebook.java:[557,31]
 cannot find symbol
  symbol:   method id()
  location: variable note2 of type org.apache.zeppelin.notebook.Note
```


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

### What is the Jira issue?

### How should this be tested?
 - Build the latest master branch with `mvn clean package -DskipTests` -> 
compilation error in `zeppelin-zengine`

 - Apply this patch and build with `mvn clean package -DskipTests` -> build 
success 
You can also check #1330 works properly.

### 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 hotfix/ZEPPELIN-1144

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

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


commit 6a3dbd3a33a4a0ffe2614946b243dad63c78e241
Author: AhyoungRyu 
Date:   2016-09-17T02:42:36Z

Fix build error in Notebook.java




---
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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1431
  
@astroshim Good catch and looks better! 
Can we make "A" and "B" have same space? Need more space to "A" :)
https://cloud.githubusercontent.com/assets/10060731/18605350/976be588-7cca-11e6-9f61-33a3295563cc.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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1431
  
@astroshim I think it was handled already in #1395. This is what I can see 
in the latest master branch :)
https://cloud.githubusercontent.com/assets/10060731/18605251/a5a8a3f0-7cc7-11e6-9df3-41f28bdfae01.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.
---

Re: [Need-Help][Eclipse][Windows] Problem

[moon soo Lee]

Hi Andrea,

I think you can try

mvn -pl 'zeppelin-server' exec:java
-Dexec.mainClass="org.apache.zeppelin.server.ZeppelinServer" -Dexec.args=""

from project root directory (not zeppelin-server)

Thanks,
moon

On Thu, Sep 15, 2016 at 5:43 AM Andrea Santurbano  wrote:

> Hi to all,
> i'm trying to execute Zeppelin via Eclipse, using this steps:
> 1) *mvn clean install -DskipTests *(in Zeppelin root)
> 2) *cd zeppelin-server*
> 3) *mvn exec:java
> -Dexec.mainClass="org.apache.zeppelin.server.ZeppelinServer"
> -Dexec.args=""*
>
> But i get the following exception:
> *java.io.FileNotFoundException: .\conf\shiro.ini (The system cannot find
> the path specified)*
>
> Where am i wrong?
> Is there some issue with Windows?
>
> I'm developing an interpreter and i need to debug both java and javascript
> code. How i can proceed?
>
> Thanks
> Andrea
>

Re: How to start Zeppelin By IDE

[moon soo Lee]

Could you try

mvn [YOUR BUILD PROFILES] package eclipse:eclipse

Thanks,
moon

On Wed, Sep 14, 2016 at 4:17 AM Andrea Santurbano  wrote:

> Hi to all,
> i have some issue on executing
> mvn eclipse:eclipse
>
> Here is log error:
> https://gist.github.com/conker84/2c779224f4fb2212077a81d387fc0ffe
>
> Please can someone help me?
> Thanks
> Andrea
>
> Il giorno mar 6 set 2016 alle ore 18:17 Jongyoul Lee 
> ha scritto:
>
> > Hello,
> >
> > I'm using IntelliJ and there's my step below.
> >
> > 1. mvn clean pakcage -DskipTest
> > 2. Debug ZeppelinServer
> >
> > Regards,
> > Jongyoul
> >
> > On Fri, Sep 2, 2016 at 6:02 AM, moon soo Lee  wrote:
> >
> > > Hi,
> > >
> > > Eclipse, i used to create eclipse project file by running "mvn
> > > eclipse:eclipse".
> > > IDEA, just importing maven project and it usually works without
> problem.
> > >
> > > You can also check
> > >
> http://zeppelin.apache.org/docs/latest/development/howtocontribute.html#
> > > integrated-development-environment-ide
> > > http://zeppelin.apache.org/docs/latest/development/
> > > howtocontributewebsite.html
> > >
> > > Thanks,
> > > moon
> > >
> > >
> > > On Fri, Sep 2, 2016 at 6:58 AM Hefei Li  wrote:
> > >
> > > > Hi,all:
> > > >
> > > > I want to try start Zeppelin server in development mode by IDE
> > > (Eclipse
> > > > or IDEA.).
> > > >
> > > > Now, can you give me some help and advice！
> > > > --
> > > >
> > > > =
> > > >
> > > > Hefei Li
> > > >
> > > > MP: +86 18701581473
> > > > MSN: lh...@live.cn
> > > > =
> > > >
> > >
> >
> >
> >
> > --
> > 이종열, Jongyoul Lee, 李宗烈
> > http://madeng.net
> >
>

Re: Exporting Spark paragraphs as Spark Applications

[Jeremy Anderson]

Luciano, I think this would be a terrific feature. I've heard the exact
same workflow you've describe in all of the research we've done.

...

Jeremy Anderson
Founder, Object Adjective
415.493.8489
jer...@objectadjective.com
objectadjective.com 



This email and any files transmitted with it are confidential and
intended solely for the use of the individual or entity to whom they are
addressed.

On 16 September 2016 at 12:19, Luciano Resende  wrote:

> While talking with a few different users, I have been seeing the use case
> of using iterative development in Notebooks or Spark Shell and then copying
> and pasting the final solution to a formal application repeating itself
> very often.
>
> I was wondering if an "Export Spark Paragraphs as a Spark Application
> (jar)" would be a feature that Zeppelin community would think it's useful.
> But keep in mind there are some limitation here : we would be constrained
> to Spark related paragraphs, etc...  but even so, I think there are
> multiple scenarios where I see that the ability to have an application that
> directly runs on Spark to be very useful.
>
> If the community is interested, let's use this thread to discuss any
> specific requirements or suggestions that others might have, and after a
> few days I would like to start prototyping this functionality.
>
> Thoughts ?
>
>
>
> --
> Luciano Resende
> http://twitter.com/lresende1975
> http://lresende.blogspot.com/
>

Exporting Spark paragraphs as Spark Applications

[Luciano Resende]

While talking with a few different users, I have been seeing the use case
of using iterative development in Notebooks or Spark Shell and then copying
and pasting the final solution to a formal application repeating itself
very often.

I was wondering if an "Export Spark Paragraphs as a Spark Application
(jar)" would be a feature that Zeppelin community would think it's useful.
But keep in mind there are some limitation here : we would be constrained
to Spark related paragraphs, etc...  but even so, I think there are
multiple scenarios where I see that the ability to have an application that
directly runs on Spark to be very useful.

If the community is interested, let's use this thread to discuss any
specific requirements or suggestions that others might have, and after a
few days I would like to start prototyping this functionality.

Thoughts ?



-- 
Luciano Resende
http://twitter.com/lresende1975
http://lresende.blogspot.com/

[jira] [Created] (ZEPPELIN-1448) Add CPU time to paragraph output for %pyspark, %spark, %python, and %sql interpreters

[Jennifer Jones (JIRA)]

Jennifer Jones created ZEPPELIN-1448:


 Summary: Add CPU time to paragraph output for %pyspark, %spark, 
%python, and %sql interpreters
 Key: ZEPPELIN-1448
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1448
 Project: Zeppelin
  Issue Type: New Feature
  Components: Interpreters, pySpark, python-interpreter, 
zeppelin-interpreter, zeppelin-server, zeppelin-zengine
Affects Versions: 0.6.1
 Environment: Mac OS X El Capitan Version 10.11.6, Spark 2.0.0, 
Zeppelin 0.6.1, and the Anaconda distribution of Python 3.5.2.
Reporter: Jennifer Jones
 Fix For: 0.6.2, 0.7.0, 0.6.1


In Zeppelin, when using the PySpark interpreter (%pyspark) in a cell or 
"paragraph," I want the output to list the CPU time in addition to the 
evaluated output and the actual (physical) time elapsed.

A specific example of a statement I want to time (not necessarily limited to 
sql queries) is something like this:

%pyspark
...
sqlctx = SQLContext(sc)
...
sqlctx.sql("SELECT feature1, feature2, feature3 FROM tableName " + 
   "WHERE feature3 = 'a' LIMIT 100").show()

or a sql count over the total number of rows in the table.

If I use  Zeppelin with the Hive interpreter in a paragraph (%hive), the output 
automatically includes actual time, CPU time, and the evaluated output.

Similarly, if I use IPython (either in a shell or in a Jupyter notebook), I can 
preface a statement with %time to have the output returned along with the CPU 
time.

Please add the capability to return CPU time in a (PySpark, etc.) paragraph in 
a Zeppelin notebook. 




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

[jira] [Created] (ZEPPELIN-1447) Auto completion as Ctrl = instead of tab completion is causing a lot of problems; please change to tab completion (high priority)

[Jennifer Jones (JIRA)]

Jennifer Jones created ZEPPELIN-1447:


 Summary: Auto completion as Ctrl = instead of tab completion is 
causing a lot of problems; please change to tab completion (high priority)
 Key: ZEPPELIN-1447
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1447
 Project: Zeppelin
  Issue Type: Improvement
  Components: interpreter-auto-completion, pySpark, python-interpreter, 
zeppelin-server
Affects Versions: 0.6.1
 Environment: Mac OS X El Capitan Version 10.11.6, Spark 2.0.0, 
Zeppelin 0.6.1, and the Anaconda distribution of Python 3.5.2.
Reporter: Jennifer Jones
 Fix For: 0.6.2, 0.7.0, 0.6.1


Operating system defaults and most, if not all, browsers use Ctrl = as the 
default zoom in shortcut.

OS defaults can be changed so that Ctrl = is no longer the default zoom, but 
the browsers (Firefox, Safari) override the system defaults with their default 
Ctrl = shortcut for zoom in. Furthermore, there is not a straightforward and 
easy way to change the default shortcuts for zoom in Firefox or Safari browsers.

This means auto completion in Zeppelin is currently unusable. (which is very 
sad :( ) 

Please change to tab completion, for Spark, PySpark, and Python in Zeppelin.

Python in Jupyter notebook (also browser-based) uses tab completion. Tab 
completion doesn't usually interfere with using the tab key in Python because a 
tab isn't normally required after a period such as "df." when one would be 
using the tab completion.




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

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

[doanduyhai]

Github user doanduyhai commented on the issue:

https://github.com/apache/zeppelin/pull/1369
  
@astroshim 

> the number seems doesn't match with count

What do you mean by this ? Which number did you expect to see ?

On the animated screencapture, the count started with 1 then 2 for both 
popups.

Then you close one so the count is reset to 0 but the 2nd popup 
is still there and continue counting to 4.

Do not forget that the `checkDownloadingDependencies()` method is called 
multiple times:

 - everytime an interpreter setting is updated (2 times indeed)
 - everytime the status returned by the server is 
**DOWNLOADING_DEPENDENCIES**


---
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.

[astroshim]

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
@Leemoonsoo I agree with your opinion. Let me fix them.


---
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 #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

[astroshim]

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1369
  
@doanduyhai I know that number means but as you can see the screenshot i 
attached, the number seems doesn't match with count.


---
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 #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

[r-kamath]

Github user r-kamath commented on the issue:

https://github.com/apache/zeppelin/pull/1430
  
@vensant Nice update. Can you also make this limit on UI dynamic by reading 
the size configured in zeppelin-site.xml or zeppelin-env.sh. Look 
for`zeppelin.websocket.max.text.message.size` or 
`ZEPPELIN_WEBSOCKET_MAX_TEXT_MESSAGE_SIZE` in the respective files.


---
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 #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

[doanduyhai]

Github user doanduyhai commented on the issue:

https://github.com/apache/zeppelin/pull/1369
  
@astroshim 

The number you see in front of each error popup is normal, it comes from 
the fact that now we set `combineDuplications  = true` for **ngToast**. The 
number corresponds to the time the error popup is duplicated


---
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 #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

[astroshim]

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1369
  
@doanduyhai Sorry to keep you waiting. 
It works like a charm now but the error count number at the beginning of 
message appears a little strange like following screen shot.

![st](https://cloud.githubusercontent.com/assets/3348133/18585813/0ef47ef2-7c54-11e6-9b59-61fe5040fa25.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 pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

[vensant]

Github user vensant closed the pull request at:

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


---
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 #1430: [ZEPPELIN-868] Notebook import fails when noteb...

[vensant]

GitHub user vensant reopened a pull request:

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

[ZEPPELIN-868] Notebook import fails when notebook is large

### What is this PR for?
A bug fix: Added validation in the note import dialog box to check for the 
uploaded json file size and throw error report if the file size exceeds 1MB, as 
the websocket frame is not able to send json file of size over 1MB.

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

### Todos
NA

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-868

### How should this be tested?
1. Deploy Zeppelin and click on 'Import Note' in the Welcome to Zeppelin 
page.
2. Click 'Choose a JSON here' and upload a json file whose file size is 
over 1MB

### Screenshots (if appropriate)

![1](https://cloud.githubusercontent.com/assets/12062069/18556899/7add7a12-7b8a-11e6-85e6-f8b4fcef2195.png)

![2](https://cloud.githubusercontent.com/assets/12062069/18556966/b10bcb0c-7b8a-11e6-9448-1381d8e05d8f.png)

![3](https://cloud.githubusercontent.com/assets/12062069/18556999/d166b8a8-7b8a-11e6-927b-caa3a56618d1.png)

![4](https://cloud.githubusercontent.com/assets/12062069/18557002/d58f8d74-7b8a-11e6-8955-710eb093a795.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/vensant/zeppelin ZEPPELIN-868

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

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


commit 213181c02d5029b143bb55cbae32408d456e3cf9
Author: vensant 
Date:   2016-09-15T14:38:47Z

Added validation in the note import dialog box to check for the uploaded 
json file size as the websocket frame is not able to send file of size over 1MB

commit c2710193d57274aba9d7e34272d6b29edd81ccea
Author: vensant 
Date:   2016-09-16T06:06:30Z

Rectified the errors due to grunt build no color failure




---
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-1445) Possibility to set resolution and dpi for R interpreter image output

[Dan Staby (JIRA)]

Dan Staby created ZEPPELIN-1445:
---

 Summary: Possibility to set resolution and dpi for R interpreter 
image output
 Key: ZEPPELIN-1445
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1445
 Project: Zeppelin
  Issue Type: Improvement
  Components: r-interpreter
Affects Versions: 0.6.1
Reporter: Dan Staby
Priority: Trivial


During ggplot2 plot output from the R interpreter, it is possible to set 
parameter "imageWidth" to get a specific output size. This change does however 
not reflect the image generation, and most settings result in poorly looking 
plots. 

Please add functionality to set image width, height and dpi for plot output 
from R. I think that one setting per paragraph would be enough. 



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

[GitHub] zeppelin pull request #:

[prabhjyotsingh]

Github user prabhjyotsingh commented on the pull request:


https://github.com/apache/zeppelin/commit/21a084b170b29dee0f2d57aae22b00c785ef0c8e#commitcomment-19051094
  
In zeppelin-web/src/app/interpreter/interpreter.controller.js:
In zeppelin-web/src/app/interpreter/interpreter.controller.js on line 57:
Yes, its not exactly polling, but it calls `getInterpreterSettings()` which 
does a `$http.get(baseUrlSrv.getRestApiBase() + '/interpreter/setting')` which 
on success calls `checkDownloadingDependencies()` until `isDownloading` is true 
in a recursive way, is it not happening in your case ? 

I see below screenshot in my developer console until download is complete 
or you move out from this(#/interpreter) page.
![Uploading Screen Shot 2016-09-16 at 2.38.57 PM.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 #:

[karup1990]

Github user karup1990 commented on the pull request:


https://github.com/apache/zeppelin/commit/21a084b170b29dee0f2d57aae22b00c785ef0c8e#commitcomment-19050314
  
In zeppelin-web/src/app/interpreter/interpreter.controller.js:
In zeppelin-web/src/app/interpreter/interpreter.controller.js on line 57:
Agreed. But 2 seconds seems low to me(especially for maven artifacts with 
transitive deps). But we don't have a definite way to determine the total time 
either.
Also we are not polling here, we are using timeout.


---
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 #1430: [ZEPPELIN-868] Notebook import fails when noteb...

[vensant]

Github user vensant closed the pull request at:

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


---
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 #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

[vensant]

Github user vensant commented on the issue:

https://github.com/apache/zeppelin/pull/1430
  
Closed and reopened the pull request in order to re-trigger the Travis CI 
build. Build failed due to test failures 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 pull request #1430: [ZEPPELIN-868] Notebook import fails when noteb...

[vensant]

GitHub user vensant reopened a pull request:

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

[ZEPPELIN-868] Notebook import fails when notebook is large

### What is this PR for?
A bug fix: Added validation in the note import dialog box to check for the 
uploaded json file size and throw error report if the file size exceeds 1MB, as 
the websocket frame is not able to send json file of size over 1MB.

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

### Todos
NA

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-868

### How should this be tested?
1. Deploy Zeppelin and click on 'Import Note' in the Welcome to Zeppelin 
page.
2. Click 'Choose a JSON here' and upload a json file whose file size is 
over 1MB

### Screenshots (if appropriate)

![1](https://cloud.githubusercontent.com/assets/12062069/18556899/7add7a12-7b8a-11e6-85e6-f8b4fcef2195.png)

![2](https://cloud.githubusercontent.com/assets/12062069/18556966/b10bcb0c-7b8a-11e6-9448-1381d8e05d8f.png)

![3](https://cloud.githubusercontent.com/assets/12062069/18556999/d166b8a8-7b8a-11e6-927b-caa3a56618d1.png)

![4](https://cloud.githubusercontent.com/assets/12062069/18557002/d58f8d74-7b8a-11e6-8955-710eb093a795.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/vensant/zeppelin ZEPPELIN-868

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

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


commit 213181c02d5029b143bb55cbae32408d456e3cf9
Author: vensant 
Date:   2016-09-15T14:38:47Z

Added validation in the note import dialog box to check for the uploaded 
json file size as the websocket frame is not able to send file of size over 1MB

commit c2710193d57274aba9d7e34272d6b29edd81ccea
Author: vensant 
Date:   2016-09-16T06:06:30Z

Rectified the errors due to grunt build no color failure




---
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 #1430: [ZEPPELIN-868] Notebook import fails when noteb...

[vensant]

Github user vensant closed the pull request at:

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


---
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 #1430: [ZEPPELIN-868] Notebook import fails when noteb...

[vensant]

GitHub user vensant reopened a pull request:

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

[ZEPPELIN-868] Notebook import fails when notebook is large

### What is this PR for?
A bug fix: Added validation in the note import dialog box to check for the 
uploaded json file size and throw error report if the file size exceeds 1MB, as 
the websocket frame is not able to send json file of size over 1MB.

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

### Todos
NA

### What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-868

### How should this be tested?
1. Deploy Zeppelin and click on 'Import Note' in the Welcome to Zeppelin 
page.
2. Click 'Choose a JSON here' and upload a json file whose file size is 
over 1MB

### Screenshots (if appropriate)

![1](https://cloud.githubusercontent.com/assets/12062069/18556899/7add7a12-7b8a-11e6-85e6-f8b4fcef2195.png)

![2](https://cloud.githubusercontent.com/assets/12062069/18556966/b10bcb0c-7b8a-11e6-9448-1381d8e05d8f.png)

![3](https://cloud.githubusercontent.com/assets/12062069/18556999/d166b8a8-7b8a-11e6-927b-caa3a56618d1.png)

![4](https://cloud.githubusercontent.com/assets/12062069/18557002/d58f8d74-7b8a-11e6-8955-710eb093a795.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/vensant/zeppelin ZEPPELIN-868

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

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


commit 213181c02d5029b143bb55cbae32408d456e3cf9
Author: vensant 
Date:   2016-09-15T14:38:47Z

Added validation in the note import dialog box to check for the uploaded 
json file size as the websocket frame is not able to send file of size over 1MB

commit c2710193d57274aba9d7e34272d6b29edd81ccea
Author: vensant 
Date:   2016-09-16T06:06:30Z

Rectified the errors due to grunt build no color failure




---
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 #1430: [ZEPPELIN-868] Notebook import fails when notebook is ...

[vensant]

Github user vensant commented on the issue:

https://github.com/apache/zeppelin/pull/1430
  
Closed and reopened the pull request in order to re-trigger the Travis CI 
build. Build failed due to test failures


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

[prabhjyotsingh]

Github user prabhjyotsingh commented on the pull request:


https://github.com/apache/zeppelin/commit/21a084b170b29dee0f2d57aae22b00c785ef0c8e#commitcomment-19049191
  
In zeppelin-web/src/app/interpreter/interpreter.controller.js:
In zeppelin-web/src/app/interpreter/interpreter.controller.js on line 57:
I too initially thought about `nosOfDeps*n second`, but download time 
depends on internet speed and size of dependencies and then some of 
dependencies may require some other/more dependencies (internally). Hence, 
settled for 2 sec poll. What do you think ?


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