You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by "tvalentyn (via GitHub)" <gi...@apache.org> on 2023/05/15 21:19:24 UTC

[GitHub] [beam] tvalentyn commented on a diff in pull request #26700: Add back cloud profiler

tvalentyn commented on code in PR #26700:
URL: https://github.com/apache/beam/pull/26700#discussion_r1194373018


##########
CHANGES.md:
##########
@@ -120,6 +120,8 @@
 * 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)).
 * 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)).
+* The google-cloud-profiler dependency was accidentally removed from Beam's Python Docker

Review Comment:
   good call.



-- 
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