pabloem commented on a change in pull request #11166: [BEAM-7923] Emit info
when capture control options are configured.
URL: https://github.com/apache/beam/pull/11166#discussion_r395232405
##########
File path: sdks/python/apache_beam/runners/interactive/interactive_beam.py
##########
@@ -109,7 +144,16 @@ def capture_size_limit(self, value):
# Sets the capture size limit to 1GB.
interactive_beam.options.capture_size_limit = 1e9
"""
- self.capture_control._capture_size_limit = value
+ if self.capture_control._capture_size_limit != value:
+ _ = ie.current_env()
+ _LOGGER.info(
+ 'You have changed capture size limit from %s bytes to %s bytes. To '
+ 'allow new data being captured under the updated size limit the '
Review comment:
```suggestion
'allow new data to be captured under the updated size limit, the '
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
