[GitHub] zeppelin issue #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

2016-10-11 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
CI is green now, merge if there is 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 issue #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

2016-10-07 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
LGTM, would be great to see Spark profiles passing in CI...



---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

2016-10-07 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
CI failures looks irrelevant, merging it into master.
profile 2
```
[32m- should able to create AngularObject
AngularModel
- should able to update AngularObject
AngularModel
- should able to remove AngularObject
AngularModelTest:
AngularModel
- should able to create AngularObject
AngularModel
- should able to update AngularObject
AngularModel
- should able to remove AngularObject
AngularElemTest:
AngularElem
- should provide onclick method *** FAILED ***
  The code passed to eventually never returned normally. Attempted 1 
times over 530.370455 milliseconds. Last failure message: 0 was not equal to 1. 
(AbstractAngularElemTest.scala:72)
AngularElem
- should print angular display directive only once in a paragraph
AngularElem
- should bind angularObject to ng-model directive
AngularElem
- should able to disassociate AngularObjects
AngularElem
- should allow access to InterpreterContext inside of callback 
function
AngularElem
- should able to be created from implicit conversion
AngularElemTest:
AngularElem
- should provide onclick method
AngularElem
- should print angular display directive only once in a paragraph
AngularElem
- should bind angularObject to ng-model directive
AngularElem
- should able to disassociate AngularObjects
AngularElem
- should allow access to InterpreterContext inside of callback 
function
AngularElem
- should able to be created from implicit conversion
Run completed in 2 seconds, 415 milliseconds.
Total number of tests run: 18
Suites: completed 5, aborted 0
Tests: succeeded 17, failed 1, canceled 0, ignored 0, pending 0
*** 1 TEST FAILED ***
```

Profile 10
```
Tests run: 9, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 108.789 sec 
<<< FAILURE! - in org.apache.zeppelin.integration.ParagraphActionsIT
testWidth(org.apache.zeppelin.integration.ParagraphActionsIT)  Time 
elapsed: 19.995 sec  <<< FAILURE!
java.lang.AssertionError: New Width is : 4
Expected: 
 but: was 
at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
at org.junit.Assert.assertThat(Assert.java:865)
at org.junit.rules.ErrorCollector$1.call(ErrorCollector.java:65)
at org.junit.rules.ErrorCollector.checkSucceeds(ErrorCollector.java:78)
at org.junit.rules.ErrorCollector.checkThat(ErrorCollector.java:63)
at 
org.apache.zeppelin.integration.ParagraphActionsIT.testWidth(ParagraphActionsIT.java:314)

testWidth(org.apache.zeppelin.integration.ParagraphActionsIT)  Time 
elapsed: 19.995 sec  <<< FAILURE!
java.lang.AssertionError: New Width is : 8
Expected: 
 but: was 
at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
at org.junit.Assert.assertThat(Assert.java:865)
at org.junit.rules.ErrorCollector$1.call(ErrorCollector.java:65)
at org.junit.rules.ErrorCollector.checkSucceeds(ErrorCollector.java:78)
at org.junit.rules.ErrorCollector.checkThat(ErrorCollector.java:63)
at 
org.apache.zeppelin.integration.ParagraphActionsIT.testWidth(ParagraphActionsIT.java:314)
```


---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

2016-10-06 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
@minahlee I just rebased. Thank you for your great efforts.


---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

2016-10-06 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
@astroshim #1483 is just merged, can you rebase master and see if CI 
failure on spark 1.1 and 1.2 is gone?


---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

2016-10-03 Thread astroshim
Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
Thank you @minahlee Please help me to solve PR problem.


---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

2016-10-03 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
Nice catch! I think it is better to fix CI failures in this PR since it 
will break CI if we merge this. I am willing to help you if you need any :)


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