You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/06/29 17:04:41 UTC
[GitHub] [beam] lukecwik commented on a diff in pull request #22049: Projection Pushdown optimizer on by default
lukecwik commented on code in PR #22049:
URL: https://github.com/apache/beam/pull/22049#discussion_r910208118
##########
CHANGES.md:
##########
@@ -28,6 +28,7 @@
## I/Os
* Support for X source added (Java/Python) ([#X](https://github.com/apache/beam/issues/X)).
+* Projection Pushdown optimizer is now on by default for streaming, matching the behavior of batch pipelines since 2.38.0. If you encounter a bug with the optimizer, please file an issue and disable the optimizer using pipeline option `--experiments=disable_projection_pushdown`.
Review Comment:
Would it be worthwhile to mention something about upgrade compatibility?
##########
CHANGES.md:
##########
@@ -28,6 +28,7 @@
## I/Os
* Support for X source added (Java/Python) ([#X](https://github.com/apache/beam/issues/X)).
+* Projection Pushdown optimizer is now on by default for streaming, matching the behavior of batch pipelines since 2.38.0. If you encounter a bug with the optimizer, please file an issue and disable the optimizer using pipeline option `--experiments=disable_projection_pushdown`.
Review Comment:
Would it be worthwhile to mention something about pipeline update compatibility?
--
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