Skip to content

Commit 7140d7f

Browse files
jnthntatumcopybara-github
authored andcommitted
Opt-in to eager field lookup by default.
PiperOrigin-RevId: 954995155
1 parent e25ca10 commit 7140d7f

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

eval/public/cel_options.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,7 @@ struct InterpreterOptions {
222222
// When enabled, the planner will attempt to use a more performant execution
223223
// path for field access when the type is known at plan time, instead of using
224224
// the generic field access implementation.
225-
bool enable_typed_field_access = false;
225+
bool enable_typed_field_access = true;
226226
};
227227
// LINT.ThenChange(//depot/google3/runtime/runtime_options.h)
228228

runtime/runtime_options.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ struct RuntimeOptions {
200200
// This is not recommended if the values bound to the activation are typically
201201
// not what the planner expected (e.g. a map that was declared as a proto or
202202
// a different message with matching field names).
203-
bool enable_typed_field_access = false;
203+
bool enable_typed_field_access = true;
204204
};
205205
// LINT.ThenChange(//depot/google3/eval/public/cel_options.h)
206206

0 commit comments

Comments
 (0)