You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by prabhjyotsingh <gi...@git.apache.org> on 2016/02/16 05:30:10 UTC
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-187507531
if you see `No output has been received in the last 10 minutes` please let me know
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-187075747
Have tried it multiple times, I think we can merge this.
Only reason it failed in last few build was
`No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.`
At least it solves above two problem (https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-184818151).
Also, this will now take a ScreenShot and post it in logs, so next time if it fails for same we will have additional information to debug.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
commit e2f62057864884c24641bd9b93bd1f9b96a1b9af
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T07:29:14Z
change import Base64
commit d3437eff52d42e4c69df6fbb2649bc1268b87015
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:08:11Z
trying to make CI fail again
WebElement paragraph2Editor = driver.findElement(By.xpath(getParagraphXPath(10) + "//textarea"));
commit c2591aec374d28af514c668367f5409d5e38203c
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:36:02Z
fail should go away
commit 88c1eadf511f53cbf3c03d66cf558c0184f4f5f5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T18:31:45Z
correct x-path
commit c3dba4bb2a64f57cdb4a08d55d3e6aa035ae2b5d
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-19T06:19:04Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-185429223
@prabhjyotsingh I think this PR is just to trigger CI? Do you intend to include fixes for the 2 issues you have identified in 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
commit e2f62057864884c24641bd9b93bd1f9b96a1b9af
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T07:29:14Z
change import Base64
commit d3437eff52d42e4c69df6fbb2649bc1268b87015
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:08:11Z
trying to make CI fail again
WebElement paragraph2Editor = driver.findElement(By.xpath(getParagraphXPath(10) + "//textarea"));
commit c2591aec374d28af514c668367f5409d5e38203c
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:36:02Z
fail should go away
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-187507463
LGTM. 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-188083016
Let me look into it, but by screen shot, it looks like spark is down.
java.net.ConnectException: Connection refused
at java.net.PlainSocketImpl.socketConnect(Native Method)
at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:339)
at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:200)
at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:182)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:579)
at org.apache.thrift.transport.TSocket.open(TSocket.java:182)
at org.apache.zeppelin.interpreter.remote.ClientFactory.create(ClientFactory.java:51)
at org.apache.zeppelin.interpreter.remote.ClientFactory.create(ClientFactory.java:37)
at org.apache.commons.pool2.BasePooledObjectFactory.makeObject(BasePooledObjectFactory.java:60)
at org.apache.commons.pool2.impl.GenericObjectPool.create(GenericObjectPool.java:861)
at org.apache.commons.pool2.impl.GenericObjectPool.borrowObject(GenericObjectPool.java:435)
at org.apache.commons.pool2.impl.GenericObjectPool.borrowObject(GenericObjectPool.java:363)
at org.apache.zeppelin.interpreter.remote.RemoteInterpreterProcess.getClient(RemoteInterpreterProcess.java:155)
at org.apache.zeppelin.interpreter.remote.RemoteInterpreter.getFormType(RemoteInterpreter.java:280)
at org.apache.zeppelin.interpreter.LazyOpenInterpreter.getFormType(LazyOpenInterpreter.java:104)
at org.apache.zeppelin.notebook.Paragraph.jobRun(Paragraph.java:215)
at org.apache.zeppelin.scheduler.Job.run(Job.java:171)
at org.apache.zeppelin.scheduler.RemoteScheduler$JobRunner.run(RemoteScheduler.java:326)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
![download](https://cloud.githubusercontent.com/assets/674497/13276532/5bf10ce4-dae6-11e5-9a68-78a63129e266.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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-186083897
@felixcheung, Yes, I do intend these 2 get merged, as this is small improvement.
But I'm still trying to find why its failing only for "paragraph5Editor"
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
commit e2f62057864884c24641bd9b93bd1f9b96a1b9af
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T07:29:14Z
change import Base64
commit d3437eff52d42e4c69df6fbb2649bc1268b87015
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:08:11Z
trying to make CI fail again
WebElement paragraph2Editor = driver.findElement(By.xpath(getParagraphXPath(10) + "//textarea"));
commit c2591aec374d28af514c668367f5409d5e38203c
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:36:02Z
fail should go away
commit 88c1eadf511f53cbf3c03d66cf558c0184f4f5f5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T18:31:45Z
correct x-path
commit c3dba4bb2a64f57cdb4a08d55d3e6aa035ae2b5d
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-19T06:19:04Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit 4351484656febb844fd5046155aabaa137579ad7
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-20T16:46:34Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit c358707ffb6c34cdd9ec3a2c6a5ed4019defc5e5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-21T06:30:25Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit 2ea0d98416bb28286e8872a5d24c6664e4f78c67
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-22T05:55:33Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
# Conflicts:
# zeppelin-server/src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java
# zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java
# zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java
commit 257150599934208b5161f0e194fe6c798cff3931
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-22T07:14:20Z
replace System.out.println with LOG.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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
commit e2f62057864884c24641bd9b93bd1f9b96a1b9af
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T07:29:14Z
change import Base64
commit d3437eff52d42e4c69df6fbb2649bc1268b87015
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:08:11Z
trying to make CI fail again
WebElement paragraph2Editor = driver.findElement(By.xpath(getParagraphXPath(10) + "//textarea"));
commit c2591aec374d28af514c668367f5409d5e38203c
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:36:02Z
fail should go away
commit 88c1eadf511f53cbf3c03d66cf558c0184f4f5f5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T18:31:45Z
correct x-path
commit c3dba4bb2a64f57cdb4a08d55d3e6aa035ae2b5d
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-19T06:19:04Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit 4351484656febb844fd5046155aabaa137579ad7
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-20T16:46:34Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit c358707ffb6c34cdd9ec3a2c6a5ed4019defc5e5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-21T06:30:25Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit 2ea0d98416bb28286e8872a5d24c6664e4f78c67
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-22T05:55:33Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
# Conflicts:
# zeppelin-server/src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java
# zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java
# zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java
commit 257150599934208b5161f0e194fe6c798cff3931
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-22T07:14:20Z
replace System.out.println with LOG.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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-184818151
Found 2 issues:
- AbstractFunctionalSuite.scala : we were never closing the webDriver
- ZeppelinIT.java : this X-Path (//div[contains(@id,\"_text\") and @class=\"text\"]) seems to be performing better.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-188080562
In the post commit build, it looks like Selenium tests are failing because of "Connection refused" from screen shots. Any idea?
https://s3.amazonaws.com/archive.travis-ci.org/jobs/111391355/log.txt
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
commit e2f62057864884c24641bd9b93bd1f9b96a1b9af
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T07:29:14Z
change import Base64
commit d3437eff52d42e4c69df6fbb2649bc1268b87015
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:08:11Z
trying to make CI fail again
WebElement paragraph2Editor = driver.findElement(By.xpath(getParagraphXPath(10) + "//textarea"));
commit c2591aec374d28af514c668367f5409d5e38203c
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:36:02Z
fail should go away
commit 88c1eadf511f53cbf3c03d66cf558c0184f4f5f5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T18:31:45Z
correct x-path
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-184527678
Thanks - couple more instances in the last day:
https://s3.amazonaws.com/archive.travis-ci.org/jobs/109303040/log.txt
https://s3.amazonaws.com/archive.travis-ci.org/jobs/109303825/log.txt
This looks different though
https://s3.amazonaws.com/archive.travis-ci.org/jobs/109303816/log.txt
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
commit e2f62057864884c24641bd9b93bd1f9b96a1b9af
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T07:29:14Z
change import Base64
commit d3437eff52d42e4c69df6fbb2649bc1268b87015
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:08:11Z
trying to make CI fail again
WebElement paragraph2Editor = driver.findElement(By.xpath(getParagraphXPath(10) + "//textarea"));
commit c2591aec374d28af514c668367f5409d5e38203c
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:36:02Z
fail should go away
commit 88c1eadf511f53cbf3c03d66cf558c0184f4f5f5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T18:31:45Z
correct x-path
commit c3dba4bb2a64f57cdb4a08d55d3e6aa035ae2b5d
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-19T06:19:04Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit 4351484656febb844fd5046155aabaa137579ad7
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-20T16:46:34Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit c358707ffb6c34cdd9ec3a2c6a5ed4019defc5e5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-21T06:30:25Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit 2ea0d98416bb28286e8872a5d24c6664e4f78c67
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-22T05:55:33Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
# Conflicts:
# zeppelin-server/src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java
# zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java
# zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java
commit 257150599934208b5161f0e194fe6c798cff3931
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-22T07:14:20Z
replace System.out.println with LOG.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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/719
test testAngularDisplay failure
### What is this PR for?
There are few times when Selenium test case "testAngularDisplay" fails. This to reproduce it on travis and fix it.
### What type of PR is it?
Bug Fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/prabhjyotsingh/incubator-zeppelin testAngularDisplayFail
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/719.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 #719
----
commit fd9c921a05784487e768fe80492d6f490f32053f
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T04:28:40Z
test testAngularDisplay failure
commit e2f62057864884c24641bd9b93bd1f9b96a1b9af
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T07:29:14Z
change import Base64
commit d3437eff52d42e4c69df6fbb2649bc1268b87015
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:08:11Z
trying to make CI fail again
WebElement paragraph2Editor = driver.findElement(By.xpath(getParagraphXPath(10) + "//textarea"));
commit c2591aec374d28af514c668367f5409d5e38203c
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T09:36:02Z
fail should go away
commit 88c1eadf511f53cbf3c03d66cf558c0184f4f5f5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-16T18:31:45Z
correct x-path
commit c3dba4bb2a64f57cdb4a08d55d3e6aa035ae2b5d
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-19T06:19:04Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit 4351484656febb844fd5046155aabaa137579ad7
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-20T16:46:34Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit c358707ffb6c34cdd9ec3a2c6a5ed4019defc5e5
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-21T06:30:25Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
commit 2ea0d98416bb28286e8872a5d24c6664e4f78c67
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-22T05:55:33Z
Merge remote-tracking branch 'origin/master' into testAngularDisplayFail
# Conflicts:
# zeppelin-server/src/test/java/org/apache/zeppelin/AbstractZeppelinIT.java
# zeppelin-server/src/test/java/org/apache/zeppelin/integration/ParagraphActionsIT.java
# zeppelin-server/src/test/java/org/apache/zeppelin/integration/ZeppelinIT.java
commit 257150599934208b5161f0e194fe6c798cff3931
Author: Prabhjyot Singh <pr...@gmail.com>
Date: 2016-02-22T07:14:20Z
replace System.out.println with LOG.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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-185306185
CI is green, looks good
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-186020261
FYI - https://s3.amazonaws.com/archive.travis-ci.org/jobs/110282609/log.txt
```
02:08:26,726 INFO org.apache.zeppelin.WebDriverManager:172 - Download of firebug version: 31, successful
02:10:10,947 ERROR org.apache.zeppelin.integration.ZeppelinIT:199 - Exception in ZeppelinIT while testAngularDisplay
org.openqa.selenium.ElementNotVisibleException: Element is not currently visible and so may not be interacted with
Command duration or timeout: 8 milliseconds
```
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: test testAngularDisplay failure
Posted by prabhjyotsingh <gi...@git.apache.org>.
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-187568405
@felixcheung sure if I see it after https://github.com/apache/incubator-zeppelin/pull/727 gets merged, I'll let you know.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---