After running the memo implementation phase, we currently rebuild the env for each resulting plan here. The rebuild and the following assertion were initially added for debugging purposes but now appear to be hiding a bug that can cause a segmentation fault if the env is not rebuilt after running physical rewrites. We should investigate and fix the segfault, and only call rebuild in debug builds. See this comment for a more detailed description of the segmentation fault.