Skip to content

Painless: Should } be a delimiter? #18821

@nik9000

Description

@nik9000

I'm porting reindex's tests from groovy to painless. Yay. One thing I found is that a statement like

if (ctx._source.bar != 1) {ctx._source.foo = 1}

doesn't compile. It needs to be

if (ctx._source.bar != 1) {ctx._source.foo = 1;}

with the semicolon between the 1 and the }.

Is it worth making that work?

Metadata

Metadata

Assignees

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions