You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2019/05/10 12:56:22 UTC

[arrow] branch master updated: ARROW-5296: [Java] Ignore timeout-based Flight tests for now

This is an automated email from the ASF dual-hosted git repository.

wesm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new ccdaa9f  ARROW-5296: [Java] Ignore timeout-based Flight tests for now
ccdaa9f is described below

commit ccdaa9f2a4c1af1222df840b608e2ef465d331fc
Author: David Li <li...@gmail.com>
AuthorDate: Fri May 10 07:56:00 2019 -0500

    ARROW-5296: [Java] Ignore timeout-based Flight tests for now
    
    Author: David Li <li...@gmail.com>
    
    Closes #4286 from lihalite/arrow-5296 and squashes the following commits:
    
    05483ffae <David Li> Ignore timeout-based Flight tests for now
---
 .../src/test/java/org/apache/arrow/flight/TestCallOptions.java       | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/java/flight/src/test/java/org/apache/arrow/flight/TestCallOptions.java b/java/flight/src/test/java/org/apache/arrow/flight/TestCallOptions.java
index c95cb98..21d7eca 100644
--- a/java/flight/src/test/java/org/apache/arrow/flight/TestCallOptions.java
+++ b/java/flight/src/test/java/org/apache/arrow/flight/TestCallOptions.java
@@ -28,12 +28,15 @@ import org.apache.arrow.flight.auth.ServerAuthHandler;
 import org.apache.arrow.memory.BufferAllocator;
 import org.apache.arrow.memory.RootAllocator;
 import org.junit.Assert;
+import org.junit.Ignore;
 import org.junit.Test;
 
 public class TestCallOptions {
 
   @Test
+  @Ignore
   public void timeoutFires() {
+    // Ignored due to CI flakiness
     test((client) -> {
       Instant start = Instant.now();
       Iterator<Result> results = client.doAction(new Action("hang"), CallOptions.timeout(1, TimeUnit.SECONDS));
@@ -49,7 +52,9 @@ public class TestCallOptions {
   }
 
   @Test
+  @Ignore
   public void underTimeout() {
+    // Ignored due to CI flakiness
     test((client) -> {
       Instant start = Instant.now();
       // This shouldn't fail and it should complete within the timeout