-
Notifications
You must be signed in to change notification settings - Fork 606
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
YQL-18052: Introduce CLI option to purebench to toggle block engine mode #4526
YQL-18052: Introduce CLI option to purebench to toggle block engine mode #4526
Conversation
⚪ |
⚪ |
⚪ |
d8894df
to
ed45800
Compare
⚪
|
⚪
|
ed45800
to
1db9939
Compare
⚪
|
⚪
|
As a result of the patch, the body of the thrown TCompileError is also yield to stderr.
This patch implements passing through the block engine mode, being set in the program factory, to the worker type context.
This patch introduces BlockEngineSettings option for program factory to tweak the block engine mode. The possible values are "force" to unconditionally enable it, "disable" for turn it off and "auto" to left the final decision to the platform heuristics.
This patch implements conversion from the string option set via the purecalc interface to the corresponding enum value. If the conversion fails, TCompileError is raised. Unfortunately, there is no way to hoist the conversion upper in the call stack, since this will lead to exposure of the private interfaces.
This patch introduces the purebench CLI option to tweak the mode of the block engine while performing the benchmarks.
1db9939
to
222bef4
Compare
⚪
|
⚪ |
⚪
|
This patchset introduces CLI option (
-b
/--block-engine
) topurebench
for toggling block engine mode. It is initially set as a string value (one from'force'
,'disable'
and'auto'
), and converted toEBlockEngineMode
in program factory constructor. If the conversion fails,TCompileError
is raised.There was also a tiny bug: the
TCompileError
error body was not printed. As a result of the first patch of the series this issue is fixed.Changelog category