date:20170404

[GitHub] zeppelin issue #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2223
  
@benoyantony You can call cancel in a separate thread, the same as 
testSparkInterpreterRDD, let me know if you have any other questions. 


---
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 #2125: [ZEPPELIN-2222] Add Network Visualization

2017-04-04 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2125
  
It's just my opinion. Thus, it's ok to use d3 if you think it's reasonable 
ð 


---
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 #2181: [ZEPPELIN-2304] Fix vis Icon sizes in helium page

2017-04-04 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2181
  
@AhyoungRyu I saw your comment now. Let me create a PR for the branch 0.7


---
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 #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread benoyantony

Github user benoyantony commented on the issue:

https://github.com/apache/zeppelin/pull/2223
  
@zjffdu , Will try to add the unit test for cancellation in 
LivySparkSQLInterpreter. Any suggestion on how to simulate a sleep in SQL ? 
Also, where are the other comments ? Sorry, I am not very familiar with the 
github code reviews. It may be very obvious.


---
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 #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-04-04 Thread 1ambda

Github user 1ambda commented on the issue:

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

![image](https://cloud.githubusercontent.com/assets/4968473/24690813/45d7d834-1a0a-11e7-88bd-ca78d69a30df.png)

@AhyoungRyu I forgot as well to modify `index.md` ð.
I'v just updated.


---
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 #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2223
  
@benoyantony Thanks for the contribution, just left some comments. 


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

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-04-04 Thread AhyoungRyu

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2098
  
@1ambda Looks great! 
So sorry I forgot to mention that there is another place to be updated 
regarding [this 
change](https://github.com/apache/zeppelin/pull/2098#issuecomment-291390619). 

https://cloud.githubusercontent.com/assets/10060731/24690393/7e4612ec-1a07-11e7-8d70-34554826a27a.png;
 width="600px">

Could you update 
[`docs/index.html`](https://github.com/apache/zeppelin/blob/master/docs/index.md#more)
 as well? 


---
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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...

2017-04-04 Thread AhyoungRyu

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/2199
  
Merge into master and branch-0.7 if there are no more comments on 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 #2211: [ZEPPELIN-2339] Change 'see more' -> 'enable' n...

2017-04-04 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #2222: add a configuration zeppelin.jdbc.auth.kerberos.proxy ...

2017-04-04 Thread AhyoungRyu

Github user AhyoungRyu commented on the issue:

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


---
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 #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread benoyantony

Github user benoyantony commented on the issue:

https://github.com/apache/zeppelin/pull/2223
  
@zjffdu , Could you please 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 pull request #2223: [ZEPPELIN-2355] Livy cancel enhancements

2017-04-04 Thread benoyantony

GitHub user benoyantony opened a pull request:

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

[ZEPPELIN-2355] Livy cancel enhancements

### What is this PR for?
The Cancel functionality for the Livy interpreter has a couple of issues. 
One issue is because a variable is not published correctly. Second issue is 
observed when  there is a delay in launching the application. Any cancel before 
application launch is ignored.  The third issue is that Cancel is not correctly 
implemented for SparkSQLInterpreter.


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

### Todos
* [ ] - Task

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

### How should this be tested?
The test cases are modified to test the changes.

### 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/benoyantony/zeppelin livy-cancel-enhancement

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

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


commit 8673acfd5da76e547b986fb9dcfdb74fc3c379cf
Author: Benoy Antony 
Date:   2017-04-05T03:27:30Z

ZEPPELIN-2355 Fix race conditions while cancelling a paragraph

commit 4f1f243b2b8e16860b25dd59f31924fb2c16c8c3
Author: Benoy Antony 
Date:   2017-04-05T03:36:16Z

remove unrelated changes in imports




---
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-2355) Fix race conditions while cancelling a paragraph

2017-04-04 Thread Benoy Antony (JIRA)

Benoy Antony created ZEPPELIN-2355:
--

 Summary: Fix race conditions while cancelling a paragraph
 Key: ZEPPELIN-2355
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2355
 Project: Zeppelin
  Issue Type: Bug
Reporter: Benoy Antony
Assignee: Benoy Antony


I experienced a few issues while testing the cancel functionality for a Livy 
paragraph. The tests were performed on a real yarn cluster with Livy running in 
cluster modes. On a real cluster, it takes some time to launch application and 
start executing the paragraphs. The current cancel function has a few 
concurrency issues.
The visible issue was that the user will keep on cancelling initially, but 
paragraph will run to the finish.

{code}
 @Override
  public void cancel(InterpreterContext context) {
if (livyVersion.isCancelSupported()) {
  String paraId = context.getParagraphId();
  Integer stmtId = paragraphId2StmtIdMap.get(paraId);
  try {
if (stmtId != null) {
  cancelStatement(stmtId);
}
  } catch (LivyException e) {
LOGGER.error("Fail to cancel statement " + stmtId + " for paragraph " + 
paraId, e);
  } finally {
paragraphId2StmtIdMap.remove(paraId);
  }
} else {
  LOGGER.warn("cancel is not supported for this version of livy: " + 
livyVersion);
}
  }
{code}

Issue 1 : The variable livyVersion is set in initLivySession(). The thread 
executing cancel may not see the value and hence throw NullPointerException.

Issue 2 : The cancel is a no-op if the statement id is not available.  A 
significant time (may be up to a minute) may pass before the statement id is 
available. The user need to keep cancelling till the statement id is available. 
There is no real way for the user to identify when the statement id is 
available.

Issue 3: THE SQL paragraph cannot be cancelled. This can be fixed by changing 
LivySparkSQLInterpreter to invoke cancel on the underlying LivySparkInterpreter




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin issue #2192: [ZEPPELIN-2320] fixing test + modifying map browsing

2017-04-04 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/2192
  
@Remilito Could you make this PR based on `branch-0.7` branch?


---
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 #2192: [ZEPPELIN-2320] fixing test + modifying map browsing

2017-04-04 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/2192
  
LGTM..
Sorry for late catch..
Merge if there is 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 #2151: [ZEPPELIN-465] Be able to run paragraph and the follow...

2017-04-04 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2151
  
Here's one related PR #2195 which do it in `ZeppelinContext`, I hope we can 
merge these 2 tickets to use the same 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 #2151: [ZEPPELIN-465] Be able to run paragraph and the follow...

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2151
  
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 #2158: [ZEPPELIN-2279] excluded comments from SQL

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2158
  
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 pull request #2214: [MINOR] remove typo in line and area chart

2017-04-04 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #2120: [ZEPPELIN-1492] fixing the issue where updating...

2017-04-04 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2134
  
LGTM and merge to master if no further discussions.

Thanks @tinkoff-dwh for the improvement!


---
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 #2219: [ZEPPELIN-2348] Line chart setting is not rendered (br...

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2219
  
@1ambda This PR has merged but @asfgit didn't close it. Could you close it 
manually?


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

[GitHub] zeppelin issue #2219: [ZEPPELIN-2348] Line chart setting is not rendered (br...

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2219
  
Merge this to branch-0.7 along with #2218


---
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 #2218: [ZEPPELIN-2348] Line chart setting is not rende...

2017-04-04 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #2216: [ZEPPELIN-2336] Fix note reload on another note...

2017-04-04 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #2216: [ZEPPELIN-2336] Fix note reload on another note create...

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2216
  
Merge to master and branch-0.7 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 #2218: [ZEPPELIN-2348] Line chart setting is not rendered (ma...

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2218
  
While this fixes broken feature, i'm merging it to master 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.
---

[jira] [Created] (ZEPPELIN-2354) Toolbar blocks majority of screen on mobile devices

2017-04-04 Thread Paul Brenner (JIRA)

Paul Brenner created ZEPPELIN-2354:
--

 Summary: Toolbar blocks majority of screen on mobile devices
 Key: ZEPPELIN-2354
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2354
 Project: Zeppelin
  Issue Type: Improvement
  Components: GUI
Affects Versions: 0.7.1
 Environment: iOS 10.3  - Safari Browser
Reporter: Paul Brenner
Priority: Minor


Zeppelin works on mobile, which is amazing, but a few tweaks would really help 
improve the experience.

1. When working on a notebook the white toolbar (with,e.g., run all, show/hide 
code, and run scheduler buttons) takes up almost the entire screen. Is there a 
way to address this? Perhaps would it be possible collapse all toolbars (both 
the white toolbar and the blue menu bar) with a button or when entering text in 
a paragraph? 
2. Paragraphs with long names can interfere with the buttons on the top right 
of the paragraph (e.g., run paragraph, collapse code) so that when trying to 
tap the run button the paragraph name will be selected instead. Is it possible 
to address this? Perhaps have the paragraph name wrap before it hits the 
buttons?



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin issue #2222: add a configuration zeppelin.jdbc.auth.kerberos.proxy ...

2017-04-04 Thread AhyoungRyu

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/
  
@wary Thanks for your contribution. Could you add 
`zeppelin.jdbc.auth.kerberos.proxy.enable` info to [this 
table](https://github.com/apache/zeppelin/blob/master/docs/interpreter/jdbc.md#more-properties)
 as well? 


---
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 #2222: add a configuration zeppelin.jdbc.auth.kerberos...

2017-04-04 Thread wary

GitHub user wary opened a pull request:

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

add a configuration zeppelin.jdbc.auth.kerberos.proxy for kerberos proxy 
behavior

â¦e to disable proxy behavior

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


### What type of PR is it?
[Improvement]
* add a configuration item  zeppelin.jdbc.auth.kerberos.proxy to disable 
kerberos behaviour 
as we knowï¼ in current version of zeppelinï¼ if we have kerberos auth 
configuredï¼and using zeppelin with a user loginï¼the jdbc interpreter will 
do a proxy behavior with the login user automaticallyï¼but in many casesï¼ we 
do not want do thisï¼and we do not want bind the zeppelin user system with 
kerberos auth system. I think it's make senses to add a configuration item to 
disable this behavior.



### Todos
* [ ] - Task

### What is the Jira issue?
* Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
* Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. 
[ZEPPELIN-533]

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

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

when we have kerberos auth configured, and using zeppelin with a login user 
add configuration:
"zeppelin.jdbc.auth.kerberos.proxy.enable=false"
for jdbc iterpreter configuration, the jdbc iterpreter will not do the 
proxy behavior with the login user

### Screenshots (if appropriate)

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

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

$ git pull https://github.com/wary/zeppelin main-master

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

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


commit 067696aaea46ab00c2fe12e707ea4bb34e1f88e8
Author: lei wang 
Date:   2017-04-04T12:39:32Z

add configure zeppelin.jdbc.auth.kerberos.proxy for kerberos auth type to 
disable proxy behavior




---
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-2352) merge_zeppelin_pr.py doesn't work for python 3

2017-04-04 Thread Jeff Zhang (JIRA)

Jeff Zhang created ZEPPELIN-2352:


 Summary: merge_zeppelin_pr.py doesn't work for python 3
 Key: ZEPPELIN-2352
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2352
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.7.1
Reporter: Jeff Zhang
Assignee: Jeff Zhang


{code}
  File "dev/merge_zeppelin_pr.py", line 65
print "Unable to fetch URL, exiting: %s" % url
   ^
SyntaxError: Missing parentheses in call to 'print'
{code}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-04-04 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2098
  
@AhyoungRyu resolved all comments. Thanks for detailed 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.
---

[jira] [Created] (ZEPPELIN-2351) View Apache Spark UI when using Zeppelin on EMR cluster and port forwarding 4040

2017-04-04 Thread Nassir (JIRA)

Nassir created ZEPPELIN-2351:


 Summary: View Apache Spark UI when using Zeppelin on EMR cluster 
and port forwarding 4040
 Key: ZEPPELIN-2351
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2351
 Project: Zeppelin
  Issue Type: Bug
Reporter: Nassir


Hi I am trying to view the Spark UI to get an idea of resource usage, 
debugging, etc. However, after creating an SSH tunnel and port forwarding 4040, 
I can view the ""Zeppelin Application UI" however, the pages do not load 
correctly and it appears as the CSS is not loaded - hence navigation is not 
working and the pages are difficult to understand.

I am basically trying to view the Spark UI where I am running spark with 
Zeppelin installation on EMR cluster.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (ZEPPELIN-2350) Specify helium package name for dev:helium not to bundle all packages

2017-04-04 Thread Hoon Park (JIRA)

Hoon Park created ZEPPELIN-2350:
---

 Summary: Specify helium package name for dev:helium not to bundle 
all packages
 Key: ZEPPELIN-2350
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2350
 Project: Zeppelin
  Issue Type: Improvement
  Components: build, front-end, helium
Reporter: Hoon Park


Specify helium package name for dev:helium not to bundle all packages. 
Otherwise refresh will be very slow.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin issue #2210: [ZEPPELIN-2088] Bundle helium packages one by one

2017-04-04 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2210
  
Sorry for bothering. Fixed that issue and tested like

- remove `conf/heli.json` and `local-repo/helium-bundle`
- build and restart Zeppelin
- install any bundle (**test 1 to see `bundlePackage` works**)
- then remove `local-repo/helium-bundle`
- restart (**test 2 to see `bundleAllPackages` works)
- open other terminal and run `npm run dev:helium`
- modify your local package and refresh to see local package is updated 
(**test3**)



---
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-2349) Graphs from Table Display are not rendered by IE 11

2017-04-04 Thread Jeffrey Ratcliffe (JIRA)

Jeffrey Ratcliffe created ZEPPELIN-2349:
---

 Summary: Graphs from Table Display are not rendered by IE 11
 Key: ZEPPELIN-2349
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2349
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end
Affects Versions: 0.7.0
 Environment: Server: Linux, Client: Windows 7, IE 11
Reporter: Jeffrey Ratcliffe


print(s"""%table
name\tsize
sun\t100
moon\t10""")

Prints the expected table, but the none of graphs are rendered when the 
appropriate button is pressed. The table is instead replaced by a blank area.

The problem has also been highlighted here:

https://community.hortonworks.com/questions/83204/does-zeppelin-support-internet-explorer-11.html



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] zeppelin pull request #2221: [ZEPPELIN-2216] general solution to precode. re...

2017-04-04 Thread tinkoff-dwh

GitHub user tinkoff-dwh opened a pull request:

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

[ZEPPELIN-2216] general solution to precode. refactoring jdbc precode

### What is this PR for?
General solution to execute precode. Refactoring jdbc precode using general 
solution. Task contains to subtasks: executeAfterOpen, executeBeforeClose. 
executeBeforeClose not done because we need the context so there is a solution 
only for executeAfterOpen. 

### What type of PR is it?
Feature | Refactoring

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

### How should this be tested?
1. Add parameter zeppelin.PySparkInterpreter.precode `someVar='text'`
2. Execute 
```
%pyspark
print(someVar)
```

### 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/tinkoff-dwh/zeppelin ZEPPELIN-2216

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

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


commit c0436a248ed67dab38630a5b6d437e723c9557dc
Author: Tinkoff DWH 
Date:   2017-04-04T06:42:54Z

[ZEPPELIN-2216] general solution to precode. refactoring jdbc precode




---
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 #2219: [ZEPPELIN-2348] Line chart setting is not rendered (br...

2017-04-04 Thread soralee

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2219
  
@1ambda @Leemoonsoo I also couldn't reproduce in branch-0.7. But I think 
the code should be modified!


---
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 #2216: [ZEPPELIN-2336] Fix note reload on another note create...

2017-04-04 Thread khalidhuseynov

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/2216
  
CI is green. i linked some of related issue in 
[ZEPPELIN-2336](https://issues.apache.org/jira/browse/ZEPPELIN-2336)


---
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 #2216: [ZEPPELIN-2336] Fix note reload on another note...

2017-04-04 Thread khalidhuseynov

Github user khalidhuseynov closed the pull request at:

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


---
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 #2216: [ZEPPELIN-2336] Fix note reload on another note...

2017-04-04 Thread khalidhuseynov

GitHub user khalidhuseynov reopened a pull request:

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

[ZEPPELIN-2336] Fix note reload on another note create/rename/remove

### What is this PR for?
This is to fix the issue with repeated note reload when some other note is 
created/renamed/removed


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

### Todos
* [x] - don't initialise controller on note list update

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

### How should this be tested?
Described in 
[ZEPPELIN-2336](https://issues.apache.org/jira/browse/ZEPPELIN-2336),
also additional test from 
[ZEPPELIN-1145](https://issues.apache.org/jira/browse/ZEPPELIN-1145) would be 
nice

### 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/khalidhuseynov/zeppelin fix/ZEPPELIN-2336

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

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


commit e53edc0db1b3314dc1cd606f2ab7d1d597836e53
Author: Khalid Huseynov 
Date:   2017-04-03T08:27:42Z

remove init controller on note_info

commit 74543368768ae6f30bb8d8e3769fff5ffe3e4e26
Author: Khalid Huseynov 
Date:   2017-04-04T07:09:19Z

fix js test




---
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 #2125: [ZEPPELIN-2222] Add Network Visualization

2017-04-04 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2125
  
Really sorry for late reply. I missed notification for a while. 

- Spell might use JSON output of neo4j interpreter utilizing 
[sigma.js](http://sigmajs.org/), 
[vis.js](http://visjs.org/network_examples.html)
- Having default network display system using generalized graph model 
(Property Graph Model) would be good idea. 
- Personally, hope to get more refined, pretty graph UI instead of d3 
network.








---
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 #2125: [ZEPPELIN-2222] Add Network Visualization

2017-04-04 Thread conker84

Github user conker84 commented on the issue:

https://github.com/apache/zeppelin/pull/2125
  
Yes it is!


---
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 #2216: [ZEPPELIN-2336] Fix note reload on another note create...

2017-04-04 Thread khalidhuseynov

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/2216
  
@Leemoonsoo i found some js tests failing in second profile, let me take a 
look


---
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 #2220: [Minor] Fix wrong Python lib path in Matplotlib...

2017-04-04 Thread AhyoungRyu

GitHub user AhyoungRyu opened a pull request:

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

[Minor] Fix wrong Python lib path in Matplotlib tutorial note

### What is this PR for?
There is a wrong information about Zeppelinâs matplotlib backend files. 
It actually located under `ZEPPELIN_HOME/interpreter/lib/python/*` but tutorial 
note says it as `ZEPPELIN_HOME/lib/python/*`. So I corrected it.

### What type of PR is it?
kind a typo fix 

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

### Screenshots (if appropriate)
 - Before 
![screen shot 2017-04-04 at 3 50 06 
pm](https://cloud.githubusercontent.com/assets/10060731/24644505/e573d3a0-194e-11e7-826f-4022bc5e1572.png)

 - After
![screen shot 2017-04-04 at 3 50 56 
pm](https://cloud.githubusercontent.com/assets/10060731/24644507/e6ae25c2-194e-11e7-91d7-f42b327d820e.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/AhyoungRyu/zeppelin 
fix/wrongPythonLibPathInTutorialNote

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

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


commit b43dc44ddd3e61e1291997fb50d38cdf50908396
Author: AhyoungRyu 
Date:   2017-04-04T06:47:21Z

Fix wrong Python lib path in tutorial note




---
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 #2219: [ZEPPELIN-2348] Line chart setting is not rendered (br...

2017-04-04 Thread 1ambda

Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2219
  
@Leemoonsoo I failed to reproduce it in branch-0.7, so it's ok to just 
close this PR if it's save. But code looks like buggy.


---
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 #2219: [ZEPPELIN-2348] Line chart setting is not rende...

2017-04-04 Thread 1ambda

GitHub user 1ambda opened a pull request:

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

[ZEPPELIN-2348] Line chart setting is not rendered (branch-0.7)

### What is this PR for?

Line chart setting is not rendered.

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

### Todos

None.

### What is the Jira issue?

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

### How should this be tested?

1. Open the default line chart's setting
2. Check checkboxes for options.

### Screenshots (if appropriate)

 After


![image](https://cloud.githubusercontent.com/assets/4968473/24643544/ad22b336-1949-11e7-95c4-5a5ba17f1d1b.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/1ambda/zeppelin 
ZEPPELIN-2348/line-chart-setting

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

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


commit 42165879af92af37079d064d2902218a55fe23b1
Author: 1ambda <1am...@gmail.com>
Date:   2017-04-04T06:44:28Z

fix: overwriting trSetting




---
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 #2216: [ZEPPELIN-2336] Fix note reload on another note create...

2017-04-04 Thread khalidhuseynov

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/2216
  
@Leemoonsoo actually I couldn't reproduce ZEPPELIN-1145, but we can reopen 
it to make sure. maybe also /cc @r-kamath as original author


---
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 #2216: [ZEPPELIN-2336] Fix note reload on another note create...

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2216
  
LGTM

I think ZEPPELIN-2336 is quite serious issue.
Once CI becomes, let's merge this PR asap to master and branch-0.7 as a 
hotfix, and reopen ZEPPELIN-1145 for further investigation.


---
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 #2218: [ZEPPELIN-2348] Line chart setting is not rendered (ma...

2017-04-04 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2218
  
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 #2216: [ZEPPELIN-2336] Fix note reload on another note...

2017-04-04 Thread khalidhuseynov

GitHub user khalidhuseynov reopened a pull request:

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

[ZEPPELIN-2336] Fix note reload on another note create/rename/remove

### What is this PR for?
This is to fix the issue with repeated note reload when some other note is 
created/renamed/removed


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

### Todos
* [x] - don't initialise controller on note list update

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

### How should this be tested?
Described in 
[ZEPPELIN-2336](https://issues.apache.org/jira/browse/ZEPPELIN-2336),
also additional test from 
[ZEPPELIN-1145](https://issues.apache.org/jira/browse/ZEPPELIN-1145) would be 
nice

### 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/khalidhuseynov/zeppelin fix/ZEPPELIN-2336

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

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


commit b568a6c80faf5d4441b16de75214acddc55abed4
Author: Khalid Huseynov 
Date:   2017-04-03T08:27:42Z

remove init controller on note_info




---
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 #2218: [ZEPPELIN-2348] Line chart setting is not rende...

2017-04-04 Thread 1ambda

GitHub user 1ambda opened a pull request:

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

[ZEPPELIN-2348] Line chart setting is not rendered (master)

### What is this PR for?

Line chart setting is not rendered.

### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

### Todos

None.

### What is the Jira issue?

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

### How should this be tested?

1. Open the default line chart's setting
2. Check checkboxes for options.

### Screenshots (if appropriate)


![image](https://cloud.githubusercontent.com/assets/4968473/24643544/ad22b336-1949-11e7-95c4-5a5ba17f1d1b.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/1ambda/zeppelin 
ZEPPELIN-2348/line-chart-setting-msater

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

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


commit 0f3ea04cb299d4080317b5bef1168d81f091ed07
Author: 1ambda <1am...@gmail.com>
Date:   2017-04-04T05:00:18Z

fix: Tr, Vis setting




---
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 #2213: [ZEPPELIN-2298] Remove -Ppyspark build profile

2017-04-04 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #2204: ZEPPELIN-2189. The order of dynamic forms shoul...

2017-04-04 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #2135: Groovy Interpreter for Apazhe Zeppelin [ZEPPELI...

2017-04-04 Thread asfgit

Github user asfgit closed the pull request at:

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


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

55 matches

Mail list logo