From 7a33a6ea3c18ed940caf162e0ff76c0ae30421a2 Mon Sep 17 00:00:00 2001 From: Qing Wang Date: Fri, 6 Mar 2020 16:22:08 +0800 Subject: [PATCH] [Java] Enable skipped direct call cases (#7363) * Comment out * Refine * Revert --- .../java/org/ray/api/test/CrossLanguageInvocationTest.java | 2 -- java/test/src/main/java/org/ray/api/test/FailureTest.java | 3 --- 2 files changed, 5 deletions(-) diff --git a/java/test/src/main/java/org/ray/api/test/CrossLanguageInvocationTest.java b/java/test/src/main/java/org/ray/api/test/CrossLanguageInvocationTest.java index fbf3e7f52..392bdfdcc 100644 --- a/java/test/src/main/java/org/ray/api/test/CrossLanguageInvocationTest.java +++ b/java/test/src/main/java/org/ray/api/test/CrossLanguageInvocationTest.java @@ -61,8 +61,6 @@ public class CrossLanguageInvocationTest extends BaseMultiLanguageTest { @Test public void testCallingPythonActor() { - // Python worker doesn't support direct call yet. - TestUtils.skipTestIfDirectActorCallEnabled(); RayPyActor actor = Ray.createPyActor(PYTHON_MODULE, "Counter", "1".getBytes()); RayObject res = Ray.callPy(actor, "increase", "1".getBytes()); Assert.assertEquals(res.get(), "2".getBytes()); diff --git a/java/test/src/main/java/org/ray/api/test/FailureTest.java b/java/test/src/main/java/org/ray/api/test/FailureTest.java index e3df394fb..5ec9c2e7c 100644 --- a/java/test/src/main/java/org/ray/api/test/FailureTest.java +++ b/java/test/src/main/java/org/ray/api/test/FailureTest.java @@ -120,9 +120,6 @@ public class FailureTest extends BaseTest { @Test public void testActorProcessDying() { TestUtils.skipTestUnderSingleProcess(); - // This test case hangs if the worker to worker connection is implemented with grpc. - // TODO (kfstorm): Should be fixed. - TestUtils.skipTestIfDirectActorCallEnabled(); RayActor actor = Ray.createActor(BadActor::new, false); try { actor.call(BadActor::badMethod2).get();