github-mirrors/kibana
1
0
Fork 0
mirror of https://github.com/elastic/kibana.git synced 2025-04-25 10:23:14 -04:00
Code Issues Projects Releases Packages Wiki Activity

Expressions debug mode (#57841) (#58354)

Browse source 
* feat: 🎸 add ability to execute expression in debug mode

* feat: 🎸 store resolved arguments in debug information

* feat: 🎸 track function execution time and set "success" flag

* feat: 🎸 store debug information for functions that throw

* feat: 🎸 use performance.now, safe "fn" reference, fix typo
This commit is contained in:
Vadim Dalecky 2020-02-24 22:13:12 +01:00 • committed by GitHub
parent 2cd5d4b7ee
commit 7dfc1aeaba
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 379 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

26
src/plugins/expressions/common/executor/executor.ts
View file

@ -31,6 +31,15 @@ import { ExpressionAstExpression, ExpressionAstNode } from '../ast';
import { typeSpecs } from '../expression_types/specs';
import { functionSpecs } from '../expression_functions/specs';
export interface ExpressionExecOptions {
/**
* Whether to execute expression in *debug mode*. In *debug mode* inputs and
* outputs as well as all resolved arguments and time it took to execute each
* function are saved and are available for introspection.
*/
debug?: boolean;
}
export class TypesRegistry implements IRegistry<ExpressionType> {
constructor(private readonly executor: Executor<any>) {}
@ -145,10 +154,14 @@ export class Executor<Context extends Record<string, unknown> = Record<string, u
return this.state.selectors.getContext();
}
public async interpret<T>(ast: ExpressionAstNode, input: T): Promise<unknown> {
public async interpret<T>(
ast: ExpressionAstNode,
input: T,
options?: ExpressionExecOptions
): Promise<unknown> {
switch (getType(ast)) {
case 'expression':
return await this.interpretExpression(ast as ExpressionAstExpression, input);
return await this.interpretExpression(ast as ExpressionAstExpression, input, options);
case 'string':
case 'number':
case 'null':
@ -161,9 +174,10 @@ export class Executor<Context extends Record<string, unknown> = Record<string, u
public async interpretExpression<T>(
ast: string | ExpressionAstExpression,
input: T
input: T,
options?: ExpressionExecOptions
): Promise<unknown> {
const execution = this.createExecution(ast);
const execution = this.createExecution(ast, undefined, options);
execution.start(input);
return await execution.result;
}
@ -192,7 +206,8 @@ export class Executor<Context extends Record<string, unknown> = Record<string, u
Output = unknown
>(
ast: string | ExpressionAstExpression,
context: ExtraContext = {} as ExtraContext
context: ExtraContext = {} as ExtraContext,
{ debug }: ExpressionExecOptions = {} as ExpressionExecOptions
): Execution<Context & ExtraContext, Input, Output> {
const params: ExecutionParams<Context & ExtraContext> = {
executor: this,
@ -200,6 +215,7 @@ export class Executor<Context extends Record<string, unknown> = Record<string, u
...this.context,
...context,
} as Context & ExtraContext,
debug,
};
if (typeof ast === 'string') params.expression = ast;