Skip to content

[Transform] disable optimizations when using scripts in group_by #60724

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

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

hendrikmuhs
Copy link

disable optimizations when using scripts in group_by, when scripts using scripts we can not predict the
outcome and we have no query counterpart. Other optimizations for other group_by's are not affected.

fixes #57332

Note: if no group_by is optimized, a warning is created (introduced in #59591)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml/Transform)

Copy link
Contributor

@przemekwitek przemekwitek left a comment

Choose a reason for hiding this comment

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

LGTM

@hendrikmuhs hendrikmuhs merged commit 8674826 into elastic:master Aug 5, 2020
@hendrikmuhs hendrikmuhs deleted the transform-#57332 branch August 5, 2020 14:56
hendrikmuhs pushed a commit that referenced this pull request Aug 5, 2020
)

disable optimizations when using scripts in group_by, when scripts using scripts we can not predict
the outcome and we have no query counterpart. Other optimizations for other group_by's are not
affected.

fixes #57332
hendrikmuhs pushed a commit to hendrikmuhs/elasticsearch that referenced this pull request Aug 6, 2020
leftover from a previous solution of elastic#57332, which has been discarded

relates elastic#60724
fixes elastic#60794
hendrikmuhs pushed a commit that referenced this pull request Aug 6, 2020
remove test, scripts are excluded in the change collector, the test is a leftover from a previous
solution of #57332, which has been discarded

relates #60724
fixes #60794
hendrikmuhs pushed a commit that referenced this pull request Aug 6, 2020
remove test, scripts are excluded in the change collector, the test is a leftover from a previous
solution of #57332, which has been discarded

relates #60724
fixes #60794
@henryju
Copy link

henryju commented Sep 3, 2020

Hi,

We are affected by this issue on Elastic Cloud. Is there any ETA for the release of 7.10?

Thanks

hendrikmuhs pushed a commit that referenced this pull request Sep 17, 2020
…62524)

the outcome and we have no query counterpart. Other optimizations for other group_by's are not
affected.

relates #57332
backport #60724
@hendrikmuhs
Copy link
Author

@henryju sorry for the late reply, I was OOF.

We revisited the backport and revised our original decision. 7.9.2 will contain a fix for this.

I can not give you any information about release dates. However, the backport to 7.9.2 reduces the time you need to wait.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Transform] scripted group_by fails for continuos transform
6 participants