Skip to content

Commit

Permalink
[ES|QL] High-level AST APIs for the WHERE command (elastic#199998)
Browse files Browse the repository at this point in the history
## Summary

Partially addresses elastic#191812

Implements high-level APIs for working with `WHERE` command.

- `commands.where.list()` — lists all `WHERE` commands.
- `commands.where.byIndex()` — finds the Nth `WHERE` command in
the query.
- `commands.where.byField()` — finds the first `WHERE` command
which uses a specified field or a param.

### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

### For maintainers

- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)

(cherry picked from commit 3e899e7)
  • Loading branch information
vadimkibana committed Nov 28, 2024
1 parent c6e7f09 commit 40ee127
Show file tree
Hide file tree
Showing 8 changed files with 730 additions and 11 deletions.
85 changes: 85 additions & 0 deletions packages/kbn-esql-ast/src/builder/builder.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ import {
ESQLPositionalParamLiteral,
ESQLOrderExpression,
ESQLSource,
ESQLParamLiteral,
ESQLFunction,
ESQLAstItem,
} from '../types';
import { AstNodeParserFields, AstNodeTemplate, PartialFields } from './types';

Expand Down Expand Up @@ -171,6 +174,53 @@ export namespace Builder {
};
};

export namespace func {
export const node = (
template: AstNodeTemplate<ESQLFunction>,
fromParser?: Partial<AstNodeParserFields>
): ESQLFunction => {
return {
...template,
...Builder.parserFields(fromParser),
type: 'function',
};
};

export const call = (
nameOrOperator: string | ESQLIdentifier | ESQLParamLiteral,
args: ESQLAstItem[],
template?: Omit<AstNodeTemplate<ESQLFunction>, 'subtype' | 'name' | 'operator' | 'args'>,
fromParser?: Partial<AstNodeParserFields>
): ESQLFunction => {
let name: string;
let operator: ESQLIdentifier | ESQLParamLiteral;
if (typeof nameOrOperator === 'string') {
name = nameOrOperator;
operator = Builder.identifier({ name });
} else {
operator = nameOrOperator;
name = LeafPrinter.print(operator);
}
return Builder.expression.func.node(
{ ...template, name, operator, args, subtype: 'variadic-call' },
fromParser
);
};

export const binary = (
name: string,
args: [left: ESQLAstItem, right: ESQLAstItem],
template?: Omit<AstNodeTemplate<ESQLFunction>, 'subtype' | 'name' | 'operator' | 'args'>,
fromParser?: Partial<AstNodeParserFields>
): ESQLFunction => {
const operator = Builder.identifier({ name });
return Builder.expression.func.node(
{ ...template, name, operator, args, subtype: 'binary-expression' },
fromParser
);
};
}

export namespace literal {
/**
* Constructs an integer literal node.
Expand All @@ -189,6 +239,21 @@ export namespace Builder {
return node;
};

export const integer = (
value: number,
template?: Omit<AstNodeTemplate<ESQLIntegerLiteral | ESQLDecimalLiteral>, 'name'>,
fromParser?: Partial<AstNodeParserFields>
): ESQLIntegerLiteral | ESQLDecimalLiteral => {
return Builder.expression.literal.numeric(
{
...template,
value,
literalType: 'integer',
},
fromParser
);
};

export const list = (
template: Omit<AstNodeTemplate<ESQLList>, 'name'>,
fromParser?: Partial<AstNodeParserFields>
Expand Down Expand Up @@ -262,5 +327,25 @@ export namespace Builder {

return node;
};

export const build = (
name: string,
options: Partial<ESQLParamLiteral> = {},
fromParser?: Partial<AstNodeParserFields>
): ESQLParam => {
const value: string = name.startsWith('?') ? name.slice(1) : name;

if (!value) {
return Builder.param.unnamed(options);
}

const isNumeric = !isNaN(Number(value)) && String(Number(value)) === value;

if (isNumeric) {
return Builder.param.positional({ ...options, value: Number(value) }, fromParser);
} else {
return Builder.param.named({ ...options, value }, fromParser);
}
};
}
}
3 changes: 2 additions & 1 deletion packages/kbn-esql-ast/src/mutate/commands/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,6 @@ import * as from from './from';
import * as limit from './limit';
import * as sort from './sort';
import * as stats from './stats';
import * as where from './where';

export { from, limit, sort, stats };
export { from, limit, sort, stats, where };
Loading

0 comments on commit 40ee127

Please sign in to comment.