[BEAM-11700] Optimizations skipped in PortableRunner when pre_optimize is 'default' or 'all'

• Type: Bug
• Status: Resolved
• Priority: P2
• Resolution: Fixed
• Fix Version/s: Missing
• Component/s: sdk-py-core
• Assignee: Yifan Mai
• Reporter: Yifan Mai
• Created: 27/Jan/21 19:17
• Updated: 13/Jan/22 21:49
• Resolved: 20/Apr/21 21:39

Description

An indentation error in portable_runner.py is causing optimizations to not run when pre_optimize is set to default or all. This was broken by #13573.

Issue Links

• is duplicated by: BEAM-11718 Actually run pre-optimize phases
• links to: GitHub Pull Request #13824
• links to: GitHub Pull Request #13882