[GitHub] zeppelin issue #3206: [WIP][ZEPPELIN-3810] Support Spark 2.4

2018-10-31 Thread ajaygk95
Github user ajaygk95 commented on the issue:

https://github.com/apache/zeppelin/pull/3206
  
> picked

I was able to get this working by doing cherry-picking. We needed some 
changes in our environment not related to zeppelin, but for the usage of 
spark-images.
Thanks for the help.


---


[GitHub] zeppelin issue #3206: [WIP][ZEPPELIN-3810] Support Spark 2.4

2018-10-29 Thread ajaygk95
Github user ajaygk95 commented on the issue:

https://github.com/apache/zeppelin/pull/3206
  
> 
> 
> The error message:
> 
> ```
> [ERROR] /home/cloud-user/ajay/code/csf-cc-zeppelin-k8szep/spark/scala-
> 
2.11/src/main/scala/org/apache/zeppelin/spark/SparkScala211Interpreter.scala:37:
 error: class 
> SparkScala211Interpreter needs to be abstract, since method interpret in 
class 
> BaseSparkScalaInterpreter of type (code: String, context: 
> 
org.apache.zeppelin.interpreter.InterpreterContext)org.apache.zeppelin.interpreter.InterpreterResult
 is 
> not defined
> ```
> 
> complains there's no `interpret` method defined at 
`BaseSparkScalaInterpreter`.

Thanks for the reply.
I found the issue. I was using zeppelin 0.8.0 source code where "interpret" 
method was overriden. I think in the current branch this method is not present.


---


[GitHub] zeppelin issue #3206: [WIP][ZEPPELIN-3810] Support Spark 2.4

2018-10-29 Thread ajaygk95
Github user ajaygk95 commented on the issue:

https://github.com/apache/zeppelin/pull/3206
  
> 
> 
> Does that happen only with this code changes? The change here does not 
touch signature at `class SparkScala211Interpreter(` and the error message 
looks pretty unrelated. The whole change proposed here does not change any 
signature.

Yes. With the original 0.8.0 source code I'm able to build. 


---


[GitHub] zeppelin issue #3206: [WIP][ZEPPELIN-3810] Support Spark 2.4

2018-10-29 Thread ajaygk95
Github user ajaygk95 commented on the issue:

https://github.com/apache/zeppelin/pull/3206
  
Hello,

I was trying to run zeppelin with spark 2.4. I have pulled your code.
While building zeppelin I'm facing the below issue,

```
[ERROR] 
/home/cloud-user/ajay/code/csf-cc-zeppelin-k8szep/spark/scala-2.11/src/main/scala/org/apache/zeppelin/spark/SparkScala211Interpreter.scala:37:
 error: class SparkScala211Interpreter needs to be abstract, since method 
interpret in class BaseSparkScalaInterpreter of type (code: String, context: 
org.apache.zeppelin.interpreter.InterpreterContext)org.apache.zeppelin.interpreter.InterpreterResult
 is not defined
[ERROR] class SparkScala211Interpreter(override val conf: SparkConf,
[ERROR]   ^
[ERROR] one error found
[INFO] 

```
Can you please help.
Thanks



---


[GitHub] zeppelin issue #3188: ZEPPELIN-3791: Bump-Up shiro version from 1.3.2 to 1.4...

2018-09-28 Thread ajaygk95
Github user ajaygk95 commented on the issue:

https://github.com/apache/zeppelin/pull/3188
  
> 
> 
    > @ajaygk95 Thanks for the contribution, please configure your travis to 
enable CI.

Hello,

I have enabled travis and i see that zeppelin build has passed (screenshot 
attached).
Can you please let me know the next steps as this is my first commit.. 


![zep-travis](https://user-images.githubusercontent.com/28923655/46195839-0762fe80-c323-11e8-9ccd-ef495c0da0ca.PNG)




---


[GitHub] zeppelin pull request #3188: ZEPPELIN-3791: Bump-Up shiro version from 1.3.2...

2018-09-25 Thread ajaygk95
GitHub user ajaygk95 opened a pull request:

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

ZEPPELIN-3791: Bump-Up shiro version from 1.3.2 to 1.4.0

### What is this PR for?
Bump Up the Shiro version in zeppelin from 1.3.2 to 1.4.0. Bumping up the 
Shiro version will help in using the newer pac4j versions. This will also help 
in integrating Zeppelin and OpenID Connect using latest versions of pac4j. 
Pac4j also mentions that it is compatible with Shiro 1.4 version.

### What type of PR is it?
Improvement

### What is the Jira issue?
ZEPPELIN-3791

### How should this be tested?
* Zeppelin build was successful with the shiro version update.
* Basic Shiro User Authentication tested and is working fine.


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

$ git pull https://github.com/ajaygk95/zeppelin master

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

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


commit dab87e058478adaaefe77edf9ef944e96ba85252
Author: G 
Date:   2018-09-25T11:35:05Z

ZEPPELIN-3791: Bump-Up shiro version from 1.3.2 to 1.4.0




---


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

2018-09-18 Thread ajaygk95
Github user ajaygk95 commented on the issue:

https://github.com/apache/zeppelin/pull/2373
  
> 
> 
> @SarunasG @1ambda @khalidhuseynov @andreaTP Hello, is there any doc of 
details for me to add libs in 0.7.3 and by changing the shiro.ini config file 
to support open id?

Hello @xixikaikai ,
I am using zeppelin 0.8.0 and with the below changes i am able to integrate 
it with keycloak.

Dependencies to be added in zeppelin-0.8.0 lib folder:
buji-pac4j-3.0.0.jar
lang-tag-1.4.3.jar
json-smart-1.3.1.jar
commons-lang3-3.5.jar
commons-collections4-4.1.jar
pac4j-core-2.3.1.jar
oauth2-oidc-sdk-5.24.2.jar
mail-1.4.7.jar
shiro-crypto-hash-1.4.0.jar
shiro-crypto-core-1.4.0.jar
shiro-crypto-cipher-1.4.0.jar
shiro-core-1.4.0.jar
shiro-config-ogdl-1.4.0.jar
shiro-config-core-1.4.0.jar
shiro-cache-1.4.0.jar
pac4j-oidc-2.3.1.jar
slf4j-api-1.7.25.jar
shiro-web-1.4.0.jar
shiro-lang-1.4.0.jar
shiro-event-1.4.0.jar


I have added the shiro.ini conf and dependency in the attachment.
[dep.txt](https://github.com/apache/zeppelin/files/2391716/dep.txt)
 



---


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

2018-09-08 Thread ajaygk95
Github user ajaygk95 commented on the issue:

https://github.com/apache/zeppelin/pull/2373
  
Hello,
Has anyone tried with zeppelin with keycloak ?


---