Skip to content

Commit

Permalink
[FEAT] [New Query Planner] Enable new query planner by default. (#1398)
Browse files Browse the repository at this point in the history
Enable new query planner by default. We retain the feature flag + the
old query planner for now, in case a nightly wheel user hits an issue
that needs a quick workaround, so we also keep the query planner feature
flag in the CI matrix.

We shouldn't merge this until:

- [x] Projection folding support is merged:
#1374
  • Loading branch information
clarkzinzow authored Sep 21, 2023
1 parent 16c1553 commit eccd030
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion daft/context.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ def _get_runner_config_from_env() -> _RunnerConfig:

def _get_planner_from_env() -> bool:
"""Returns whether or not to use the new query planner."""
return bool(int(os.getenv("DAFT_NEW_QUERY_PLANNER", default="0")))
return bool(int(os.getenv("DAFT_NEW_QUERY_PLANNER", default="1")))


@dataclasses.dataclass(frozen=True)
Expand Down

0 comments on commit eccd030

Please sign in to comment.