Exclude StringSet tests from portable runners and Dataflow LegacyRunn…

…er (#31818) * Exclude StringSet tests from portable runners and Dataflow LegacyRunner * trigger files
apache · Jul 10, 2024 · b129433 · b129433
1 parent 8153867
commit b129433
Show file tree

Hide file tree

Showing 9 changed files with 11 additions and 5 deletions.
diff --git a/.github/trigger_files/beam_PostCommit_Java_PVR_Flink_Streaming.json b/.github/trigger_files/beam_PostCommit_Java_PVR_Flink_Streaming.json
@@ -1,4 +1,4 @@
 {
  "comment": "Modify this file in a trivial way to cause this test suite to run",
- "https://github.com/apache/beam/pull/31156": "noting that PR #31156 should run this test"
+ "modification": 0
 }
diff --git a/.github/trigger_files/beam_PostCommit_Java_PVR_Samza.json b/.github/trigger_files/beam_PostCommit_Java_PVR_Samza.json
@@ -1,5 +1,4 @@
 {
  "comment": "Modify this file in a trivial way to cause this test suite to run",
- "https://github.com/apache/beam/pull/31156": "noting that PR #31156 should run this test",
- "https://github.com/apache/beam/pull/31270": "re-adds specialized Samza translation of Redistribute"
+ "modification": 0
 }
diff --git a/.github/trigger_files/beam_PostCommit_Java_PVR_Spark3_Streaming.json b/.github/trigger_files/beam_PostCommit_Java_PVR_Spark3_Streaming.json
@@ -1,4 +1,4 @@
 {
  "comment": "Modify this file in a trivial way to cause this test suite to run",
- "https://github.com/apache/beam/pull/31156": "noting that PR #31156 should run this test"
+ "modification": 0
 }
diff --git a/.github/trigger_files/beam_PostCommit_Java_PVR_Spark_Batch.json b/.github/trigger_files/beam_PostCommit_Java_PVR_Spark_Batch.json
@@ -1,4 +1,4 @@
 {
  "comment": "Modify this file in a trivial way to cause this test suite to run",
- "modification": 1
+ "modification": 0
 }
diff --git a/runners/flink/job-server/flink_job_server.gradle b/runners/flink/job-server/flink_job_server.gradle
@@ -171,6 +171,7 @@ def portableValidatesRunnerTask(String name, boolean streaming, boolean checkpoi
  excludeCategories 'org.apache.beam.sdk.testing.UsesCustomWindowMerging'
  excludeCategories 'org.apache.beam.sdk.testing.UsesFailureMessage'
  excludeCategories 'org.apache.beam.sdk.testing.UsesGaugeMetrics'
+ excludeCategories 'org.apache.beam.sdk.testing.UsesStringSetMetrics'
  excludeCategories 'org.apache.beam.sdk.testing.UsesParDoLifecycle'
  excludeCategories 'org.apache.beam.sdk.testing.UsesMapState'
  excludeCategories 'org.apache.beam.sdk.testing.UsesMultimapState'

diff --git a/runners/google-cloud-dataflow-java/build.gradle b/runners/google-cloud-dataflow-java/build.gradle
@@ -183,6 +183,7 @@ def commonLegacyExcludeCategories = [
  'org.apache.beam.sdk.testing.UsesExternalService',
  'org.apache.beam.sdk.testing.UsesDistributionMetrics',
  'org.apache.beam.sdk.testing.UsesGaugeMetrics',
+ 'org.apache.beam.sdk.testing.UsesStringSetMetrics',
  'org.apache.beam.sdk.testing.UsesMultimapState',
  'org.apache.beam.sdk.testing.UsesTestStream',
  'org.apache.beam.sdk.testing.UsesParDoLifecycle',
@@ -193,6 +194,7 @@ def commonLegacyExcludeCategories = [
 def commonRunnerV2ExcludeCategories = [
  'org.apache.beam.sdk.testing.UsesExternalService',
  'org.apache.beam.sdk.testing.UsesGaugeMetrics',
+ 'org.apache.beam.sdk.testing.UsesStringSetMetrics',
  'org.apache.beam.sdk.testing.UsesSetState',
  'org.apache.beam.sdk.testing.UsesMapState',
  'org.apache.beam.sdk.testing.UsesMultimapState',

diff --git a/runners/portability/java/build.gradle b/runners/portability/java/build.gradle
@@ -159,6 +159,7 @@ def createUlrValidatesRunnerTask = { name, environmentType, dockerImageTask = ""
  excludeCategories 'org.apache.beam.sdk.testing.UsesExternalService'
  excludeCategories 'org.apache.beam.sdk.testing.UsesSdkHarnessEnvironment'
  excludeCategories 'org.apache.beam.sdk.testing.UsesGaugeMetrics'
+ excludeCategories 'org.apache.beam.sdk.testing.UsesStringSetMetrics'
  excludeCategories 'org.apache.beam.sdk.testing.UsesOnWindowExpiration'
  excludeCategories 'org.apache.beam.sdk.testing.UsesMapState'
  excludeCategories 'org.apache.beam.sdk.testing.UsesMultimapState'

diff --git a/runners/samza/job-server/build.gradle b/runners/samza/job-server/build.gradle
@@ -90,6 +90,7 @@ def portableValidatesRunnerTask(String name, boolean docker) {
  excludeCategories 'org.apache.beam.sdk.testing.UsesCustomWindowMerging'
  excludeCategories 'org.apache.beam.sdk.testing.UsesFailureMessage'
  excludeCategories 'org.apache.beam.sdk.testing.UsesGaugeMetrics'
+ excludeCategories 'org.apache.beam.sdk.testing.UsesStringSetMetrics'
  excludeCategories 'org.apache.beam.sdk.testing.UsesParDoLifecycle'
  excludeCategories 'org.apache.beam.sdk.testing.UsesMapState'
  excludeCategories 'org.apache.beam.sdk.testing.UsesMultimapState'

diff --git a/runners/spark/job-server/spark_job_server.gradle b/runners/spark/job-server/spark_job_server.gradle
@@ -117,6 +117,7 @@ def portableValidatesRunnerTask(String name, boolean streaming, boolean docker,
  excludeCategories 'org.apache.beam.sdk.testing.UsesCustomWindowMerging'
  excludeCategories 'org.apache.beam.sdk.testing.UsesFailureMessage'
  excludeCategories 'org.apache.beam.sdk.testing.UsesGaugeMetrics'
+ excludeCategories 'org.apache.beam.sdk.testing.UsesStringSetMetrics'
  excludeCategories 'org.apache.beam.sdk.testing.UsesPerKeyOrderedDelivery'
  excludeCategories 'org.apache.beam.sdk.testing.UsesParDoLifecycle'
  excludeCategories 'org.apache.beam.sdk.testing.UsesMapState'
@@ -185,6 +186,7 @@ def portableValidatesRunnerTask(String name, boolean streaming, boolean docker,
  excludeCategories 'org.apache.beam.sdk.testing.UsesCustomWindowMerging'
  excludeCategories 'org.apache.beam.sdk.testing.UsesFailureMessage'
  excludeCategories 'org.apache.beam.sdk.testing.UsesGaugeMetrics'
+ excludeCategories 'org.apache.beam.sdk.testing.UsesStringSetMetrics'
  excludeCategories 'org.apache.beam.sdk.testing.UsesPerKeyOrderedDelivery'
  excludeCategories 'org.apache.beam.sdk.testing.UsesPerKeyOrderInBundle'
  excludeCategories 'org.apache.beam.sdk.testing.UsesParDoLifecycle'