[ 
https://issues.apache.org/jira/browse/FLINK-5101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15834417#comment-15834417
 ] 

ASF GitHub Bot commented on FLINK-5101:
---------------------------------------

Github user StephanEwen commented on a diff in the pull request:

    https://github.com/apache/flink/pull/2866#discussion_r97310568
  
    --- Diff: 
flink-streaming-connectors/flink-connector-cassandra/src/test/java/org/apache/flink/streaming/connectors/cassandra/CassandraConnectorITCase.java
 ---
    @@ -173,39 +157,49 @@ public static void startCassandra() throws 
IOException {
                        cassandra.start();
                }
     
    -           try {
    -                   Thread.sleep(1000 * 10);
    -           } catch (InterruptedException e) { //give cassandra a few 
seconds to start up
    +           // give cassandra a few seconds to start up
    +           long start = System.currentTimeMillis();
    +           long deadline = start + 1000 * 10;
    +           while (System.currentTimeMillis() < deadline) {
    +                   try {
    +                           Thread.sleep(1000);
    +                   } catch (InterruptedException ignored) {
    +                   }
                }
     
    -           cluster = builder.getCluster();
    -           session = cluster.connect();
    +           // start establishing a connection within 30 seconds
    +           start = System.currentTimeMillis();
    --- End diff --
    
    I would suggest to use `System.nanoTime()`, because 
`System.curremtTimeMillis` is not stable. Seems especially unstable on Travis.


> Test CassandraConnectorITCase instable
> --------------------------------------
>
>                 Key: FLINK-5101
>                 URL: https://issues.apache.org/jira/browse/FLINK-5101
>             Project: Flink
>          Issue Type: Bug
>          Components: Cassandra Connector
>            Reporter: Stefan Richter
>            Assignee: Chesnay Schepler
>
> I observed this test fail on Travis (very rarely):
>  
>  Running 
> org.apache.flink.streaming.connectors.cassandra.CassandraConnectorITCase
> Tests run: 7, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 80.843 sec 
> <<< FAILURE! - in 
> org.apache.flink.streaming.connectors.cassandra.CassandraConnectorITCase
> testCassandraBatchFormats(org.apache.flink.streaming.connectors.cassandra.CassandraConnectorITCase)
>   Time elapsed: 5.82 sec  <<< FAILURE!
> java.lang.AssertionError: expected:<40> but was:<20>
>       at org.junit.Assert.fail(Assert.java:88)
>       at org.junit.Assert.failNotEquals(Assert.java:834)
>       at org.junit.Assert.assertEquals(Assert.java:645)
>       at org.junit.Assert.assertEquals(Assert.java:631)
>       at 
> org.apache.flink.streaming.connectors.cassandra.CassandraConnectorITCase.testCassandraBatchFormats(CassandraConnectorITCase.java:442)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to