date:20170711

[GitHub] zeppelin issue #2460: [gh-pages] fix hover event in technologies section on ...

2017-07-11 Thread soralee

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2460
  
Cool! Thanks @1ambda for checking! 
I also just checked it works well on mobile


---
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 #2460: [gh-pages] fix hover event in technologies section on ...

2017-07-11 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2460
  
and works well in mobile (iphone 6s chrome, sarari)


---
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 #2460: [gh-pages] fix hover event in technologies section on ...

2017-07-11 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2460
  
Deployed.


---
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 #2460: [gh-pages] fix hover event in technologies sect...

2017-07-11 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #2467: [ZEPPELIN-2730] compare with maxLength after getting a...

2017-07-11 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2467
  
Thanks @gaoyang143 for the fix.
LGTM and merge to master if 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 #2485: [ZEPPELIN-2758]. NumberFormatException on importing no...

2017-07-11 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2485
  
 @karuppayya I remove `NotebookTypeAdapterFactory.java` which I think is 
not necessary. Let me know your concern. 


---
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 #2238: 0.7.1 with ssl

2017-07-11 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2238
  
They might be flaky tests. Could you retry failed jobs and see if it goes 
green?


---
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 #2486: [ZEPPELIN-2768]. Bump up Spark version to 2.2.0

2017-07-11 Thread zjffdu

GitHub user zjffdu opened a pull request:

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

[ZEPPELIN-2768]. Bump up Spark version to 2.2.0

### What is this PR for?
Spark 2.2.0 is just released, this PR is to support spark 2.2.0. 

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

### Todos
* [ ] - Task

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

### How should this be tested?
Travis is updated

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

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

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


commit 8fa130b8c1fdd350e67026ba010db5f598d432c1
Author: Jeff Zhang 
Date:   2017-07-12T03:51:47Z

[ZEPPELIN-2768]. Bump up Spark version to 2.2.0




---
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-2768) Bump up Spark version to 2.2.0

2017-07-11 Thread Jeff Zhang (JIRA)

Jeff Zhang created ZEPPELIN-2768:


 Summary: Bump up Spark version to 2.2.0
 Key: ZEPPELIN-2768
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2768
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Jeff Zhang






--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin pull request #2485: [ZEPPELIN-2758]. NumberFormatException on impor...

2017-07-11 Thread zjffdu

GitHub user zjffdu opened a pull request:

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

[ZEPPELIN-2758]. NumberFormatException on importing notebook

### What is this PR for?

This is due to the custom TypeAdapter for Paragraph. I don't know why we 
introduce it. To me it is not necessary. So I remove this in this PR. And the 
spark UI still can be accessed in frontend. 

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

### Todos
* [ ] - Task

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

### How should this be tested?
Unit test is updated and manually tested

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

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

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


commit b51d5665bbffb987fe3acc1b9d27adc3f717dd3c
Author: Jeff Zhang 
Date:   2017-07-12T03:38:41Z

[ZEPPELIN-2758]. NumberFormatException on importing notebook




---
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 #2481: [HOTFIX][ZEPPELIN-2760] fix JDBC regression caused aft...

2017-07-11 Thread prabhjyotsingh

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2481
  
@herval  and @felixcheung Thank you for the review. Will merge this if no 
more 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 #2471: [ZEPPELIN-2744] [minor] Increase error handling in JDB...

2017-07-11 Thread prabhjyotsingh

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2471
  
Sure, I see your point, will close this, and mark ZEPPELIN-2744 works as 
designed.


---
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 #2238: 0.7.1 with ssl

2017-07-11 Thread instaclustr-oss

Github user instaclustr-oss commented on the issue:

https://github.com/apache/zeppelin/pull/2238
  
Hi @Leemoonsoo 

Sorry for taking awhile to setup the CI. Now that I have, I see some jobs 
fail. What should I do about 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.
---

[GitHub] zeppelin issue #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2017-07-11 Thread herval

Github user herval commented on the issue:

https://github.com/apache/zeppelin/pull/2439
  
all tests passing now


---
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 #2484: [ZEPPELIN-2711] basic metrics for paragraphs & noteboo...

2017-07-11 Thread herval

Github user herval commented on the issue:

https://github.com/apache/zeppelin/pull/2484
  
This is the basic set of functionality needed to start monitoring some 
common metrics - Comments & observations welcome!


---
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 #2478: [ZEPPELIN-2761] - Neo4j Interpreter

2017-07-11 Thread jexp

Github user jexp commented on the issue:

https://github.com/apache/zeppelin/pull/2478
  
I looked through the code and the UX and can confirm that this is great 
work from a Neo4j point of 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 issue #2483: [ZEPPELIN-2766] Make online resources url configurable...

2017-07-11 Thread andreaTP

Github user andreaTP commented on the issue:

https://github.com/apache/zeppelin/pull/2483
  
yes of course @felixcheung , basically I'm exposing through maven 
properties the ability for users to customize where to download from certain 
packets that are normally downloaded from very specific internet resources.
In this way one can build zeppelin from sources specifying faster or 
internal or more reliable addresses for those packages.

You can note that this does not break anything but just expose some plugin 
capabilities to users and fallback to proper default one.



---
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-2767) Loading scatter chart in chrome, safari freezes the application - when show tables command is executed

2017-07-11 Thread jahira ibrahim (JIRA)

jahira ibrahim created ZEPPELIN-2767:


 Summary: Loading scatter chart in chrome, safari freezes the 
application - when show tables command is executed
 Key: ZEPPELIN-2767
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2767
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Reporter: jahira ibrahim


In chrome,Execute the  show table command. The result will show the tables 
which are created. If the result has at least one record, and if scatter chart 
is selected, the entire application freezes. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin issue #2480: [ZEPPELIN-1249] build all submodules w/ the same scala...

2017-07-11 Thread herval

Github user herval commented on the issue:

https://github.com/apache/zeppelin/pull/2480
  
Regarding verification, how do you suggest I go about that? We've been 
using this setup internally for some time, but we only use a few of the 
interpreters (spark, scalding, jdbc, md) - not sure if that could lead to any 
sort of runtime issue on other interpreters, but I find it highly unlikely..

Can you clarify on the flexibility bit? Not sure I understand what's the 
benefit of using different versions at the same time (not to mention Scala 2.10 
is EOL since 2016).


---
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 #2484: [ZEPPELIN-2711] [WIP] basic metrics for paragra...

2017-07-11 Thread herval

GitHub user herval opened a pull request:

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

[ZEPPELIN-2711] [WIP] basic metrics for paragraphs & notebook 
view/create/run

### What is this PR for?

This exposes JMX metrics for a few operations on Zeppelin. The first step 
here is to expose them via JMX, then later introduce a JSON endpoint (which 
would make it easier to integrate these metrics on common monitoring systems 
such as Prometheus, Graphite, Influx, etc

I'm putting this out as WIP before finishing the tests & JSON bits in order 
to kick-off any design discussion and address comments sooner than later

### What type of PR is it?
Feature

### Todos
* [ ] - Tests
* [ ] - JSON endpoint

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

### How should this be tested?
TODO

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update?
Yes

* Is there breaking changes for older versions?
No

* Does this needs documentation?
Yes

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

$ git pull https://github.com/herval/zeppelin metrics

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

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


commit 41bfbe3adf3b6a99c8b8794ae225ec68e98b5848
Author: Herval Freire 
Date:   2017-07-11T00:21:50Z

basic metrics for paragraphs & notebook view/create/run




---
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 #2439: [ZEPPELIN-2680] allow opening notebook as a reader

2017-07-11 Thread soralee

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2439
  
Tested and LGTM! 
It looks cool that when a `writers` logins a note, it shows report mode.


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

[GitHub] zeppelin issue #2481: [HOTFIX][ZEPPELIN-2760] fix JDBC regression caused aft...

2017-07-11 Thread herval

Github user herval commented on the issue:

https://github.com/apache/zeppelin/pull/2481
  
oh, Indeed. There's some inheritance there ð¶


---
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 #2481: [HOTFIX][ZEPPELIN-2760] fix JDBC regression caused aft...

2017-07-11 Thread prabhjyotsingh

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2481
  
Well, any test case calling `open()` will hit `isKerboseEnabled()`, and in 
`JdbcInterpreterTest` when ever `isKerboseEnabled` function is called will 
returning false all the time.


Ref: 
https://github.com/apache/zeppelin/blob/master/jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java#L173

https://github.com/apache/zeppelin/blob/master/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/KerberosInterpreter.java#L64


---
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 #2481: [HOTFIX][ZEPPELIN-2760] fix JDBC regression caused aft...

2017-07-11 Thread herval

Github user herval commented on the issue:

https://github.com/apache/zeppelin/pull/2481
  
I actually don't see any test on `JdbcInterpreterTest` covering that code 
path at all - I think the test suite is passing simply because that bit is not 
tested, and feel it should be


---
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 #2477: [ZEPPELIN-2342] Improving Test Environment for ...

2017-07-11 Thread 1ambda

GitHub user 1ambda reopened a pull request:

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

[ZEPPELIN-2342] Improving Test Environment for zeppelin-web (e2e, coverage)

### What is this PR for?

Improved testing environment for zeppelin-web

- added `test:watch` command for continuous testing 
- displaying coverage report in console
- e2e testing in zeppelin-web (can be written easily compare to the 
zeppelin-server integration tests. For example: 
[home.spec.js](https://github.com/1ambda/zeppelin/blob/f90deb4bf3db57e36232c9c94ae3b07dfd7c2331/zeppelin-web/e2e/home.spec.js))

This PR adds [a zeppelin-web specific travis 
profile](https://github.com/1ambda/zeppelin/blob/ad08a3bc0da420f7a7b4771d1aee8bfdd962efa7/.travis.yml#L55)
 and it usually takes 8-9 mins to execute unit + e2e tests (zeppelin-web) 

- https://travis-ci.org/1ambda/zeppelin/jobs/252453901

 Details

- the headless chrome browser works with chromedriver 2.29+ 
(https://chromium.googlesource.com/chromium/src/+/c75dbfd25cc376b92a494d340259056a01e1432e)
- in the case of ubuntu precise (our travis container), should use 
`/usr/bin/chromium-browser`
- `xvfb` is required, please refer 
https://docs.travis-ci.com/user/gui-and-headless-browsers/#Using-xvfb-to-Run-Tests-That-Require-a-GUI
 and 
https://iamakulov.com/notes/install-google-chrome-on-travis/#what-does-the-code-mean
- for protractor conf, please check 
https://github.com/travis-ci/travis-ci/issues/7650#issuecomment-302467188
- can't install stable version of chrome, precise (12.04) support was 
dropped by chrome
- chromium-browser (37.0) is shipped by default in precise and chromedriver 
version 2.11 is compatible with it. 
(https://support.saucelabs.com/hc/en-us/articles/225095307-Chromedriver-version-for-each-Chrome-browser-version,
 https://github.com/travis-ci/travis-ci/issues/5899)


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

### Todos
* [x] - `test:watch`
* [x] - displaying coverage reports in console
* [x] - e2e testing in zeppelin-web

### What is the Jira issue?

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

### How should this be tested?

CI should be green. If you want to test locally,

- stop zeppelin which is running localhost:8080
- `mvn package -DskipTests -DskipRat -Pweb-ci -Pweb-e2e -B`
- `mvn verify -DskipRat -pl 'zeppelin-web' -Pweb-ci -Pweb-e2e -B`


 Screenshots (coverage)


![image](https://user-images.githubusercontent.com/4968473/28031333-dc700c3a-65e1-11e7-96ca-582a9ffa97dd.png)

 Screenshots (e2e)


![image](https://user-images.githubusercontent.com/4968473/28032319-0e8b5adc-65e5-11e7-81ba-4e33c5f15bc0.png)

### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - YES, updated


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

$ git pull https://github.com/1ambda/zeppelin 
ZEPPELIN-2342/differentiate-unit-and-integration-testing

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

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


commit e70281bad21ea1d66b21acd6b5e33c84bfcf720f
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T06:38:49Z

feat: Use spec repoter and coverage

commit 39dc77e4f3b9aab90c101e29a02cecf9a5696c11
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T07:03:29Z

chore: Add test:watch command

commit 15473de6b59f8941562f1b6ae10f4c8b13dbaad2
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T08:29:23Z

test: Add a basic e2e test for home

commit 76f4f584f4624a63edd5ec4fae09e5d44332bd41
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T08:33:20Z

docs: Update test section in zeppelin-web/README.md

commit b5a17a4baf112e89c747d4b18fbff33f6d4a6551
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T09:26:50Z

chore: Bind browser e2e test into CI process

commit 742bb961a2691e35084120ceca28639a13671a2f
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T09:40:37Z

chore: Exclude e2e specs from RAT

commit eb447ef8ef199b72c0a613ecfaef60f55062a48c
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T10:20:44Z

WIP: add -X log to selenium profioe

commit 2ed397331007e1c4863d983519a30afab958fbc8
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T11:18:03Z

fix: set gecko false for webdriver manager. See 
https://github.com/angular/webdriver-manager/issues/216

commit 0907d7286dedd83df6afb5bfc11a57b682479c7f
Author: 1ambda <1am...@gmail.com>
Date:   2017-07-10T11:18:16Z

chore: Ingnore protractor conf in RAT

commit 0c09727e95fd1216d6113b53865171466a776852

[GitHub] zeppelin pull request #2477: [ZEPPELIN-2342] Improving Test Environment for ...

2017-07-11 Thread 1ambda

Github user 1ambda closed the pull request at:

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


---
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 #2483: [ZEPPELIN-2766] Make online resources url confi...

2017-07-11 Thread andreaTP

GitHub user andreaTP reopened a pull request:

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

[ZEPPELIN-2766] Make online resources url configurable at compile time

### What is this PR for?
At compile time Zeppelin is downloading several external resources.
I want to be able to provide alternative URLs to compile when internet is 
not available(i.e. behind corporate proxy).

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

### What is the Jira issue?
[ZEPPELIN-2766]

### How should this be tested?
```
mvn -DskipTests -Dplugin.frontend.downloadRoot= 
-Dplugin.frontend.nodeDownloadRoot= 
-Dplugin.frontend.yarnDownloadRoot= 
-Dplugin.frontend.npmDownloadRoot= 
-Dpypi.repo.url= clean package
```

### 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/nokia/zeppelin softCodeRepos

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

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


commit faf8362e2e7b0ea308434c8c5fb96ae8cb07a1d2
Author: andrea 
Date:   2017-07-11T13:55:01Z

Keep online resources configurable




---
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 #2483: [ZEPPELIN-2766] Make online resources url confi...

2017-07-11 Thread andreaTP

Github user andreaTP closed the pull request at:

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


---
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 #2448: [ZEPPELIN-2702] save notes in reader-friendly format w...

2017-07-11 Thread khalidhuseynov

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/2448
  
@zjffdu yeah, the renaming would be done probably on saving which would 
apply if user enters note, and does some action so that save->conversion 
(rename) is initiated. This would work further everytime to rename from 
`title1.zpln` -> `title2.zpln`. But I was concerned more about scenario when 
user has 100 notes, and if he wants to convert them into new format (rename), 
he would need to enter every note and do something to initiate conversion. That 
would be little troublesome for user, so i thought maybe to initiate it during 
initial load for all notes as well. For now i'll change to save, and we can 
discuss further about loading 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 pull request #2483: [ZEPPELIN-2766] Make online resources url confi...

2017-07-11 Thread andreaTP

GitHub user andreaTP opened a pull request:

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

[ZEPPELIN-2766] Make online resources url configurable at compile time

### What is this PR for?
At compile time Zeppelin is downloading several external resources.
I want to be able to provide alternative URLs to compile when internet is 
not available(i.e. behind corporate proxy).

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

### What is the Jira issue?
[ZEPPELIN-2766]

### How should this be tested?
```
mvn -DskipTests -Dplugin.frontend.downloadRoot= 
-Dplugin.frontend.nodeDownloadRoot= 
-Dplugin.frontend.yarnDownloadRoot= 
-Dplugin.frontend.npmDownloadRoot= 
-Dpypi.repo.url= clean package
```

### 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/nokia/zeppelin softCodeRepos

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

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


commit faf8362e2e7b0ea308434c8c5fb96ae8cb07a1d2
Author: andrea 
Date:   2017-07-11T13:55:01Z

Keep online resources configurable




---
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-2766) Make online resources url configurable at compile time

2017-07-11 Thread Andrea Peruffo (JIRA)

Andrea Peruffo created ZEPPELIN-2766:


 Summary: Make online resources url configurable at compile time
 Key: ZEPPELIN-2766
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2766
 Project: Zeppelin
  Issue Type: Improvement
Reporter: Andrea Peruffo


At compile time Zeppelin is downloading several external resources.
I want to be able to provide alternative URLs to compile when internet is not 
available(i.e. behind corporate proxy).





--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin issue #2448: [ZEPPELIN-2702] save notes in reader-friendly format w...

2017-07-11 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2448
  
How about do the renaming in saving ? We just need to save to the new file 
and delete the old note file. 


---
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 #2448: [ZEPPELIN-2702] save notes in reader-friendly format w...

2017-07-11 Thread khalidhuseynov

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/2448
  
@Leemoonsoo @1ambda @zjffdu I wanted to have opinion on the note conversion 
strategy. lazy strategy - converting when note is renamed, vs aggressive - 
converting all notes when starting zeppelin. currently it's implemented here 
with lazy strategy, however the more i think, the more aggressive way seems 
more relevant. One of the problems with lazy strategy is that, if user has many 
notes, it would be burdensome to convert notes to new format by manually 
renaming each one. 


---
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 #2482: [ZEPPELIN-2765] Configurable X-FRAME-OPTIONS for Zeppe...

2017-07-11 Thread krishna-pandey

Github user krishna-pandey commented on the issue:

https://github.com/apache/zeppelin/pull/2482
  
@Leemoonsoo, @felixcheung, @jongyoul, @prabhjyotsingh Please help review 
this.


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

[GitHub] zeppelin pull request #2482: [ZEPPELIN-2765] Configurable X-FRAME-OPTIONS fo...

2017-07-11 Thread krishna-pandey

GitHub user krishna-pandey opened a pull request:

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

[ZEPPELIN-2765] Configurable X-FRAME-OPTIONS for Zeppelin

### What is this PR for?
The X-Frame-Options HTTP response header can be used to indicate whether or 
not a browser should be allowed to render a page in a  or . 
Sites can use this to avoid Clickjacking attacks, by ensuring that their 
content is not embedded into other sites. Set the X-Frame-Options header for 
all responses containing HTML content. The possible values are "DENY", 
"SAMEORIGIN", or "ALLOW-FROM uri"

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

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

### How should this be tested?
The application (Zeppelin) loads in iframe. Put below code in a html file 
and open in browser:



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

$ git pull https://github.com/krishna-pandey/zeppelin ZEPPELIN-2765

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

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


commit 518f1a4a213614d038b4fd554614afe8ec2c8c20
Author: krishna-pandey 
Date:   2017-07-11T12:09:37Z

Configurable X-FRAME-OPTIONS for Zeppelin




---
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-2765) Configurable X-FRAME-OPTIONS for Zeppelin

2017-07-11 Thread Krishna Pandey (JIRA)

Krishna Pandey created ZEPPELIN-2765:


 Summary: Configurable X-FRAME-OPTIONS for Zeppelin
 Key: ZEPPELIN-2765
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2765
 Project: Zeppelin
  Issue Type: Improvement
  Components: zeppelin-server
Affects Versions: 0.7.0
Reporter: Krishna Pandey


*Information systems must not be susceptible to Cross-frame Scripting (XFS) 
attacks and clickjacking.*
Information systems must implement proper input validation and output encoding. 
With proper input validation and output encoding, information systems will not 
be susceptible to cross-frame scripting, either stored or reflected. A 
successful cross-frame scripting attack may redirect a user to a malicious 
third-party page.
*Technical Risk* : Without proper input validation and output encoding, 
information systems are susceptible to cross-frame scripting, which may result 
in unauthorized access or malicious attacks against the user.
*Corrective Action* : This issue can be fixed using the following methods1. 
Implementing the response header X-Frame- Options and set the value to 'Deny' 
or 'same origin'.2. Sending the proper browser response headers that instruct 
the browser to not allow framing from other domains . 

The application (Zeppelin) loads in iframe.
https://localhost:8443/#/; width="100%" height="600">

The X-Frame-Options HTTP response header can be used to indicate whether or not 
a browser should be allowed to render a page in a  or . Sites 
can use this to avoid Clickjacking attacks, by ensuring that their content is 
not embedded into other sites. Set the X-Frame-Options header for all responses 
containing HTML content. The possible values are "DENY", "SAMEORIGIN", or 
"ALLOW-FROM uri"

*X-Frame-Options Header Types*
There are three possible values for the X-Frame-Options header:

*DENY*, which prevents any domain from framing the content. The "DENY" setting 
is recommended unless a specific need has been identified for framing.
*SAMEORIGIN*, which only allows the current site to frame the content.
*ALLOW-FROM uri*, which permits the specified 'uri' to frame this page. (e.g., 
ALLOW-FROM http://www.example.com).



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin issue #2481: [HOTFIX][ZEPPELIN-2760] fix JDBC regression caused aft...

2017-07-11 Thread prabhjyotsingh

Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/2481
  
@herval The reason this exception didn't get caught with any of the 
existing test cases is because this particular dependency (for 
UserGroupInformation) is marked as `provided` 
(https://github.com/apache/zeppelin/blob/master/jdbc/pom.xml#L106) and while 
executing any of the test cases it gets resolved but doesn't get bundled. 


---
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 #2474: [ZEPPELIN-2753] Basic Implementation of IPython Interp...

2017-07-11 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2474
  
@Leemoonsoo @1ambda @felixcheung @prabhjyotsingh Please help review. 


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

[GitHub] zeppelin issue #2329: [ZEPPELIN-2040] ClusterManager to support launching in...

2017-07-11 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2329
  
@jongyoul I still could not get shell interpreter work in yarn mode. But 
this time it is due to AM fail to launch. Here's the log in AM container.
```
Error: Could not find or load main class 
org.apache.zeppelin.cluster.yarn.YarnRemoteInterpreterServer
```

I would suggest you to add unit test to verify it (Launch ShellInterpreter 
or some dummy interpreter in MiniYarnCluster) 


---
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-2764) Change all links from ZeppelinHub to Zepl

2017-07-11 Thread Khalid Huseynov (JIRA)

Khalid Huseynov created ZEPPELIN-2764:
-

 Summary: Change all links from ZeppelinHub to Zepl
 Key: ZEPPELIN-2764
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2764
 Project: Zeppelin
  Issue Type: Task
Reporter: Khalid Huseynov
Assignee: Khalid Huseynov
 Fix For: 0.8.0


zeppelinhub service was moved under zepl, thus need to update the links, 
package names, conf files, etc.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin issue #2481: [HOTFIX][ZEPPELIN-2760] fix JDBC regression caused aft...

2017-07-11 Thread herval

Github user herval commented on the issue:

https://github.com/apache/zeppelin/pull/2481
  
would it be worth adding a test around this? I imagine the tests would fail 
with a "class not found", so it would be easy to add one that proves that you 
don't get the error if auth.type is empty


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

41 matches

Mail list logo