Skip to content

Scripting: Add back joda to whitelist#35915

Merged
rjernst merged 1 commit intoelastic:masterfrom
rjernst:timeapi11
Nov 27, 2018
Merged

Scripting: Add back joda to whitelist#35915
rjernst merged 1 commit intoelastic:masterfrom
rjernst:timeapi11

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented Nov 26, 2018

Watcher still exposes some dates as joda DateTime objects. This commit
adds back joda to the painless whitelist so they can still be accessed.

closes #35913

Watcher still exposes some dates as joda DateTime objects. This commit
adds back joda to the painless whitelist so they can still be accessed.

closes elastic#35913
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-features

@rjernst rjernst requested a review from spinscale November 26, 2018 17:26
Copy link
Copy Markdown
Contributor

@spinscale spinscale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rjernst rjernst merged commit 72d7f23 into elastic:master Nov 27, 2018
@rjernst rjernst deleted the timeapi11 branch November 27, 2018 00:33
rjernst added a commit that referenced this pull request Nov 27, 2018
Watcher still exposes some dates as joda DateTime objects. This commit
adds back joda to the painless whitelist so they can still be accessed.

closes #35913
rjernst added a commit that referenced this pull request Nov 27, 2018
Watcher still exposes some dates as joda DateTime objects. This commit
adds back joda to the painless whitelist so they can still be accessed.

closes #35913
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache v6.5.2 v6.6.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Watcher: Scripting does not allow use of DateTime objects anymore

5 participants