You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by ke...@apache.org on 2026/03/04 16:31:11 UTC

(beam) branch master updated (d42111cd019 -> e7a97c26893)

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

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


    from d42111cd019 Merge pull request #37748: Where a Throwable is being caught, take care to rethrow OutOfMemoryError
     add 505ac1a9b22 Fix MissingSummary errorprone Warnings Across Codebase
     new e7a97c26893 Fix MissingSummary errorprone Warnings Across Codebase (#37759)

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../org/apache/beam/gradle/BeamModulePlugin.groovy |  1 -
 .../gcp/datastream/DatastreamResourceManager.java  | 18 +++++---
 .../beam/it/truthmatchers/ListAccumulator.java     |  2 +-
 .../org/apache/beam/runners/core/DoFnRunner.java   |  3 +-
 .../runners/core/PushbackSideInputDoFnRunner.java  |  3 +-
 .../beam/runners/core/metrics/BoundedTrieData.java |  2 +-
 .../core/metrics/MonitoringInfoMetricName.java     |  2 +-
 .../runners/core/metrics/SimpleStateRegistry.java  |  2 +-
 .../core/triggers/AfterWatermarkStateMachine.java  |  2 +-
 .../beam/runners/dataflow/util/PackageUtil.java    | 10 ++---
 .../dataflow/worker/BatchModeExecutionContext.java |  6 ++-
 .../MetricsToPerStepNamespaceMetricsConverter.java |  7 ++--
 .../worker/StreamingStepMetricsContainer.java      |  5 ++-
 .../util/ThreadLocalByteStringOutputStream.java    |  2 +-
 .../util/common/worker/WorkProgressUpdater.java    |  6 ++-
 .../windmill/client/AbstractWindmillStream.java    |  2 +
 .../windmill/state/WindmillTagEncodingV1.java      |  2 +-
 .../windmill/state/WindmillTagEncodingV2.java      |  2 +-
 .../control/ProcessBundleDescriptors.java          |  2 +-
 .../beam/runners/jobsubmission/JobInvocation.java  |  2 +-
 .../samza/runtime/ClassicBundleManager.java        | 11 +++--
 .../runners/samza/translation/ConfigBuilder.java   |  2 +-
 .../spark/translation/SparkInputDataProcessor.java |  4 +-
 .../java/org/apache/beam/sdk/PipelineResult.java   |  4 +-
 .../sdk/coders/CannotProvideCoderException.java    |  2 +-
 .../org/apache/beam/sdk/io/CompressedSource.java   | 20 ++++-----
 .../java/org/apache/beam/sdk/io/FileBasedSink.java | 21 +++++-----
 .../java/org/apache/beam/sdk/io/TFRecordIO.java    | 10 ++---
 .../main/java/org/apache/beam/sdk/io/TextIO.java   | 16 +++----
 .../java/org/apache/beam/sdk/io/TextSource.java    |  6 +--
 .../apache/beam/sdk/metrics/BoundedTrieResult.java |  2 +-
 .../apache/beam/sdk/metrics/StringSetResult.java   |  2 +-
 .../apache/beam/sdk/runners/AppliedPTransform.java |  2 +-
 .../beam/sdk/transforms/GroupIntoBatches.java      |  6 +--
 .../apache/beam/sdk/transforms/Redistribute.java   |  6 ++-
 .../reflect/ByteBuddyDoFnInvokerFactory.java       |  4 +-
 .../sdk/transforms/reflect/DoFnSignatures.java     |  4 +-
 .../sdk/transforms/windowing/AfterWatermark.java   |  4 +-
 .../org/apache/beam/sdk/util/MutationDetector.java |  2 +-
 .../util/construction/PTransformReplacements.java  |  2 +-
 .../org/apache/beam/sdk/values/WindowedValues.java |  2 +-
 .../euphoria/core/annotation/operator/Basic.java   |  4 +-
 .../euphoria/core/annotation/operator/Derived.java |  4 +-
 .../core/annotation/operator/Recommended.java      |  8 ++--
 .../extensions/euphoria/core/client/lib/Split.java |  4 +-
 .../core/client/operator/AssignEventTime.java      |  9 ++--
 .../euphoria/core/client/operator/ReduceByKey.java |  4 +-
 .../sdk/extensions/gcp/util/CustomHttpErrors.java  |  2 +-
 .../sdk/extensions/gcp/util/GceMetadataUtil.java   |  2 +-
 .../extensions/gcp/util/HttpCallCustomError.java   |  2 +-
 .../sdk/extensions/gcp/util/HttpCallMatcher.java   |  2 +-
 .../gcp/util/RetryHttpRequestInitializer.java      | 12 +++++-
 .../kafka/factories/FileAwareFactoryFn.java        |  8 ++--
 .../beam/sdk/extensions/ml/DLPDeidentifyText.java  | 49 ++++++++++++++++------
 .../beam/sdk/extensions/ml/DLPInspectText.java     | 36 ++++++++++++----
 .../beam/sdk/extensions/ml/DLPReidentifyText.java  | 48 +++++++++++++++------
 .../ml/RecommendationAICreateCatalogItem.java      | 18 +++++---
 .../ml/RecommendationAIImportCatalogItems.java     | 27 ++++++++----
 .../ml/RecommendationAIImportUserEvents.java       | 35 ++++++++++++----
 .../sdk/extensions/ml/RecommendationAIPredict.java | 32 ++++++++++----
 .../ml/RecommendationAIWriteUserEvent.java         | 24 ++++++++---
 .../ordered/ContiguousSequenceRange.java           |  6 +--
 .../ordered/OrderedEventProcessorResult.java       | 14 +++----
 .../ordered/OrderedProcessingHandler.java          |  2 +-
 .../ordered/OrderedProcessingStatus.java           | 28 ++++++-------
 .../beam/sdk/extensions/ordered/ProcessorDoFn.java |  2 +-
 .../extensions/sketching/ApproximateDistinct.java  |  5 ++-
 .../harness/data/PTransformFunctionRegistry.java   |  4 +-
 .../apache/beam/fn/harness/logging/QuotaEvent.java | 16 ++++++-
 .../beam/sdk/io/aws2/kinesis/KinesisRecord.java    |  8 ++--
 .../apache/beam/sdk/io/cassandra/RingRange.java    |  4 +-
 .../beam/sdk/io/common/NetworkTestHelper.java      |  2 +-
 .../sdk/io/gcp/bigquery/BigQuerySinkMetrics.java   | 13 +++---
 .../beam/sdk/io/gcp/bigquery/BigQueryUtils.java    | 23 +++++-----
 .../changestreams/dao/MetadataTableAdminDao.java   |  2 +-
 .../changestreams/dao/MetadataTableDao.java        |  8 ++--
 .../beam/sdk/io/gcp/firestore/FirestoreDoFn.java   |  8 ++--
 .../apache/beam/sdk/io/gcp/firestore/RpcQos.java   | 10 ++---
 .../apache/beam/sdk/io/gcp/healthcare/FhirIO.java  |  2 +-
 .../beam/sdk/io/gcp/spanner/OrderedCode.java       |  2 +-
 .../java/org/apache/beam/sdk/io/jdbc/JdbcUtil.java |  4 +-
 .../KafkaIOReadImplementationCompatibility.java    |  8 ++--
 .../beam/sdk/io/sparkreceiver/HasOffset.java       |  8 +++-
 .../beam/sdk/io/sparkreceiver/ReceiverBuilder.java |  4 +-
 .../java/org/apache/beam/sdk/io/xml/XmlIO.java     | 14 +++----
 .../apache/beam/sdk/nexmark/NexmarkQueryName.java  |  4 +-
 86 files changed, 471 insertions(+), 273 deletions(-)


(beam) 01/01: Fix MissingSummary errorprone Warnings Across Codebase (#37759)

Posted by ke...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit e7a97c26893e9d25fb0224c3c5b3bcdd5ffe569a
Merge: d42111cd019 505ac1a9b22
Author: Kenneth Knowles <kl...@google.com>
AuthorDate: Wed Mar 4 11:31:02 2026 -0500

    Fix MissingSummary errorprone Warnings Across Codebase (#37759)

 .../org/apache/beam/gradle/BeamModulePlugin.groovy |  1 -
 .../gcp/datastream/DatastreamResourceManager.java  | 18 +++++---
 .../beam/it/truthmatchers/ListAccumulator.java     |  2 +-
 .../org/apache/beam/runners/core/DoFnRunner.java   |  3 +-
 .../runners/core/PushbackSideInputDoFnRunner.java  |  3 +-
 .../beam/runners/core/metrics/BoundedTrieData.java |  2 +-
 .../core/metrics/MonitoringInfoMetricName.java     |  2 +-
 .../runners/core/metrics/SimpleStateRegistry.java  |  2 +-
 .../core/triggers/AfterWatermarkStateMachine.java  |  2 +-
 .../beam/runners/dataflow/util/PackageUtil.java    | 10 ++---
 .../dataflow/worker/BatchModeExecutionContext.java |  6 ++-
 .../MetricsToPerStepNamespaceMetricsConverter.java |  7 ++--
 .../worker/StreamingStepMetricsContainer.java      |  5 ++-
 .../util/ThreadLocalByteStringOutputStream.java    |  2 +-
 .../util/common/worker/WorkProgressUpdater.java    |  6 ++-
 .../windmill/client/AbstractWindmillStream.java    |  2 +
 .../windmill/state/WindmillTagEncodingV1.java      |  2 +-
 .../windmill/state/WindmillTagEncodingV2.java      |  2 +-
 .../control/ProcessBundleDescriptors.java          |  2 +-
 .../beam/runners/jobsubmission/JobInvocation.java  |  2 +-
 .../samza/runtime/ClassicBundleManager.java        | 11 +++--
 .../runners/samza/translation/ConfigBuilder.java   |  2 +-
 .../spark/translation/SparkInputDataProcessor.java |  4 +-
 .../java/org/apache/beam/sdk/PipelineResult.java   |  4 +-
 .../sdk/coders/CannotProvideCoderException.java    |  2 +-
 .../org/apache/beam/sdk/io/CompressedSource.java   | 20 ++++-----
 .../java/org/apache/beam/sdk/io/FileBasedSink.java | 21 +++++-----
 .../java/org/apache/beam/sdk/io/TFRecordIO.java    | 10 ++---
 .../main/java/org/apache/beam/sdk/io/TextIO.java   | 16 +++----
 .../java/org/apache/beam/sdk/io/TextSource.java    |  6 +--
 .../apache/beam/sdk/metrics/BoundedTrieResult.java |  2 +-
 .../apache/beam/sdk/metrics/StringSetResult.java   |  2 +-
 .../apache/beam/sdk/runners/AppliedPTransform.java |  2 +-
 .../beam/sdk/transforms/GroupIntoBatches.java      |  6 +--
 .../apache/beam/sdk/transforms/Redistribute.java   |  6 ++-
 .../reflect/ByteBuddyDoFnInvokerFactory.java       |  4 +-
 .../sdk/transforms/reflect/DoFnSignatures.java     |  4 +-
 .../sdk/transforms/windowing/AfterWatermark.java   |  4 +-
 .../org/apache/beam/sdk/util/MutationDetector.java |  2 +-
 .../util/construction/PTransformReplacements.java  |  2 +-
 .../org/apache/beam/sdk/values/WindowedValues.java |  2 +-
 .../euphoria/core/annotation/operator/Basic.java   |  4 +-
 .../euphoria/core/annotation/operator/Derived.java |  4 +-
 .../core/annotation/operator/Recommended.java      |  8 ++--
 .../extensions/euphoria/core/client/lib/Split.java |  4 +-
 .../core/client/operator/AssignEventTime.java      |  9 ++--
 .../euphoria/core/client/operator/ReduceByKey.java |  4 +-
 .../sdk/extensions/gcp/util/CustomHttpErrors.java  |  2 +-
 .../sdk/extensions/gcp/util/GceMetadataUtil.java   |  2 +-
 .../extensions/gcp/util/HttpCallCustomError.java   |  2 +-
 .../sdk/extensions/gcp/util/HttpCallMatcher.java   |  2 +-
 .../gcp/util/RetryHttpRequestInitializer.java      | 12 +++++-
 .../kafka/factories/FileAwareFactoryFn.java        |  8 ++--
 .../beam/sdk/extensions/ml/DLPDeidentifyText.java  | 49 ++++++++++++++++------
 .../beam/sdk/extensions/ml/DLPInspectText.java     | 36 ++++++++++++----
 .../beam/sdk/extensions/ml/DLPReidentifyText.java  | 48 +++++++++++++++------
 .../ml/RecommendationAICreateCatalogItem.java      | 18 +++++---
 .../ml/RecommendationAIImportCatalogItems.java     | 27 ++++++++----
 .../ml/RecommendationAIImportUserEvents.java       | 35 ++++++++++++----
 .../sdk/extensions/ml/RecommendationAIPredict.java | 32 ++++++++++----
 .../ml/RecommendationAIWriteUserEvent.java         | 24 ++++++++---
 .../ordered/ContiguousSequenceRange.java           |  6 +--
 .../ordered/OrderedEventProcessorResult.java       | 14 +++----
 .../ordered/OrderedProcessingHandler.java          |  2 +-
 .../ordered/OrderedProcessingStatus.java           | 28 ++++++-------
 .../beam/sdk/extensions/ordered/ProcessorDoFn.java |  2 +-
 .../extensions/sketching/ApproximateDistinct.java  |  5 ++-
 .../harness/data/PTransformFunctionRegistry.java   |  4 +-
 .../apache/beam/fn/harness/logging/QuotaEvent.java | 16 ++++++-
 .../beam/sdk/io/aws2/kinesis/KinesisRecord.java    |  8 ++--
 .../apache/beam/sdk/io/cassandra/RingRange.java    |  4 +-
 .../beam/sdk/io/common/NetworkTestHelper.java      |  2 +-
 .../sdk/io/gcp/bigquery/BigQuerySinkMetrics.java   | 13 +++---
 .../beam/sdk/io/gcp/bigquery/BigQueryUtils.java    | 23 +++++-----
 .../changestreams/dao/MetadataTableAdminDao.java   |  2 +-
 .../changestreams/dao/MetadataTableDao.java        |  8 ++--
 .../beam/sdk/io/gcp/firestore/FirestoreDoFn.java   |  8 ++--
 .../apache/beam/sdk/io/gcp/firestore/RpcQos.java   | 10 ++---
 .../apache/beam/sdk/io/gcp/healthcare/FhirIO.java  |  2 +-
 .../beam/sdk/io/gcp/spanner/OrderedCode.java       |  2 +-
 .../java/org/apache/beam/sdk/io/jdbc/JdbcUtil.java |  4 +-
 .../KafkaIOReadImplementationCompatibility.java    |  8 ++--
 .../beam/sdk/io/sparkreceiver/HasOffset.java       |  8 +++-
 .../beam/sdk/io/sparkreceiver/ReceiverBuilder.java |  4 +-
 .../java/org/apache/beam/sdk/io/xml/XmlIO.java     | 14 +++----
 .../apache/beam/sdk/nexmark/NexmarkQueryName.java  |  4 +-
 86 files changed, 471 insertions(+), 273 deletions(-)

diff --cc buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
index a83fc22934f,777000749e4..134af9a8b4c
--- a/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
+++ b/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
@@@ -1556,9 -1556,9 +1556,8 @@@ class BeamModulePlugin implements Plugi
              "JavaUtilDate",
              "JodaConstructors",
              "MalformedInlineTag",
-             "MissingSummary",
              "MixedMutabilityReturnType",
              "PreferJavaTimeOverload",
 -            "MutablePublicArray",
              "NonCanonicalType",
              "ProtectedMembersInFinalClass",
              "Slf4jFormatShouldBeConst",