(beam) branch master updated: Don't run DicomIO Integration Tests on GCP DF Runner (#30103)

[yhu]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 772cf0a5c01 Don't run DicomIO Integration Tests on GCP DF Runner 
(#30103)
772cf0a5c01 is described below

commit 772cf0a5c01ca64de9392b17f1acaa7caf7a3498
Author: Svetak Sundhar <svetaksund...@google.com>
AuthorDate: Thu Jan 25 18:38:39 2024 +0000

    Don't run DicomIO Integration Tests on GCP DF Runner (#30103)
    
    * [BEAM-12550] Implement Parallelizable Skew and Kurtosis
    (Skew implementation)
    
    R: @TheNeuralBit
    
    * dont run on Dataflow Java V2
    
    * spotless
    
    * run PostCommit
    
    * delete postcommit trigger
    
    ---------
    
    Co-authored-by: svetakvsundhar <svetak.sund...@gmail.com>
---
 runners/google-cloud-dataflow-java/build.gradle                        | 2 ++
 .../test/java/org/apache/beam/sdk/io/gcp/healthcare/DicomIOReadIT.java | 3 ---
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/runners/google-cloud-dataflow-java/build.gradle 
b/runners/google-cloud-dataflow-java/build.gradle
index f34db247a0f..8a89c489ceb 100644
--- a/runners/google-cloud-dataflow-java/build.gradle
+++ b/runners/google-cloud-dataflow-java/build.gradle
@@ -566,6 +566,7 @@ task googleCloudPlatformLegacyWorkerIntegrationTest(type: 
Test, dependsOn: copyG
   exclude '**/BigQueryIOStorageReadTableRowIT.class'
   exclude '**/PubsubReadIT.class'
   exclude '**/FhirIOReadIT.class'
+  exclude '**/DicomIOReadIT.class'
   exclude '**/gcp/spanner/changestreams/it/*.class'
   maxParallelForks 4
   classpath = configurations.googleCloudPlatformIntegrationTest
@@ -610,6 +611,7 @@ task googleCloudPlatformRunnerV2IntegrationTest(type: Test) 
{
   exclude '**/SpannerWriteIT.class'
   exclude '**/*KmsKeyIT.class'
   exclude '**/FhirIOReadIT.class'
+  exclude '**/DicomIOReadIT.class'
   exclude '**/FhirIOWriteIT.class'
   exclude '**/FhirIOLROIT.class'
   exclude '**/FhirIOSearchIT.class'
diff --git 
a/sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/healthcare/DicomIOReadIT.java
 
b/sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/healthcare/DicomIOReadIT.java
index f6f8e8b8fcc..570844011f3 100644
--- 
a/sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/healthcare/DicomIOReadIT.java
+++ 
b/sdks/java/io/google-cloud-platform/src/test/java/org/apache/beam/sdk/io/gcp/healthcare/DicomIOReadIT.java
@@ -27,7 +27,6 @@ import org.apache.beam.sdk.transforms.Create;
 import org.junit.After;
 import org.junit.Assert;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 
@@ -62,7 +61,6 @@ public class DicomIOReadIT {
     client.deleteDicomStore(healthcareDataset + "/dicomStores/" + storeName);
   }
 
-  @Ignore("https://github.com/apache/beam/issues/28099";)
   @Test
   public void testDicomMetadataRead() {
     String webPath =
@@ -90,7 +88,6 @@ public class DicomIOReadIT {
     pipeline.run();
   }
 
-  @Ignore("https://github.com/apache/beam/issues/28099";)
   @Test
   public void testDicomFailedMetadataRead() {
     String badWebPath = "foo";