You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by "akashorabek (via GitHub)" <gi...@apache.org> on 2024/04/08 10:33:27 UTC
[PR] Add PubSubIO Stress test [beam]
akashorabek opened a new pull request, #30886:
URL: https://github.com/apache/beam/pull/30886
This pull request introduces stress tests for PubSubIO, designed to assess the performance under various conditions. The stress tests simulate dynamic load increases and evaluate the behavior of PubSubIO.
Changes:
Added stress tests for PubSubIO.
Implemented dynamic load increases over time to simulate varying workloads.
Added support for exporting metrics to InfluxDB or BigQuery based on the configuration parameter.
[Write job example](https://console.cloud.google.com/dataflow/jobs/us-central1/2024-04-08_02_09_14-10513489467703410835;step=Write%20to%20PubSub;mainTab=JOB_GRAPH;bottomTab=WORKER_LOGS;bottomStepTab=DATA_SAMPLING;logsSeverity=ERROR;graphView=0?project=apache-beam-testing)
[Read job example](https://console.cloud.google.com/dataflow/jobs/us-central1/2024-04-08_02_09_57-14737073549262708363;step=Counting%20element;mainTab=JOB_GRAPH;bottomTab=JOB_LOGS;bottomStepTab=DATA_SAMPLING;logsSeverity=INFO;graphView=0?project=apache-beam-testing&pageState=(%22dfTime%22:(%22l%22:%22dfJobMaxTime%22)))
------------------------
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
- [ ] Mention the appropriate issue in your description (for example: `addresses #123`), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment `fixes #<ISSUE NUMBER>` instead.
- [ ] Update `CHANGES.md` with noteworthy changes.
- [ ] If this contribution is large, please file an Apache [Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
See the [Contributor Guide](https://beam.apache.org/contribute) for more tips on [how to make review process smoother](https://github.com/apache/beam/blob/master/CONTRIBUTING.md#make-the-reviewers-job-easier).
To check the build health, please visit [https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md](https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md)
GitHub Actions Tests Status (on master branch)
------------------------------------------------------------------------------------------------
[](https://github.com/apache/beam/actions?query=workflow%3A%22Build+python+source+distribution+and+wheels%22+branch%3Amaster+event%3Aschedule)
[](https://github.com/apache/beam/actions?query=workflow%3A%22Python+Tests%22+branch%3Amaster+event%3Aschedule)
[](https://github.com/apache/beam/actions?query=workflow%3A%22Java+Tests%22+branch%3Amaster+event%3Aschedule)
[](https://github.com/apache/beam/actions?query=workflow%3A%22Go+tests%22+branch%3Amaster+event%3Aschedule)
See [CI.md](https://github.com/apache/beam/blob/master/CI.md) for more information about GitHub Actions CI or the [workflows README](https://github.com/apache/beam/blob/master/.github/workflows/README.md) to see a list of phrases to trigger workflows.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add PubSubIO Stress test [beam]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #30886:
URL: https://github.com/apache/beam/pull/30886#issuecomment-2043317594
Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add PubSubIO Stress test [beam]
Posted by "Abacn (via GitHub)" <gi...@apache.org>.
Abacn merged PR #30886:
URL: https://github.com/apache/beam/pull/30886
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add PubSubIO Stress test [beam]
Posted by "Abacn (via GitHub)" <gi...@apache.org>.
Abacn commented on code in PR #30886:
URL: https://github.com/apache/beam/pull/30886#discussion_r1561016059
##########
it/google-cloud-platform/src/test/java/org/apache/beam/it/gcp/pubsub/PubSubIOST.java:
##########
@@ -0,0 +1,510 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.beam.it.gcp.pubsub;
+
+import static org.junit.Assert.assertNotEquals;
+import static org.junit.Assert.assertTrue;
+
+import com.fasterxml.jackson.annotation.JsonProperty;
+import com.google.protobuf.ByteString;
+import com.google.pubsub.v1.SubscriptionName;
+import com.google.pubsub.v1.TopicName;
+import java.io.IOException;
+import java.io.Serializable;
+import java.nio.ByteBuffer;
+import java.nio.charset.StandardCharsets;
+import java.text.ParseException;
+import java.time.Duration;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
+import java.util.Objects;
+import org.apache.beam.it.common.PipelineLauncher;
+import org.apache.beam.it.common.PipelineOperator;
+import org.apache.beam.it.common.TestProperties;
+import org.apache.beam.it.common.utils.ResourceManagerUtils;
+import org.apache.beam.it.gcp.IOStressTestBase;
+import org.apache.beam.runners.dataflow.options.DataflowPipelineWorkerPoolOptions;
+import org.apache.beam.sdk.extensions.protobuf.Proto3SchemaMessages.Primitive;
+import org.apache.beam.sdk.io.Read;
+import org.apache.beam.sdk.io.gcp.pubsub.PubsubIO;
+import org.apache.beam.sdk.io.gcp.pubsub.PubsubMessage;
+import org.apache.beam.sdk.io.gcp.pubsub.PubsubOptions;
+import org.apache.beam.sdk.io.synthetic.SyntheticSourceOptions;
+import org.apache.beam.sdk.io.synthetic.SyntheticUnboundedSource;
+import org.apache.beam.sdk.options.PipelineOptionsFactory;
+import org.apache.beam.sdk.testing.TestPipeline;
+import org.apache.beam.sdk.testing.TestPipelineOptions;
+import org.apache.beam.sdk.testutils.publishing.InfluxDBSettings;
+import org.apache.beam.sdk.transforms.DoFn;
+import org.apache.beam.sdk.transforms.ParDo;
+import org.apache.beam.sdk.transforms.Reshuffle;
+import org.apache.beam.sdk.values.KV;
+import org.apache.beam.sdk.values.PCollection;
+import org.apache.beam.vendor.guava.v32_1_2_jre.com.google.common.base.MoreObjects;
+import org.apache.beam.vendor.guava.v32_1_2_jre.com.google.common.base.Strings;
+import org.apache.beam.vendor.guava.v32_1_2_jre.com.google.common.collect.ImmutableMap;
+import org.checkerframework.checker.nullness.qual.Nullable;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+
+/**
+ * PubSubIO stress test. The test is designed to assess the performance of PubSubIO under various
+ * conditions.
+ *
+ * <p>Usage: <br>
+ * - To run medium-scale stress tests: {@code gradle
+ * :it:google-cloud-platform:PubSubStressTestMedium} - To run large-scale stress tests: {@code
+ * gradle :it:google-cloud-platform:PubSubStressTestLarge}
+ */
+public class PubSubIOST extends IOStressTestBase {
+ private static final int NUMBER_OF_BUNDLES_FOR_MEDIUM = 20;
+ private static final int NUMBER_OF_BUNDLES_FOR_LARGE = 200;
+ private static final String READ_ELEMENT_METRIC_NAME = "read_count";
+ private static final String WRITE_ELEMENT_METRIC_NAME = "write_count";
+ private static final String MAP_RECORDS_STEP_NAME = "Map records";
+ private static final String WRITE_TO_PUBSUB_STEP_NAME = "Write to PubSub";
+ private static final Map<String, Configuration> TEST_CONFIGS_PRESET;
+ private static TopicName topicName;
+ private static String testConfigName;
+ private static Configuration configuration;
+ private static SubscriptionName subscription;
+ private static InfluxDBSettings influxDBSettings;
+ private static PubsubResourceManager resourceManager;
+
+ @Rule public transient TestPipeline writePipeline = TestPipeline.create();
+ @Rule public transient TestPipeline readPipeline = TestPipeline.create();
+
+ static {
+ try {
+ TEST_CONFIGS_PRESET =
+ ImmutableMap.of(
+ "medium",
+ Configuration.fromJsonString(
+ "{\"numRecords\":2000000,\"rowsPerSecond\":25000,\"minutes\":10,\"valueSizeBytes\":1000,\"pipelineTimeout\":20,\"runner\":\"DataflowRunner\"}",
+ Configuration.class),
+ "large",
+ Configuration.fromJsonString(
+ "{\"numRecords\":20000000,\"rowsPerSecond\":25000,\"minutes\":40,\"valueSizeBytes\":1000,\"pipelineTimeout\":70,\"runner\":\"DataflowRunner\"}",
+ Configuration.class));
+ } catch (IOException e) {
+ throw new RuntimeException(e);
+ }
+ }
+
+ @Before
+ public void setup() throws IOException {
+ resourceManager =
+ PubsubResourceManager.builder("io-pubsub-st", project, CREDENTIALS_PROVIDER).build();
+ topicName = resourceManager.createTopic("topic");
+ subscription = resourceManager.createSubscription(topicName, "subscription");
+ PipelineOptionsFactory.register(TestPipelineOptions.class);
+
+ // parse configuration
+ testConfigName =
+ TestProperties.getProperty("configuration", "local", TestProperties.Type.PROPERTY);
+ configuration = TEST_CONFIGS_PRESET.get(testConfigName);
+ if (configuration == null) {
+ try {
+ configuration = Configuration.fromJsonString(testConfigName, Configuration.class);
+ } catch (IOException e) {
+ throw new IllegalArgumentException(
+ String.format(
+ "Unknown test configuration: [%s]. Pass to a valid configuration json, or use"
+ + " config presets: %s",
+ testConfigName, TEST_CONFIGS_PRESET.keySet()));
+ }
+ }
+
+ // Explicitly set up number of bundles in SyntheticUnboundedSource since it has a bug in
+ // implementation where
+ // number of lost data in streaming pipeline equals to number of initial bundles.
+ configuration.forceNumInitialBundles =
+ testConfigName.equals("medium")
+ ? NUMBER_OF_BUNDLES_FOR_MEDIUM
+ : NUMBER_OF_BUNDLES_FOR_LARGE;
+
+ // tempLocation needs to be set for DataflowRunner
+ if (!Strings.isNullOrEmpty(tempBucketName)) {
+ String tempLocation = String.format("gs://%s/temp/", tempBucketName);
+ writePipeline.getOptions().as(TestPipelineOptions.class).setTempRoot(tempLocation);
+ writePipeline.getOptions().setTempLocation(tempLocation);
+ readPipeline.getOptions().as(TestPipelineOptions.class).setTempRoot(tempLocation);
+ readPipeline.getOptions().setTempLocation(tempLocation);
+ }
+ writePipeline.getOptions().as(PubsubOptions.class).setProject(project);
+ readPipeline.getOptions().as(PubsubOptions.class).setProject(project);
+
+ if (configuration.exportMetricsToInfluxDB) {
+ configuration.influxHost =
+ TestProperties.getProperty("influxHost", "", TestProperties.Type.PROPERTY);
+ configuration.influxDatabase =
+ TestProperties.getProperty("influxDatabase", "", TestProperties.Type.PROPERTY);
+ configuration.influxMeasurement =
+ TestProperties.getProperty("influxMeasurement", "", TestProperties.Type.PROPERTY);
+ }
+ }
+
+ @After
+ public void tearDownClass() {
+ ResourceManagerUtils.cleanResources(resourceManager);
+ }
+
+ @Test
+ public void testStringWriteAndRead() throws IOException {
+ configuration.writeAndReadFormat = WriteAndReadFormat.STRING.toString();
+ testWriteAndRead();
+ }
+
+ @Test
+ public void testAvroGenericClassWriteAndRead() throws IOException {
+ configuration.writeAndReadFormat = WriteAndReadFormat.AVRO.toString();
+ testWriteAndRead();
+ }
+
+ @Test
+ public void testProtoPrimitiveWriteAndRead() throws IOException {
+ configuration.writeAndReadFormat = WriteAndReadFormat.PROTO.toString();
+ testWriteAndRead();
+ }
+
+ @Test
+ public void testPubsubMessageWriteAndRead() throws IOException {
+ configuration.writeAndReadFormat = WriteAndReadFormat.PUBSUB_MESSAGE.toString();
+ testWriteAndRead();
+ }
+
+ public void testWriteAndRead() throws IOException {
+ if (configuration.exportMetricsToInfluxDB) {
+ influxDBSettings =
+ InfluxDBSettings.builder()
+ .withHost(configuration.influxHost)
+ .withDatabase(configuration.influxDatabase)
+ .withMeasurement(
+ configuration.influxMeasurement
+ + "_"
+ + testConfigName
+ + "_"
+ + configuration.writeAndReadFormat)
+ .get();
+ }
+
+ WriteAndReadFormat format = WriteAndReadFormat.valueOf(configuration.writeAndReadFormat);
+ PipelineLauncher.LaunchInfo writeLaunchInfo = generateDataAndWrite(format);
+ PipelineLauncher.LaunchInfo readLaunchInfo = testRead(format);
+ try {
+ PipelineOperator.Result readResult =
+ pipelineOperator.waitUntilDone(
+ createConfig(readLaunchInfo, Duration.ofMinutes(configuration.pipelineTimeout)));
+
+ // Check the initial launch didn't fail
+ assertNotEquals(PipelineOperator.Result.LAUNCH_FAILED, readResult);
+
+ // check metrics
+ double writeNumRecords =
+ pipelineLauncher.getMetric(
+ project,
+ region,
+ writeLaunchInfo.jobId(),
+ getBeamMetricsName(PipelineMetricsType.COUNTER, WRITE_ELEMENT_METRIC_NAME));
+ double readNumRecords =
+ pipelineLauncher.getMetric(
+ project,
+ region,
+ readLaunchInfo.jobId(),
+ getBeamMetricsName(PipelineMetricsType.COUNTER, READ_ELEMENT_METRIC_NAME));
+
+ // Assert that writeNumRecords equals or greater than readNumRecords since there might be
+ // duplicates when testing big amount of data
+ assertTrue(writeNumRecords >= readNumRecords);
+
+ // export metrics
+ MetricsConfiguration writeMetricsConfig =
+ MetricsConfiguration.builder()
+ .setInputPCollection("Map records.out0")
+ .setInputPCollectionV2("Map records/ParMultiDo(MapKVToPubSubType).out0")
+ .setOutputPCollection("Counting element.out0")
+ .setOutputPCollectionV2("Counting element/ParMultiDo(Counting).out0")
+ .build();
+
+ MetricsConfiguration readMetricsConfig =
+ MetricsConfiguration.builder()
+ .setOutputPCollection("Counting element.out0")
+ .setOutputPCollectionV2("Counting element/ParMultiDo(Counting).out0")
+ .build();
+
+ exportMetrics(
+ writeLaunchInfo,
+ writeMetricsConfig,
+ configuration.exportMetricsToInfluxDB,
+ influxDBSettings);
+ exportMetrics(
+ readLaunchInfo,
+ readMetricsConfig,
+ configuration.exportMetricsToInfluxDB,
+ influxDBSettings);
+ } catch (ParseException | InterruptedException e) {
+ throw new RuntimeException(e);
+ } finally {
+ cancelJobIfRunning(writeLaunchInfo);
+ cancelJobIfRunning(readLaunchInfo);
+ }
+ }
+
+ /**
+ * The method creates a pipeline to simulate data generation and write operations to PubSub, based
+ * on the specified configuration parameters. The stress test involves varying the load
+ * dynamically over time, with options to use configurable parameters.
+ */
+ private PipelineLauncher.LaunchInfo generateDataAndWrite(WriteAndReadFormat format)
+ throws IOException {
+ int startMultiplier =
+ Math.max(configuration.rowsPerSecond, DEFAULT_ROWS_PER_SECOND) / DEFAULT_ROWS_PER_SECOND;
+ List<LoadPeriod> loadPeriods =
+ getLoadPeriods(configuration.minutes, DEFAULT_LOAD_INCREASE_ARRAY);
+
+ PCollection<KV<byte[], byte[]>> dataFromSource =
+ writePipeline.apply(
+ "Read from source", Read.from(new SyntheticUnboundedSource(configuration)));
+
+ if (startMultiplier > 1) {
+ dataFromSource =
+ dataFromSource
+ .apply(
+ "One input to multiple outputs",
+ ParDo.of(new MultiplierDoFn<>(startMultiplier, loadPeriods)))
+ .apply("Reshuffle fanout", Reshuffle.of())
+ .apply("Counting element", ParDo.of(new CountingFn<>(WRITE_ELEMENT_METRIC_NAME)));
+ ;
+ }
+
+ switch (format) {
+ case STRING:
+ dataFromSource
+ .apply(MAP_RECORDS_STEP_NAME, ParDo.of(new MapKVtoString()))
+ .apply(WRITE_TO_PUBSUB_STEP_NAME, PubsubIO.writeStrings().to(topicName.toString()));
+ break;
+ case AVRO:
+ dataFromSource
+ .apply(MAP_RECORDS_STEP_NAME, ParDo.of(new MapKVtoGenericClass()))
+ .apply(
+ WRITE_TO_PUBSUB_STEP_NAME,
+ PubsubIO.writeAvros(GenericClass.class).to(topicName.toString()));
+ break;
+ case PROTO:
+ dataFromSource
+ .apply(MAP_RECORDS_STEP_NAME, ParDo.of(new MapKVtoPrimitiveProto()))
+ .apply(
+ WRITE_TO_PUBSUB_STEP_NAME,
+ PubsubIO.writeProtos(Primitive.class).to(topicName.toString()));
+ break;
+ case PUBSUB_MESSAGE:
+ dataFromSource
+ .apply(MAP_RECORDS_STEP_NAME, ParDo.of(new MapKVtoPubSubMessage()))
+ .apply(WRITE_TO_PUBSUB_STEP_NAME, PubsubIO.writeMessages().to(topicName.toString()));
+ break;
+ }
+
+ PipelineLauncher.LaunchConfig options =
+ PipelineLauncher.LaunchConfig.builder("write-pubsub")
+ .setSdk(PipelineLauncher.Sdk.JAVA)
+ .setPipeline(writePipeline)
+ .addParameter("runner", configuration.runner)
+ .addParameter(
+ "autoscalingAlgorithm",
+ DataflowPipelineWorkerPoolOptions.AutoscalingAlgorithmType.THROUGHPUT_BASED
+ .toString())
+ .addParameter("numWorkers", String.valueOf(configuration.numWorkers))
+ .addParameter("maxNumWorkers", String.valueOf(configuration.maxNumWorkers))
+ .addParameter("streaming", "true")
+ .addParameter("experiments", "use_runner_v2")
+ .build();
+
+ return pipelineLauncher.launch(project, region, options);
+ }
+
+ private PipelineLauncher.LaunchInfo testRead(WriteAndReadFormat format) throws IOException {
+ PubsubIO.Read<?> read = null;
+
+ switch (format) {
+ case STRING:
+ read = PubsubIO.readStrings().fromSubscription(subscription.toString());
+ break;
+ case AVRO:
+ read = PubsubIO.readAvros(GenericClass.class).fromSubscription(subscription.toString());
+ break;
+ case PROTO:
+ read = PubsubIO.readProtos(Primitive.class).fromSubscription(subscription.toString());
+ break;
+ case PUBSUB_MESSAGE:
+ read = PubsubIO.readMessages().fromSubscription(subscription.toString());
+ break;
+ }
+
+ readPipeline
+ .apply("Read from PubSub", read)
+ .apply("Counting element", ParDo.of(new CountingFn<>(READ_ELEMENT_METRIC_NAME)));
+
+ PipelineLauncher.LaunchConfig readOptions =
+ PipelineLauncher.LaunchConfig.builder("read-pubsub")
+ .setSdk(PipelineLauncher.Sdk.JAVA)
+ .setPipeline(readPipeline)
+ .addParameter("runner", configuration.runner)
+ .addParameter("streaming", "true")
+ .addParameter("experiments", "use_runner_v2")
+ .addParameter("numWorkers", String.valueOf(configuration.numWorkers))
+ .addParameter("maxNumWorkers", String.valueOf(configuration.maxNumWorkers))
+ .build();
+
+ return pipelineLauncher.launch(project, region, readOptions);
+ }
+
+ private void cancelJobIfRunning(PipelineLauncher.LaunchInfo pipelineLaunchInfo)
+ throws IOException {
+ if (pipelineLauncher.getJobStatus(project, region, pipelineLaunchInfo.jobId())
+ == PipelineLauncher.JobState.RUNNING) {
+ pipelineLauncher.cancelJob(project, region, pipelineLaunchInfo.jobId());
+ }
+ }
+
+ /** Mapper class to convert data from KV<byte[], byte[]> to String. */
+ private static class MapKVtoString extends DoFn<KV<byte[], byte[]>, String> {
+ @ProcessElement
+ public void process(ProcessContext context) {
+ byte[] byteValue = Objects.requireNonNull(context.element()).getValue();
+ context.output(ByteString.copyFrom(byteValue).toString(StandardCharsets.UTF_8));
+ }
+ }
+
+ /** Mapper class to convert data from KV<byte[], byte[]> to GenericClass. */
+ private static class MapKVtoGenericClass extends DoFn<KV<byte[], byte[]>, GenericClass> {
+ @ProcessElement
+ public void process(ProcessContext context) {
+ byte[] byteValue = Objects.requireNonNull(context.element()).getValue();
+ GenericClass pojo = new GenericClass(byteValue);
+ context.output(pojo);
+ }
+ }
+
+ /** Mapper class to convert data from KV<byte[], byte[]> to Proto Primitive. */
+ private static class MapKVtoPrimitiveProto extends DoFn<KV<byte[], byte[]>, Primitive> {
+ @ProcessElement
+ public void process(ProcessContext context) {
+ byte[] byteValue = Objects.requireNonNull(context.element()).getValue();
+ Primitive proto =
+ Primitive.newBuilder()
+ .setPrimitiveBytes(ByteString.copyFrom(byteValue))
+ .setPrimitiveInt32(ByteBuffer.wrap(byteValue).getInt())
+ .build();
+ context.output(proto);
+ }
+ }
+
+ /** Mapper class to convert data from KV<byte[], byte[]> to PubSubMessage. */
+ private static class MapKVtoPubSubMessage extends DoFn<KV<byte[], byte[]>, PubsubMessage> {
+ @ProcessElement
+ public void process(ProcessContext context) {
+ byte[] byteValue = Objects.requireNonNull(context.element()).getValue();
+ PubsubMessage pubsubMessage = new PubsubMessage(byteValue, Collections.emptyMap());
+ context.output(pubsubMessage);
+ }
+ }
+
+ /** Example of Generic class to test PubSubIO.writeAvros()/readAvros methods. */
+ static class GenericClass implements Serializable {
+ byte[] byteField;
+
+ public GenericClass() {}
+
+ public GenericClass(byte[] byteField) {
+ this.byteField = byteField;
+ }
+
+ @Override
+ public String toString() {
+ return MoreObjects.toStringHelper(getClass()).add("byteField", byteField).toString();
+ }
+
+ @Override
+ public int hashCode() {
+ return Objects.hash(Arrays.hashCode(byteField));
+ }
+
+ @Override
+ public boolean equals(@Nullable Object other) {
+ if (other == null || !(other instanceof GenericClass)) {
+ return false;
+ }
+ GenericClass o = (GenericClass) other;
+ return Arrays.equals(byteField, o.byteField);
+ }
+ }
+
+ private enum WriteAndReadFormat {
+ STRING,
+ AVRO,
+ PROTO,
+ PUBSUB_MESSAGE
+ }
+
+ /** Options for PubSub IO load test. */
+ static class Configuration extends SyntheticSourceOptions {
+ /** Pipeline timeout in minutes. Must be a positive value. */
+ @JsonProperty public int pipelineTimeout = 20;
+
+ /** Runner specified to run the pipeline. */
+ @JsonProperty public String runner = "DirectRunner";
+
+ /** PubSub write and read format: STRING/AVRO/PROTO/PUBSUB_MESSAGE. */
+ @JsonProperty public String writeAndReadFormat = "STRING";
+
+ /** Number of workers for the pipeline. */
+ @JsonProperty public int numWorkers = 20;
+
+ /** Maximum number of workers for the pipeline. */
+ @JsonProperty public int maxNumWorkers = 100;
+
+ /**
+ * Rate of generated elements sent to the source table. Will run with a minimum of 1k rows per
+ * second.
+ */
+ @JsonProperty public int rowsPerSecond = DEFAULT_ROWS_PER_SECOND;
+
+ /** Rows will be generated for this many minutes. */
+ @JsonProperty public int minutes = 15;
+
+ /**
+ * Determines the destination for exporting metrics. If set to true, metrics will be exported to
+ * InfluxDB and displayed using Grafana. If set to false, metrics will be exported to BigQuery
+ * and displayed with Looker Studio.
+ */
+ @JsonProperty public boolean exportMetricsToInfluxDB = true;
Review Comment:
If this is true but influxHost is not provided, could the test still run successfully (just skip export) ? From current logic it seems it will always call exportMetrics
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add PubSubIO Stress test [beam]
Posted by "akashorabek (via GitHub)" <gi...@apache.org>.
akashorabek commented on PR #30886:
URL: https://github.com/apache/beam/pull/30886#issuecomment-2048543216
R: @Abacn
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add PubSubIO Stress test [beam]
Posted by "akashorabek (via GitHub)" <gi...@apache.org>.
akashorabek commented on PR #30886:
URL: https://github.com/apache/beam/pull/30886#issuecomment-2043315705
R: @Abacn
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add PubSubIO Stress test [beam]
Posted by "akashorabek (via GitHub)" <gi...@apache.org>.
akashorabek commented on code in PR #30886:
URL: https://github.com/apache/beam/pull/30886#discussion_r1561276933
##########
it/google-cloud-platform/src/test/java/org/apache/beam/it/gcp/pubsub/PubSubIOST.java:
##########
@@ -0,0 +1,510 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.beam.it.gcp.pubsub;
+
+import static org.junit.Assert.assertNotEquals;
+import static org.junit.Assert.assertTrue;
+
+import com.fasterxml.jackson.annotation.JsonProperty;
+import com.google.protobuf.ByteString;
+import com.google.pubsub.v1.SubscriptionName;
+import com.google.pubsub.v1.TopicName;
+import java.io.IOException;
+import java.io.Serializable;
+import java.nio.ByteBuffer;
+import java.nio.charset.StandardCharsets;
+import java.text.ParseException;
+import java.time.Duration;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
+import java.util.Objects;
+import org.apache.beam.it.common.PipelineLauncher;
+import org.apache.beam.it.common.PipelineOperator;
+import org.apache.beam.it.common.TestProperties;
+import org.apache.beam.it.common.utils.ResourceManagerUtils;
+import org.apache.beam.it.gcp.IOStressTestBase;
+import org.apache.beam.runners.dataflow.options.DataflowPipelineWorkerPoolOptions;
+import org.apache.beam.sdk.extensions.protobuf.Proto3SchemaMessages.Primitive;
+import org.apache.beam.sdk.io.Read;
+import org.apache.beam.sdk.io.gcp.pubsub.PubsubIO;
+import org.apache.beam.sdk.io.gcp.pubsub.PubsubMessage;
+import org.apache.beam.sdk.io.gcp.pubsub.PubsubOptions;
+import org.apache.beam.sdk.io.synthetic.SyntheticSourceOptions;
+import org.apache.beam.sdk.io.synthetic.SyntheticUnboundedSource;
+import org.apache.beam.sdk.options.PipelineOptionsFactory;
+import org.apache.beam.sdk.testing.TestPipeline;
+import org.apache.beam.sdk.testing.TestPipelineOptions;
+import org.apache.beam.sdk.testutils.publishing.InfluxDBSettings;
+import org.apache.beam.sdk.transforms.DoFn;
+import org.apache.beam.sdk.transforms.ParDo;
+import org.apache.beam.sdk.transforms.Reshuffle;
+import org.apache.beam.sdk.values.KV;
+import org.apache.beam.sdk.values.PCollection;
+import org.apache.beam.vendor.guava.v32_1_2_jre.com.google.common.base.MoreObjects;
+import org.apache.beam.vendor.guava.v32_1_2_jre.com.google.common.base.Strings;
+import org.apache.beam.vendor.guava.v32_1_2_jre.com.google.common.collect.ImmutableMap;
+import org.checkerframework.checker.nullness.qual.Nullable;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+
+/**
+ * PubSubIO stress test. The test is designed to assess the performance of PubSubIO under various
+ * conditions.
+ *
+ * <p>Usage: <br>
+ * - To run medium-scale stress tests: {@code gradle
+ * :it:google-cloud-platform:PubSubStressTestMedium} - To run large-scale stress tests: {@code
+ * gradle :it:google-cloud-platform:PubSubStressTestLarge}
+ */
+public class PubSubIOST extends IOStressTestBase {
+ private static final int NUMBER_OF_BUNDLES_FOR_MEDIUM = 20;
+ private static final int NUMBER_OF_BUNDLES_FOR_LARGE = 200;
+ private static final String READ_ELEMENT_METRIC_NAME = "read_count";
+ private static final String WRITE_ELEMENT_METRIC_NAME = "write_count";
+ private static final String MAP_RECORDS_STEP_NAME = "Map records";
+ private static final String WRITE_TO_PUBSUB_STEP_NAME = "Write to PubSub";
+ private static final Map<String, Configuration> TEST_CONFIGS_PRESET;
+ private static TopicName topicName;
+ private static String testConfigName;
+ private static Configuration configuration;
+ private static SubscriptionName subscription;
+ private static InfluxDBSettings influxDBSettings;
+ private static PubsubResourceManager resourceManager;
+
+ @Rule public transient TestPipeline writePipeline = TestPipeline.create();
+ @Rule public transient TestPipeline readPipeline = TestPipeline.create();
+
+ static {
+ try {
+ TEST_CONFIGS_PRESET =
+ ImmutableMap.of(
+ "medium",
+ Configuration.fromJsonString(
+ "{\"numRecords\":2000000,\"rowsPerSecond\":25000,\"minutes\":10,\"valueSizeBytes\":1000,\"pipelineTimeout\":20,\"runner\":\"DataflowRunner\"}",
+ Configuration.class),
+ "large",
+ Configuration.fromJsonString(
+ "{\"numRecords\":20000000,\"rowsPerSecond\":25000,\"minutes\":40,\"valueSizeBytes\":1000,\"pipelineTimeout\":70,\"runner\":\"DataflowRunner\"}",
+ Configuration.class));
+ } catch (IOException e) {
+ throw new RuntimeException(e);
+ }
+ }
+
+ @Before
+ public void setup() throws IOException {
+ resourceManager =
+ PubsubResourceManager.builder("io-pubsub-st", project, CREDENTIALS_PROVIDER).build();
+ topicName = resourceManager.createTopic("topic");
+ subscription = resourceManager.createSubscription(topicName, "subscription");
+ PipelineOptionsFactory.register(TestPipelineOptions.class);
+
+ // parse configuration
+ testConfigName =
+ TestProperties.getProperty("configuration", "local", TestProperties.Type.PROPERTY);
+ configuration = TEST_CONFIGS_PRESET.get(testConfigName);
+ if (configuration == null) {
+ try {
+ configuration = Configuration.fromJsonString(testConfigName, Configuration.class);
+ } catch (IOException e) {
+ throw new IllegalArgumentException(
+ String.format(
+ "Unknown test configuration: [%s]. Pass to a valid configuration json, or use"
+ + " config presets: %s",
+ testConfigName, TEST_CONFIGS_PRESET.keySet()));
+ }
+ }
+
+ // Explicitly set up number of bundles in SyntheticUnboundedSource since it has a bug in
+ // implementation where
+ // number of lost data in streaming pipeline equals to number of initial bundles.
+ configuration.forceNumInitialBundles =
+ testConfigName.equals("medium")
+ ? NUMBER_OF_BUNDLES_FOR_MEDIUM
+ : NUMBER_OF_BUNDLES_FOR_LARGE;
+
+ // tempLocation needs to be set for DataflowRunner
+ if (!Strings.isNullOrEmpty(tempBucketName)) {
+ String tempLocation = String.format("gs://%s/temp/", tempBucketName);
+ writePipeline.getOptions().as(TestPipelineOptions.class).setTempRoot(tempLocation);
+ writePipeline.getOptions().setTempLocation(tempLocation);
+ readPipeline.getOptions().as(TestPipelineOptions.class).setTempRoot(tempLocation);
+ readPipeline.getOptions().setTempLocation(tempLocation);
+ }
+ writePipeline.getOptions().as(PubsubOptions.class).setProject(project);
+ readPipeline.getOptions().as(PubsubOptions.class).setProject(project);
+
+ if (configuration.exportMetricsToInfluxDB) {
+ configuration.influxHost =
+ TestProperties.getProperty("influxHost", "", TestProperties.Type.PROPERTY);
+ configuration.influxDatabase =
+ TestProperties.getProperty("influxDatabase", "", TestProperties.Type.PROPERTY);
+ configuration.influxMeasurement =
+ TestProperties.getProperty("influxMeasurement", "", TestProperties.Type.PROPERTY);
+ }
+ }
+
+ @After
+ public void tearDownClass() {
+ ResourceManagerUtils.cleanResources(resourceManager);
+ }
+
+ @Test
+ public void testStringWriteAndRead() throws IOException {
+ configuration.writeAndReadFormat = WriteAndReadFormat.STRING.toString();
+ testWriteAndRead();
+ }
+
+ @Test
+ public void testAvroGenericClassWriteAndRead() throws IOException {
+ configuration.writeAndReadFormat = WriteAndReadFormat.AVRO.toString();
+ testWriteAndRead();
+ }
+
+ @Test
+ public void testProtoPrimitiveWriteAndRead() throws IOException {
+ configuration.writeAndReadFormat = WriteAndReadFormat.PROTO.toString();
+ testWriteAndRead();
+ }
+
+ @Test
+ public void testPubsubMessageWriteAndRead() throws IOException {
+ configuration.writeAndReadFormat = WriteAndReadFormat.PUBSUB_MESSAGE.toString();
+ testWriteAndRead();
+ }
+
+ public void testWriteAndRead() throws IOException {
+ if (configuration.exportMetricsToInfluxDB) {
+ influxDBSettings =
+ InfluxDBSettings.builder()
+ .withHost(configuration.influxHost)
+ .withDatabase(configuration.influxDatabase)
+ .withMeasurement(
+ configuration.influxMeasurement
+ + "_"
+ + testConfigName
+ + "_"
+ + configuration.writeAndReadFormat)
+ .get();
+ }
+
+ WriteAndReadFormat format = WriteAndReadFormat.valueOf(configuration.writeAndReadFormat);
+ PipelineLauncher.LaunchInfo writeLaunchInfo = generateDataAndWrite(format);
+ PipelineLauncher.LaunchInfo readLaunchInfo = testRead(format);
+ try {
+ PipelineOperator.Result readResult =
+ pipelineOperator.waitUntilDone(
+ createConfig(readLaunchInfo, Duration.ofMinutes(configuration.pipelineTimeout)));
+
+ // Check the initial launch didn't fail
+ assertNotEquals(PipelineOperator.Result.LAUNCH_FAILED, readResult);
+
+ // check metrics
+ double writeNumRecords =
+ pipelineLauncher.getMetric(
+ project,
+ region,
+ writeLaunchInfo.jobId(),
+ getBeamMetricsName(PipelineMetricsType.COUNTER, WRITE_ELEMENT_METRIC_NAME));
+ double readNumRecords =
+ pipelineLauncher.getMetric(
+ project,
+ region,
+ readLaunchInfo.jobId(),
+ getBeamMetricsName(PipelineMetricsType.COUNTER, READ_ELEMENT_METRIC_NAME));
+
+ // Assert that writeNumRecords equals or greater than readNumRecords since there might be
+ // duplicates when testing big amount of data
+ assertTrue(writeNumRecords >= readNumRecords);
+
+ // export metrics
+ MetricsConfiguration writeMetricsConfig =
+ MetricsConfiguration.builder()
+ .setInputPCollection("Map records.out0")
+ .setInputPCollectionV2("Map records/ParMultiDo(MapKVToPubSubType).out0")
+ .setOutputPCollection("Counting element.out0")
+ .setOutputPCollectionV2("Counting element/ParMultiDo(Counting).out0")
+ .build();
+
+ MetricsConfiguration readMetricsConfig =
+ MetricsConfiguration.builder()
+ .setOutputPCollection("Counting element.out0")
+ .setOutputPCollectionV2("Counting element/ParMultiDo(Counting).out0")
+ .build();
+
+ exportMetrics(
+ writeLaunchInfo,
+ writeMetricsConfig,
+ configuration.exportMetricsToInfluxDB,
+ influxDBSettings);
+ exportMetrics(
+ readLaunchInfo,
+ readMetricsConfig,
+ configuration.exportMetricsToInfluxDB,
+ influxDBSettings);
+ } catch (ParseException | InterruptedException e) {
+ throw new RuntimeException(e);
+ } finally {
+ cancelJobIfRunning(writeLaunchInfo);
+ cancelJobIfRunning(readLaunchInfo);
+ }
+ }
+
+ /**
+ * The method creates a pipeline to simulate data generation and write operations to PubSub, based
+ * on the specified configuration parameters. The stress test involves varying the load
+ * dynamically over time, with options to use configurable parameters.
+ */
+ private PipelineLauncher.LaunchInfo generateDataAndWrite(WriteAndReadFormat format)
+ throws IOException {
+ int startMultiplier =
+ Math.max(configuration.rowsPerSecond, DEFAULT_ROWS_PER_SECOND) / DEFAULT_ROWS_PER_SECOND;
+ List<LoadPeriod> loadPeriods =
+ getLoadPeriods(configuration.minutes, DEFAULT_LOAD_INCREASE_ARRAY);
+
+ PCollection<KV<byte[], byte[]>> dataFromSource =
+ writePipeline.apply(
+ "Read from source", Read.from(new SyntheticUnboundedSource(configuration)));
+
+ if (startMultiplier > 1) {
+ dataFromSource =
+ dataFromSource
+ .apply(
+ "One input to multiple outputs",
+ ParDo.of(new MultiplierDoFn<>(startMultiplier, loadPeriods)))
+ .apply("Reshuffle fanout", Reshuffle.of())
+ .apply("Counting element", ParDo.of(new CountingFn<>(WRITE_ELEMENT_METRIC_NAME)));
+ ;
+ }
+
+ switch (format) {
+ case STRING:
+ dataFromSource
+ .apply(MAP_RECORDS_STEP_NAME, ParDo.of(new MapKVtoString()))
+ .apply(WRITE_TO_PUBSUB_STEP_NAME, PubsubIO.writeStrings().to(topicName.toString()));
+ break;
+ case AVRO:
+ dataFromSource
+ .apply(MAP_RECORDS_STEP_NAME, ParDo.of(new MapKVtoGenericClass()))
+ .apply(
+ WRITE_TO_PUBSUB_STEP_NAME,
+ PubsubIO.writeAvros(GenericClass.class).to(topicName.toString()));
+ break;
+ case PROTO:
+ dataFromSource
+ .apply(MAP_RECORDS_STEP_NAME, ParDo.of(new MapKVtoPrimitiveProto()))
+ .apply(
+ WRITE_TO_PUBSUB_STEP_NAME,
+ PubsubIO.writeProtos(Primitive.class).to(topicName.toString()));
+ break;
+ case PUBSUB_MESSAGE:
+ dataFromSource
+ .apply(MAP_RECORDS_STEP_NAME, ParDo.of(new MapKVtoPubSubMessage()))
+ .apply(WRITE_TO_PUBSUB_STEP_NAME, PubsubIO.writeMessages().to(topicName.toString()));
+ break;
+ }
+
+ PipelineLauncher.LaunchConfig options =
+ PipelineLauncher.LaunchConfig.builder("write-pubsub")
+ .setSdk(PipelineLauncher.Sdk.JAVA)
+ .setPipeline(writePipeline)
+ .addParameter("runner", configuration.runner)
+ .addParameter(
+ "autoscalingAlgorithm",
+ DataflowPipelineWorkerPoolOptions.AutoscalingAlgorithmType.THROUGHPUT_BASED
+ .toString())
+ .addParameter("numWorkers", String.valueOf(configuration.numWorkers))
+ .addParameter("maxNumWorkers", String.valueOf(configuration.maxNumWorkers))
+ .addParameter("streaming", "true")
+ .addParameter("experiments", "use_runner_v2")
+ .build();
+
+ return pipelineLauncher.launch(project, region, options);
+ }
+
+ private PipelineLauncher.LaunchInfo testRead(WriteAndReadFormat format) throws IOException {
+ PubsubIO.Read<?> read = null;
+
+ switch (format) {
+ case STRING:
+ read = PubsubIO.readStrings().fromSubscription(subscription.toString());
+ break;
+ case AVRO:
+ read = PubsubIO.readAvros(GenericClass.class).fromSubscription(subscription.toString());
+ break;
+ case PROTO:
+ read = PubsubIO.readProtos(Primitive.class).fromSubscription(subscription.toString());
+ break;
+ case PUBSUB_MESSAGE:
+ read = PubsubIO.readMessages().fromSubscription(subscription.toString());
+ break;
+ }
+
+ readPipeline
+ .apply("Read from PubSub", read)
+ .apply("Counting element", ParDo.of(new CountingFn<>(READ_ELEMENT_METRIC_NAME)));
+
+ PipelineLauncher.LaunchConfig readOptions =
+ PipelineLauncher.LaunchConfig.builder("read-pubsub")
+ .setSdk(PipelineLauncher.Sdk.JAVA)
+ .setPipeline(readPipeline)
+ .addParameter("runner", configuration.runner)
+ .addParameter("streaming", "true")
+ .addParameter("experiments", "use_runner_v2")
+ .addParameter("numWorkers", String.valueOf(configuration.numWorkers))
+ .addParameter("maxNumWorkers", String.valueOf(configuration.maxNumWorkers))
+ .build();
+
+ return pipelineLauncher.launch(project, region, readOptions);
+ }
+
+ private void cancelJobIfRunning(PipelineLauncher.LaunchInfo pipelineLaunchInfo)
+ throws IOException {
+ if (pipelineLauncher.getJobStatus(project, region, pipelineLaunchInfo.jobId())
+ == PipelineLauncher.JobState.RUNNING) {
+ pipelineLauncher.cancelJob(project, region, pipelineLaunchInfo.jobId());
+ }
+ }
+
+ /** Mapper class to convert data from KV<byte[], byte[]> to String. */
+ private static class MapKVtoString extends DoFn<KV<byte[], byte[]>, String> {
+ @ProcessElement
+ public void process(ProcessContext context) {
+ byte[] byteValue = Objects.requireNonNull(context.element()).getValue();
+ context.output(ByteString.copyFrom(byteValue).toString(StandardCharsets.UTF_8));
+ }
+ }
+
+ /** Mapper class to convert data from KV<byte[], byte[]> to GenericClass. */
+ private static class MapKVtoGenericClass extends DoFn<KV<byte[], byte[]>, GenericClass> {
+ @ProcessElement
+ public void process(ProcessContext context) {
+ byte[] byteValue = Objects.requireNonNull(context.element()).getValue();
+ GenericClass pojo = new GenericClass(byteValue);
+ context.output(pojo);
+ }
+ }
+
+ /** Mapper class to convert data from KV<byte[], byte[]> to Proto Primitive. */
+ private static class MapKVtoPrimitiveProto extends DoFn<KV<byte[], byte[]>, Primitive> {
+ @ProcessElement
+ public void process(ProcessContext context) {
+ byte[] byteValue = Objects.requireNonNull(context.element()).getValue();
+ Primitive proto =
+ Primitive.newBuilder()
+ .setPrimitiveBytes(ByteString.copyFrom(byteValue))
+ .setPrimitiveInt32(ByteBuffer.wrap(byteValue).getInt())
+ .build();
+ context.output(proto);
+ }
+ }
+
+ /** Mapper class to convert data from KV<byte[], byte[]> to PubSubMessage. */
+ private static class MapKVtoPubSubMessage extends DoFn<KV<byte[], byte[]>, PubsubMessage> {
+ @ProcessElement
+ public void process(ProcessContext context) {
+ byte[] byteValue = Objects.requireNonNull(context.element()).getValue();
+ PubsubMessage pubsubMessage = new PubsubMessage(byteValue, Collections.emptyMap());
+ context.output(pubsubMessage);
+ }
+ }
+
+ /** Example of Generic class to test PubSubIO.writeAvros()/readAvros methods. */
+ static class GenericClass implements Serializable {
+ byte[] byteField;
+
+ public GenericClass() {}
+
+ public GenericClass(byte[] byteField) {
+ this.byteField = byteField;
+ }
+
+ @Override
+ public String toString() {
+ return MoreObjects.toStringHelper(getClass()).add("byteField", byteField).toString();
+ }
+
+ @Override
+ public int hashCode() {
+ return Objects.hash(Arrays.hashCode(byteField));
+ }
+
+ @Override
+ public boolean equals(@Nullable Object other) {
+ if (other == null || !(other instanceof GenericClass)) {
+ return false;
+ }
+ GenericClass o = (GenericClass) other;
+ return Arrays.equals(byteField, o.byteField);
+ }
+ }
+
+ private enum WriteAndReadFormat {
+ STRING,
+ AVRO,
+ PROTO,
+ PUBSUB_MESSAGE
+ }
+
+ /** Options for PubSub IO load test. */
+ static class Configuration extends SyntheticSourceOptions {
+ /** Pipeline timeout in minutes. Must be a positive value. */
+ @JsonProperty public int pipelineTimeout = 20;
+
+ /** Runner specified to run the pipeline. */
+ @JsonProperty public String runner = "DirectRunner";
+
+ /** PubSub write and read format: STRING/AVRO/PROTO/PUBSUB_MESSAGE. */
+ @JsonProperty public String writeAndReadFormat = "STRING";
+
+ /** Number of workers for the pipeline. */
+ @JsonProperty public int numWorkers = 20;
+
+ /** Maximum number of workers for the pipeline. */
+ @JsonProperty public int maxNumWorkers = 100;
+
+ /**
+ * Rate of generated elements sent to the source table. Will run with a minimum of 1k rows per
+ * second.
+ */
+ @JsonProperty public int rowsPerSecond = DEFAULT_ROWS_PER_SECOND;
+
+ /** Rows will be generated for this many minutes. */
+ @JsonProperty public int minutes = 15;
+
+ /**
+ * Determines the destination for exporting metrics. If set to true, metrics will be exported to
+ * InfluxDB and displayed using Grafana. If set to false, metrics will be exported to BigQuery
+ * and displayed with Looker Studio.
+ */
+ @JsonProperty public boolean exportMetricsToInfluxDB = true;
Review Comment:
If some of the Influx parameters are not passed, then it automactically skips the exporting and test runs successfully. This logic is already implemented in `exportMetircs `method.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org