[GitHub] zeppelin issue #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID Connect

2017-06-27 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/2373
  
thanks
merging if no more comment


---
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 #2447: [ZEPPELIN-2700] Provide a link to interpreter b...

2017-06-27 Thread 1ambda
Github user 1ambda closed the pull request at:

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


---
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 #2447: [ZEPPELIN-2700] Provide a link to interpreter b...

2017-06-27 Thread 1ambda
GitHub user 1ambda reopened a pull request:

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

[ZEPPELIN-2700] Provide a link to interpreter binding mode doc

### What is this PR for?

Provide a link to the interpreter binding mode documentation page in the 
interpreter page to help understand what this feature is for 
(`usage/interpreter/interpreter_binding_mode.html`)

- **documentation is not deployed to web yet. So clicking this icon will 
redirect you to 404 page**

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

### Todos

DONE

### What is the Jira issue?

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

### How should this be tested?

1. Open the interpreter page 
2. Check the links (URL, icon, ..)

### Screenshots (if appropriate)

 Before


![image](https://user-images.githubusercontent.com/4968473/27620029-ad687f7e-5c01-11e7-96fb-6e53243346f4.png)


![image](https://user-images.githubusercontent.com/4968473/27620030-b0390b24-5c01-11e7-8918-9db376e99b49.png)

 After


![image](https://user-images.githubusercontent.com/4968473/27620018-946c3f88-5c01-11e7-878b-7348ca4be5e8.png)


![image](https://user-images.githubusercontent.com/4968473/27620023-9f784f98-5c01-11e7-8d1a-160d52b98037.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-2700/provide-link-for-interpreter-binding-mode-doc

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

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


commit 2291e63a0945e310660f92a5c80bdcce13e817fe
Author: 1ambda <1am...@gmail.com>
Date:   2017-06-28T03:33:54Z

style: reindent html

commit 72d6c063579c98df18f0672eb037a60f0af1fb45
Author: 1ambda <1am...@gmail.com>
Date:   2017-06-28T03:37:23Z

fix: Remove useless margin-bottom

commit 41261239dcb1f4e5f16b5fd01bafcb79c286ad8e
Author: 1ambda <1am...@gmail.com>
Date:   2017-06-28T03:55:24Z

feat: Add link to interpreter mode doc




---
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 #2444: [ZEPPELIN-2691] fix: should display setting for table

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2444
  
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 pull request #2447: [ZEPPELIN-2700] Provide a link to interpreter b...

2017-06-27 Thread 1ambda
GitHub user 1ambda opened a pull request:

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

[ZEPPELIN-2700] Provide a link to interpreter binding mode doc

### What is this PR for?

Provide a link to the interpreter binding mode documentation page in the 
interpreter page to help understand what this feature is for 
(`usage/interpreter/interpreter_binding_mode.html`)

- **documentation is not deployed to web yet. So clicking this icon will 
redirect to you to 404 page**

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

### Todos

DONE

### What is the Jira issue?

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

### How should this be tested?

1. Open the interpreter page 
2. Check the links (URL, icon, ..)

### Screenshots (if appropriate)

 Before


![image](https://user-images.githubusercontent.com/4968473/27620029-ad687f7e-5c01-11e7-96fb-6e53243346f4.png)


![image](https://user-images.githubusercontent.com/4968473/27620030-b0390b24-5c01-11e7-8918-9db376e99b49.png)

 After


![image](https://user-images.githubusercontent.com/4968473/27620018-946c3f88-5c01-11e7-878b-7348ca4be5e8.png)


![image](https://user-images.githubusercontent.com/4968473/27620023-9f784f98-5c01-11e7-8d1a-160d52b98037.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-2700/provide-link-for-interpreter-binding-mode-doc

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

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


commit 2291e63a0945e310660f92a5c80bdcce13e817fe
Author: 1ambda <1am...@gmail.com>
Date:   2017-06-28T03:33:54Z

style: reindent html

commit 72d6c063579c98df18f0672eb037a60f0af1fb45
Author: 1ambda <1am...@gmail.com>
Date:   2017-06-28T03:37:23Z

fix: Remove useless margin-bottom

commit 41261239dcb1f4e5f16b5fd01bafcb79c286ad8e
Author: 1ambda <1am...@gmail.com>
Date:   2017-06-28T03:55:24Z

feat: Add link to interpreter mode doc




---
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 #2436: [ZEPPELIN-2690] fix: should respect helium vis ...

2017-06-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2437
  
@jongyoul Thanks! 

I fixed the incorrect description. Now it looks like


![image](https://user-images.githubusercontent.com/4968473/27618720-f5f11c46-5bf8-11e7-9fd7-c9cd978582ae.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 #2444: [ZEPPELIN-2691] fix: should display setting for table

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2444
  
@jongyoul Thanks for the 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 #2444: [ZEPPELIN-2691] fix: should display setting for table

2017-06-27 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/2444
  
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 #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-06-27 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/2437
  
Yes, correct. Except `Shared`, user cannot share anything directly. 


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


[GitHub] zeppelin issue #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2437
  
@zjffdu yes, it would be more clear. 

@jongyoul. As zjffdu said, variables (e.g scala variables, ...) in the 
scope mode can't be shared directly. is it correct?


---
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 #2394: [gh-pages] [ZEPPELIN-2597] adding technologies and dep...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2394
  
@soralee Could you resolve conflict?


---
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 #2401: [gh-pages] [ZEPPELIN-2597] updating navbar and ...

2017-06-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #2444: [ZEPPELIN-2691] fix: should display setting for table

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2444
  
CI failed but irrelevant.

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

```

Results :

Tests in error: 
  
ZeppelinIT.testSparkInterpreterDependencyLoading:249->AbstractZeppelinIT.waitForParagraph:68->AbstractZeppelinIT.pollingWait:94
 » Timeout
  ParagraphActionsIT.testRunOnSelectionChange:284 » NoSuchElement Unable 
to loca...
  SparkParagraphIT.testPySpark:157 » NoSuchElement Unable to locate 
element: {"m...

Tests run: 24, Failures: 0, Errors: 3, Skipped: 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.
---


[GitHub] zeppelin issue #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2437
  
CI failed but irrelevant.


---
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 #2446: [ZEPPELIN-2699] Helium REST API bug-fixes. Also update...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2446
  
Could you write test 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 #2440: [ZEPPELIN-2587] allow logging in if you're anonymous

2017-06-27 Thread herval
Github user herval commented on the issue:

https://github.com/apache/zeppelin/pull/2440
  
@zjffdu added one, let me know if that covers 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 #2407: [ZEPPELIN-1907] Shell Interpreter does not renew ticke...

2017-06-27 Thread Tagar
Github user Tagar commented on the issue:

https://github.com/apache/zeppelin/pull/2407
  
Great addition. Thank you.
Btw, `renew_lifetime` although can be set in `krb5.conf`, it's maximum 
value is limited on Active Directory side if AD is used for Kerberos, look for 
example [Kerberos 
Policy](https://technet.microsoft.com/en-us/library/cc757692(v=ws.10).aspx#w2k3tr_sepol_accou_set_hpjo)
 .
Might be worth adding to the documentation?

> A user whoever configures Zeppelin with this configuration i.e. have a 
keytab/principal configured and enable "user impersonation", is aware that this 
keytab file is being shared with various users (which is not the default 
interpreter setting that we ship with Zeppelin), and if (s)he doesn't want this 
behaviour they can always fall back to the default option (which is not 
configuring any).

Would be great if Zeppelin would launch user's own Zeppelin interpreter 
processes under their own uid through setuid() call. So then keytabs could be 
locked down to be accessible to that one user. For example, after I 
LDAP-authenticated as "tagar" user, Zeppelin will drop down uid to tagar user 
and its keytab will have unix access bits set to `0600`. Makes sense? This 
probably should be part of another jira though. 


---
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 #2446: [ZEPPELIN-2699] Helium REST API bug-fixes. Also...

2017-06-27 Thread necosta
GitHub user necosta opened a pull request:

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

[ZEPPELIN-2699] Helium REST API bug-fixes. Also updated documentation

### What is this PR for?
Fixing bug on enabling Helium plugins (artifact is empty, not null) when 
sending curl command.
After noticing out-of-date documentation, I updated it
Also noticed issues on missing try/catch expressions and missing info 
messages on enable/disable packages.

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

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

### How should this be tested?
Run "curl -X POST http://localhost:8080/api/helium/enable/{packageName};
Check that the package gets enabled

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


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

$ git pull https://github.com/nokia/zeppelin zeppelin2699

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

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


commit eb5a163a3c6d82f85410cb57b1fae0ca2aa0aad3
Author: Nelson Costa 
Date:   2017-06-27T20:15:11Z

[ZEPPELIN-2699] Helium REST API bug-fixes. Also updated documentation




---
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-2699) Helium - REST APIs bug-fixes + Updated documentation

2017-06-27 Thread Nelson Costa (JIRA)
Nelson Costa created ZEPPELIN-2699:
--

 Summary: Helium - REST APIs bug-fixes + Updated documentation
 Key: ZEPPELIN-2699
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2699
 Project: Zeppelin
  Issue Type: Bug
  Components: helium, rest-api
Affects Versions: 0.8.0
Reporter: Nelson Costa
Assignee: Nelson Costa
Priority: Minor
 Fix For: 0.8.0


When I was using the enable-Helium-plugin REST API, I noticed a bug.
Specifically, "artifact" parameter is empty (not null) when sending command
{noformat}
curl -X POST http://localhost:8080/api/helium/enable/{packageName}
{noformat}
This causes a bug down the line, where the code does not pick up the correct 
package.

After fixing this issue, I noticed the documentation didn't reflect the REST 
API at all. I fixed this.
Also, noticed missing info messages as well as missing try/catch exceptions on 
REST API code. I have fixed that as well.



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


[GitHub] zeppelin issue #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID Connect

2017-06-27 Thread andreaTP
Github user andreaTP commented on the issue:

https://github.com/apache/zeppelin/pull/2373
  
ok, now also CI is happy  :-)


---
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 #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID C...

2017-06-27 Thread andreaTP
Github user andreaTP closed the pull request at:

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


---
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 #2445: ZEPPELIN-2674 Ignite version upgraded up to 2.0

2017-06-27 Thread agura
GitHub user agura opened a pull request:

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

ZEPPELIN-2674 Ignite version upgraded up to 2.0

### What is this PR for?
Apache Ignite 2.0 brings many improvements and new SQL related 
functionality. This PR upgrades Ignite version for Ignite interpreter.

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

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

### How should this be tested?
Unit-tests are enough.


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

$ git pull https://github.com/agura/incubator-zeppelin zeppelin-2674

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

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






---
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 #2435: [ZEPPELIN-2688]. Upgrade ace to 1.2.7

2017-06-27 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2435
  
Thanks for review @1ambda 


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


[GitHub] zeppelin pull request #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID C...

2017-06-27 Thread andreaTP
Github user andreaTP closed the pull request at:

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


---
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 #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID C...

2017-06-27 Thread andreaTP
GitHub user andreaTP reopened a pull request:

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

[ZEPPELIN-2598] Securing Zeppelin with OpenID Connect

### What is this PR for?
Integrating Open ID connect login into Zeppelin leveraging Shiro(already 
present) and Pac4J( that needs to be in the classpath).
Modifications done here should not affect any existing mechanisms but 
simply integrates and enable new once.

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

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

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

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

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






---
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 #2435: [ZEPPELIN-2688]. Upgrade ace to 1.2.7

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2435
  
Tested locally and works well.


![image](https://user-images.githubusercontent.com/4968473/27584400-df1bd762-5b72-11e7-8bac-2ee577c01a1c.png)

Here is a comparison between two versions. 

- https://github.com/ajaxorg/ace/compare/v1.2.6...v1.2.7

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 #2436: [ZEPPELIN-2690] fix: should respect helium vis order i...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2436
  
CI failed but irrelevant. 

```java

Results :

Tests in error: 
  
ZeppelinIT.testSparkInterpreterDependencyLoading:249->AbstractZeppelinIT.waitForParagraph:68->AbstractZeppelinIT.pollingWait:94
 » Timeout

Tests run: 24, Failures: 0, Errors: 1, Skipped: 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.
---


[GitHub] zeppelin pull request #2444: [ZEPPELIN-2691] fix: should display setting for...

2017-06-27 Thread 1ambda
GitHub user 1ambda opened a pull request:

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

[ZEPPELIN-2691] fix: should display setting for table

### What is this PR for?

The table has few settings, but currently, the menu is not displayed. 

I attached screenshots for comparison.

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

### Todos

DONE

### What is the Jira issue?

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

### How should this be tested?

Create a table result and check whether `setting` menu is displayed or not.

```scala
print("%table a\tb\n1\t2")
```

### Screenshots (if appropriate)

 Before


![image](https://user-images.githubusercontent.com/4968473/27583766-74e42c70-5b70-11e7-9cb7-537f40f1b699.png)

 After


![image](https://user-images.githubusercontent.com/4968473/27583753-6accc828-5b70-11e7-85f2-c71e53bbda80.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-2961/should-show-setting-for-table

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

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


commit 14c017fee70dcddc0eb5c5a4215a14a00c4988c4
Author: 1ambda <1am...@gmail.com>
Date:   2017-06-27T10:36:50Z

fix: Display setting for table




---
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 #2435: [ZEPPELIN-2688]. Upgrade ace to 1.2.7

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2435
  
Let me review and comment soon.


---
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 #2436: [ZEPPELIN-2690] fix: should respect helium vis order i...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2436
  
just rebased since https://github.com/apache/zeppelin/pull/2424 was merged. 
(the base branch of this PR)


---
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 #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2437
  
@zjffdu Thanks for the review.

Now it looks like


![image](https://user-images.githubusercontent.com/4968473/27581456-bd729506-5b68-11e7-9ccf-7ccf5545bf89.png)


![image](https://user-images.githubusercontent.com/4968473/27581463-c0ed8754-5b68-11e7-9502-619c27bceec5.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 #2424: [ZEPPELIN-2670] fix: DON'T reset all helium con...

2017-06-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #2415: [ZEPPELIN-2661]Test: personalized mode action

2017-06-27 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2415
  
Yeah, make sense. Let's fix the flaky test step by step. 


---
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 #2443: [ZEPPELIN-2698] Apply KerberosInterpreter to JD...

2017-06-27 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

[ZEPPELIN-2698] Apply KerberosInterpreter to JDBCInterpreter

### What is this PR for?
This is to apply new KerberosInterpreter mechanism to JDBCInterpreter for 
relogin from keytab, instead of on fail relogin.



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

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

### How should this be tested?
In JDBC interpreter setting add following properties
- zeppelin.jdbc.auth.type = KERBEROS
- zeppelin.jdbc.principal = principal value
- zeppelin.jdbc.keytab.location = keytab location

Now try and run any of hive's query (say `show tables`) it should return 
with valid results.
Again, wait for this kerberos ticket to expire (usually its 24hrs) then try 
the above again, and it should work.

### 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/prabhjyotsingh/zeppelin ZEPPELIN-2698

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

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


commit 57ea80c0c355752304d0eb7dabc9452ceeebc6c3
Author: Prabhjyot Singh 
Date:   2017-06-27T08:55:13Z

apply KerberosInterpreter to JDBCInterpreter




---
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 #2437: [ZEPPELIN-2582][DOCS] docs for interpreter binding mod...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2437
  
@zjffdu enhanced description according to your suggestion. could you check 
that?


![image](https://user-images.githubusercontent.com/4968473/27579533-d75f5a90-5b62-11e7-869c-266c2af62999.png)


![image](https://user-images.githubusercontent.com/4968473/27579536-db94d644-5b62-11e7-9adc-2c4e593091b3.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.
---


[jira] [Created] (ZEPPELIN-2698) Apply KerberosInterpreter to JDBCInterpreter

2017-06-27 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-2698:
-

 Summary: Apply KerberosInterpreter to JDBCInterpreter
 Key: ZEPPELIN-2698
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2698
 Project: Zeppelin
  Issue Type: Bug
Reporter: Prabhjyot Singh
Assignee: Prabhjyot Singh


This is to apply new KerberosInterpreter mechanism to JDBCInterpreter for 
relogin from keytab, instead of on fail relogin.



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


[jira] [Created] (ZEPPELIN-2697) allow specification of cassandra user/passwd settings in a paragraph

2017-06-27 Thread Luiz Angelo Steffenel (JIRA)
Luiz Angelo Steffenel created ZEPPELIN-2697:
---

 Summary: allow specification of cassandra user/passwd settings in 
a paragraph
 Key: ZEPPELIN-2697
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2697
 Project: Zeppelin
  Issue Type: Improvement
  Components: zeppelin-interpreter
Affects Versions: 0.7.2, 0.8.0
Reporter: Luiz Angelo Steffenel
Priority: Trivial


Currently the authentification (user/password) for the cassandra interpreter is 
tied to the global interpreter parameters (cassandra.credentials.username and 
cassandra.credentials.password). In a multi-user environment this is an issue, 
as we are still unable to have a different configuration per user (see 
https://issues.apache.org/jira/browse/ZEPPELIN-1337). 

It would be interesting to allow users set their own authentification 
parameters in a paragraph and use these values instead of the default ones. 
Such verification could be triggered for example by the parsion of the "use 
" command.

Even after the ZEPPELIN-1337 issue is solved, this would allow users to 
temporarily modify cassandra auth parameters without having to change their 
main configuration (for example, when testing grant rights).  





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


[GitHub] zeppelin issue #2394: [gh-pages] [ZEPPELIN-2597] adding technologies and dep...

2017-06-27 Thread soralee
Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/2394
  
Yeah, I updated it (: 
thanks @1ambda 


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


[jira] [Created] (ZEPPELIN-2696) strange dynamic form action in personalized mode

2017-06-27 Thread Sora Lee (JIRA)
Sora Lee created ZEPPELIN-2696:
--

 Summary: strange dynamic form action in personalized mode
 Key: ZEPPELIN-2696
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2696
 Project: Zeppelin
  Issue Type: Bug
Reporter: Sora Lee


Sometimes dynamic form is disappeared in personalized mode.
Here is scenario.
{quote}
user1 : login, run dynamic form paragraph, turn on personalized mode, logout
user2 : login, run dynamic form paragraph, if check dynamic form is appeared or 
not (=> dynamic from is disappeared), logout.
user1 : login, if check dynamic form paragraph is appeared or not (=> it was 
disappeared as well), logout
user2 : login, check if dynamic form is appeared or not. in my case it was 
appeared.
{quote}

And this is my test code
{quote}
%spark println("Status: "+ z.textbox("name", "Before"))
{quote}




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


[jira] [Created] (ZEPPELIN-2695) two bugs of graph mode in personalized mode

2017-06-27 Thread Sora Lee (JIRA)
Sora Lee created ZEPPELIN-2695:
--

 Summary: two bugs of graph mode in personalized mode
 Key: ZEPPELIN-2695
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2695
 Project: Zeppelin
  Issue Type: Bug
Reporter: Sora Lee
Assignee: Sora Lee


Here is two scenarios.
{quote}
1. user1 : login, run graph paragraph, change active mode, turn on personalized 
mode, logout
2. user2 : login, change active mode(not change), refresh window and check 
active mode (be changed) => it should be not changed, but result is changed
{quote}
{quote}
1. user1: login, run graph paragraph, change active mode, turn on personalized 
mode, logout
2. user2 : login, run graph paragraph, logout
3. user1 : login, change active graph, logout
4. user2 : login, check active graph => changed; it should be not changed, but 
result is changed
{quote}
I'm not sure this bugs is a same issue.



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


[GitHub] zeppelin pull request #2423: [ZEPPELIN-2672] Fix interpreter repos url toolt...

2017-06-27 Thread necosta
GitHub user necosta reopened a pull request:

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

[ZEPPELIN-2672] Fix interpreter repos url tooltip

### What is this PR for?
Fixing regression on showing interpreter repos tooltip

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

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

### How should this be tested?
Zeppelin > Interpreter > Repository > (Mouse over) central/local/other

### Screenshots 
After this fix:

![image](https://user-images.githubusercontent.com/26248959/27374071-6be20736-5662-11e7-9f30-7514fe342821.png)


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


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

$ git pull https://github.com/nokia/zeppelin zeppelin2672

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

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


commit c2fa47afb01ed9a7cab9b88fb637455b8eed2259
Author: Nelson Costa 
Date:   2017-06-20T18:49:48Z

[ZEPPELIN-2672] Fix interpreter repos url tooltip

commit 2d6514f4a5e89248657403167fae38d4cbfe179f
Author: Nelson Costa 
Date:   2017-06-21T14:34:58Z

[ZEPPELIN-2672] Also fixed column selector

commit 5413c4ba26577d50b2b0aae83fdbf6cd4c3d7b43
Author: Nelson Costa 
Date:   2017-06-23T08:29:15Z

[ZEPPELIN-2672] Remove popover directive files

commit 50ce1c7b57c2a2418d079cf0277ddf60c1f95262
Author: Nelson Costa 
Date:   2017-06-27T07:00:20Z

[ZEPPELIN-2672] Final adjusments




---
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 #2423: [ZEPPELIN-2672] Fix interpreter repos url toolt...

2017-06-27 Thread necosta
Github user necosta closed the pull request at:

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


---
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 #2394: [gh-pages] [ZEPPELIN-2597] adding technologies and dep...

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue:

https://github.com/apache/zeppelin/pull/2394
  
Looks good to me and agree that use old links unile we deploy 0.8.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.
---