tvalentyn commented on code in PR #25895: URL: https://github.com/apache/beam/pull/25895#discussion_r1152482278
########## CHANGES.md: ########## @@ -67,13 +67,14 @@ * Schema'd PTransforms can now be directly applied to Beam dataframes just like PCollections. (Note that when doing multiple operations, it may be more efficient to explicitly chain the operations like `df | (Transform1 | Transform2 | ...)` to avoid excessive conversions.) -* The Go SDK adds new transforms periodic.Impulse and periodic.Sequence that extends support +* The Go SDK adds new transforms periodic.Impulse and periodic.Sequence that extends support for slowly updating side input patterns. ([#23106](https://github.com/apache/beam/issues/23106)) ## Breaking Changes * If a main session fails to load, the pipeline will now fail at worker startup. ([#25401](https://github.com/apache/beam/issues/25401)). * Python pipeline options will now ignore unparsed command line flags prefixed with a single dash. ([#25943](https://github.com/apache/beam/issues/25943)). +* SmallestPerKey combiner requires specifying keyword only arguments 'key' and 'reverse'. ([#25888](https://github.com/apache/beam/issues/25888)). Review Comment: ```suggestion * The SmallestPerKey combiner now requires keyword-only arguments for specifying optional parameters, such as `key` and `reverse`. ([#25888](https://github.com/apache/beam/issues/25888)). ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
