[GitHub] zeppelin pull request #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

### What is this PR for?
This PR fix several issues of LivyInterpeter test. 
* Livy interpreter's test code is not in the right place, so it never runs.
* LivyHelperTest would fail.
* No Integration test for LivyInterpreter so any following change is not 
easy to be tested.

This PR would fix the above issues. Regarding the integration test, some of 
livy's artifact is not available in repository, so I have to copy them to 
livy/local-maven-repo as local repository.  And LivyInterperter's integration 
test require spark and livy to be installed. For now you have to download spark 
and livy manually. Please use spark 1.5.x and livy 0.2 which is currently 
supported. Download livy 0.2.0 from here 
[https://github.com/cloudera/livy/releases](https://github.com/cloudera/livy/releases)

And use the following command to execute the LivyIntegrationTest.
```
export SPARK_HOME=
export LIVY_HOME=
mvn clean package -pl 'livy' -Dtest=LivyIntegrationTest
```
If you hit any issues, you can check logs under `livy/target/tmp`.

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

### Todos
* [ ] - Task

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

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18861677/b1389622-84b9-11e6-8b0a-424457ded975.png)

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



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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1477

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

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


commit f560a92979faff27d96895abc054878035ccf519
Author: Jeff Zhang 
Date:   2016-09-22T08:42:14Z

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

commit 86324666c43a81a1578b44e324c541902aa7ff15
Author: Jeff Zhang 
Date:   2016-09-27T06:35:15Z

fix rat check

commit e8ceff5eee3c1e8e83c4505be63aa430e8e9b7bf
Author: Jeff Zhang 
Date:   2016-09-27T06:50:45Z

add missing livy jars

commit 1b9fbbc6256f803fa8b778b9b744c64751bb8969
Author: Jeff Zhang 
Date:   2016-09-27T08:04:24Z

add missing dependencies

commit fee61f9b6d12134df193710035a23d78029c89ca
Author: Jeff Zhang 
Date:   2016-10-07T06:18:11Z

add more 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 pull request #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

### What is this PR for?
This PR fix several issues of LivyInterpeter test. 
* Livy interpreter's test code is not in the right place, so it never runs.
* LivyHelperTest would fail.
* No Integration test for LivyInterpreter so any following change is not 
easy to be tested.

This PR would fix the above issues. Regarding the integration test, some of 
livy's artifact is not available in repository, so I have to copy them to 
livy/local-maven-repo as local repository.  And LivyInterperter's integration 
test require spark and livy to be installed. For now you have to download spark 
and livy manually. Please use spark 1.5.x and livy 0.2 which is currently 
supported. Download livy 0.2.0 from here 
[https://github.com/cloudera/livy/releases](https://github.com/cloudera/livy/releases)

And use the following command to execute the LivyIntegrationTest.
```
export SPARK_HOME=
export LIVY_HOME=
mvn clean package -pl 'livy' -Dtest=LivyIntegrationTest
```
If you hit any issues, you can check logs under `livy/target/tmp`.

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

### Todos
* [ ] - Task

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

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18861677/b1389622-84b9-11e6-8b0a-424457ded975.png)

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



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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1477

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

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


commit 7e1cf6c003693d5a06826884ceccfc0affb9aaa7
Author: Jeff Zhang 
Date:   2016-09-22T08:42:14Z

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

commit e2f1115f0e0b0adafb628363cb8ea5bca769bf36
Author: Jeff Zhang 
Date:   2016-09-27T06:35:15Z

fix rat check

commit 6a7caac18431cc149f4bc2e34d2d5cd78ac5559f
Author: Jeff Zhang 
Date:   2016-09-27T06:50:45Z

add missing livy jars

commit b9abcc5822fe416feabf6f6cc619f070002bb322
Author: Jeff Zhang 
Date:   2016-09-27T08:04:24Z

add missing dependencies

commit 0b350c0b388d3f48d5504acb5ef67f38d6d877c1
Author: Jeff Zhang 
Date:   2016-10-07T06:18:11Z

add more 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 pull request #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

### What is this PR for?
This PR fix several issues of LivyInterpeter test. 
* Livy interpreter's test code is not in the right place, so it never runs.
* LivyHelperTest would fail.
* No Integration test for LivyInterpreter so any following change is not 
easy to be tested.

This PR would fix the above issues. Regarding the integration test, some of 
livy's artifact is not available in repository, so I have to copy them to 
livy/local-maven-repo as local repository.  And LivyInterperter's integration 
test require spark and livy to be installed. For now you have to download spark 
and livy manually. Please use spark 1.5.x and livy 0.2 which is currently 
supported. Download livy 0.2.0 from here 
[https://github.com/cloudera/livy/releases](https://github.com/cloudera/livy/releases)

And use the following command to execute the LivyIntegrationTest.
```
export SPARK_HOME=
export LIVY_HOME=
mvn clean package -pl 'livy' -Dtest=LivyIntegrationTest
```
If you hit any issues, you can check logs under `livy/target/tmp`.

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

### Todos
* [ ] - Task

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

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18861677/b1389622-84b9-11e6-8b0a-424457ded975.png)

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



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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1477

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

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


commit 7e1cf6c003693d5a06826884ceccfc0affb9aaa7
Author: Jeff Zhang 
Date:   2016-09-22T08:42:14Z

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

commit e2f1115f0e0b0adafb628363cb8ea5bca769bf36
Author: Jeff Zhang 
Date:   2016-09-27T06:35:15Z

fix rat check

commit 6a7caac18431cc149f4bc2e34d2d5cd78ac5559f
Author: Jeff Zhang 
Date:   2016-09-27T06:50:45Z

add missing livy jars

commit b9abcc5822fe416feabf6f6cc619f070002bb322
Author: Jeff Zhang 
Date:   2016-09-27T08:04:24Z

add missing dependencies

commit 0b350c0b388d3f48d5504acb5ef67f38d6d877c1
Author: Jeff Zhang 
Date:   2016-10-07T06:18:11Z

add more 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 pull request #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

### What is this PR for?
This PR fix several issues of LivyInterpeter test. 
* Livy interpreter's test code is not in the right place, so it never runs.
* LivyHelperTest would fail.
* No Integration test for LivyInterpreter so any following change is not 
easy to be tested.

This PR would fix the above issues. Regarding the integration test, some of 
livy's artifact is not available in repository, so I have to copy them to 
livy/local-maven-repo as local repository.  And LivyInterperter's integration 
test require spark and livy to be installed. For now you have to download spark 
and livy manually. Please use spark 1.5.x and livy 0.2 which is currently 
supported. Download livy 0.2.0 from here 
[https://github.com/cloudera/livy/releases](https://github.com/cloudera/livy/releases)

And use the following command to execute the LivyIntegrationTest.
```
export SPARK_HOME=
export LIVY_HOME=
mvn clean package -pl 'livy' -Dtest=LivyIntegrationTest
```
If you hit any issues, you can check logs under `livy/target/tmp`.

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

### Todos
* [ ] - Task

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

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18861677/b1389622-84b9-11e6-8b0a-424457ded975.png)

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



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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1477

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

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


commit 152d346bb89f9b2de1c0f844ca53c93d8e9301ee
Author: Jeff Zhang 
Date:   2016-09-22T08:42:14Z

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

commit 3d02606642c2892ffc5a910a718145c6fb629ed7
Author: Jeff Zhang 
Date:   2016-09-27T06:35:15Z

fix rat check

commit 80657633ae18c50b556100da4360d81c57cfbc1a
Author: Jeff Zhang 
Date:   2016-09-27T06:50:45Z

add missing livy jars




---
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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

Github user zjffdu closed the pull request at:

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


---
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 #1462: ZEPPELIN-1477. Add Integration Test for LivyInt...

[zjffdu]

GitHub user zjffdu opened a pull request:

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

ZEPPELIN-1477. Add Integration Test for LivyInterpreter

### What is this PR for?
This PR fix several issues of LivyInterpeter test. 
* Livy interpreter's test code is not in the right place, so it never runs.
* LivyHelperTest would fail.
* No Integration test for LivyInterpreter so any following change is not 
easy to be tested.

This PR would fix the above issues. Regarding the integration test, some of 
livy's artifact is not available in repository, so I have to copy them to 
livy/local-maven-repo as local repository.  And LivyInterperter's integration 
test require spark and livy to be installed. For now you have to download spark 
and livy manually. Please use spark 1.5.x and livy 0.2. Download livy 0.2.0 
from here https://github.com/cloudera/livy/releases

And use the following command to execute the LivyIntegrationTest.
```
export SPARK_HOME=<>
export LIVY_HOME=<>
mvn clean package -pl 'livy' -Dtest=LivyIntegrationTest
```
If you hit any issues, you can check logs under `livy/target/tmp`.

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

### Todos
* [ ] - Task

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

### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/164491/18861677/b1389622-84b9-11e6-8b0a-424457ded975.png)

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



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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-1477

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

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


commit 152d346bb89f9b2de1c0f844ca53c93d8e9301ee
Author: Jeff Zhang 
Date:   2016-09-22T08:42:14Z

ZEPPELIN-1477. Add Integration Test for LivyInterpreter




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