The ProjectionExec constructor takes a raw projection object and parses it. This parsing code is copy-pasted from the ParsedProjection class, and all changes to the projection code requires keeping the two parsing engines in sync. Instead, the ProjectionExec constructor should take a ParsedProjection object and not have access to the raw projection object.

This will require non-trivial cleanup of the ProjectionExec and ParsedProjection classes.