Skip to content

Commit 3233b69

Browse files
committed
Add missing base case
1 parent 837ba2f commit 3233b69

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

Diff for: sdks/python/apache_beam/runners/direct/direct_runner.py

+4-1
Original file line numberDiff line numberDiff line change
@@ -146,10 +146,11 @@ def visit_transform(self, applied_ptransform):
146146
if timer.time_domain == TimeDomain.REAL_TIME:
147147
self.supported_by_prism_runner = False
148148

149+
# Use BundleBasedDirectRunner if other runners are missing needed features.
149150
runner = BundleBasedDirectRunner()
150151

151152
# Check whether all transforms used in the pipeline are supported by the
152-
# FnApiRunner, and the pipeline was not meant to be run as streaming.
153+
# PrismRunner
153154
if _PrismRunnerSupportVisitor().accept(pipeline):
154155
_LOGGER.info('Running pipeline with PrismRunner.')
155156
from apache_beam.runners.portability import prism_runner
@@ -173,6 +174,8 @@ def visit_transform(self, applied_ptransform):
173174
_LOGGER.info('Falling back to DirectRunner')
174175
runner = BundleBasedDirectRunner()
175176

177+
# Check whether all transforms used in the pipeline are supported by the
178+
# FnApiRunner, and the pipeline was not meant to be run as streaming.
176179
if _FnApiRunnerSupportVisitor().accept(pipeline):
177180
from apache_beam.portability.api import beam_provision_api_pb2
178181
from apache_beam.runners.portability.fn_api_runner import fn_runner

0 commit comments

Comments
 (0)