[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/706 --- 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 fea

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-15 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-184409223 merging today if no more comments --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-14 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-184011432 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

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-14 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-183846032 @felixcheung made relevant changes, merged with https://github.com/apache/incubator-zeppelin/commit/65b47e50549e8c64b5aff9a6ceec16668d1ca52e, and http

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-13 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-183822913 @doanduyhai I guess we could check on this - if it has become a frequent problem we could get travis to retry tests. I have not seen this much, on the ot

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-13 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-183683883 Thanks @felixcheung So I had a look into the log file, below is some extract: ``` DEBUG [17:21:08,728][] ACHILLES_DDL_SCRIPT@:execute

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-12 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-183580662 @doanduyhai job log is here https://travis-ci.org/apache/incubator-zeppelin/jobs/108074063 --- If your project is set up for it, you can reply to this e

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-12 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-183360306 @felixcheung The error about the CassandraInterpreterTest is because for unit test, I start an embedded Cassandra server and sends CQL queries to

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-12 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-183245267 So as you can see, it's re-running all interpreter tests when TEST_SELENIUM="true" https://s3.amazonaws.com/archive.travis-ci.org/jobs/108461599/log.t

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-10 Thread prabhjyotsingh
GitHub user prabhjyotsingh reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/706 Create seperate matrix for selenium test case ### What is this PR for? A lot of time travis CI breaks because of selenium test case, and at that point it becomes trivial to

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-182696007 actually, I think they are addressing different parts of it. It might still make sense to have a separate matrix for just UI tests - to clarify, I was sa

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-182692431 Closing this, as this is been taken care by https://github.com/apache/incubator-zeppelin/pull/708 --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-10 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/706 --- 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

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-182248074 Could you check out #708 ? We should avoid rerunning all tests for TEST_SELENIUM --- If your project is set up for it, you can reply to this email and h

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-09 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-182179778 @doanduyhai any idea with test failure? ``` DEBUG [17:21:25,783][] ACHILLES_DDL_SCRIPT@:executeScript SCRIPT : CREATE INDEX IF NOT

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-09 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/706#issuecomment-181975866 Build #3100.4 is failing (https://travis-ci.org/apache/incubator-zeppelin/jobs/108074063) for Results : Tests in error:

[GitHub] incubator-zeppelin pull request: Create seperate matrix for seleni...

2016-02-09 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/incubator-zeppelin/pull/706 Create seperate matrix for selenium test case ### What is this PR for? A lot of time travis CI breaks because of selenium test case, and at that point it becomes trivial to kn