VictorPlusC commented on a change in pull request #16741:
URL: https://github.com/apache/beam/pull/16741#discussion_r807379285
##########
File path:
sdks/python/apache_beam/runners/interactive/interactive_environment.py
##########
@@ -258,31 +274,53 @@ def inspector_with_synthetic(self):
synthetic variables generated by Interactive Beam. Internally used."""
return self._inspector_with_synthetic
+ def cleanup_pipeline(self, pipeline):
+ from apache_beam.runners.interactive import background_caching_job as bcj
+ bcj.attempt_to_cancel_background_caching_job(pipeline)
+ bcj.attempt_to_stop_test_stream_service(pipeline)
+ cache_manager = self.get_cache_manager(pipeline)
+ # Recording manager performs cache manager cleanup during eviction, so we
+ # don't need to clean it up here.
+ if cache_manager and self.get_recording_manager(pipeline) is None:
+ cache_manager.cleanup()
+ if self.options.cleanup_cluster:
Review comment:
I've removed the cleanup_cluster option completely now, thanks!
--
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]
