Skip to content
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

SQL: unwrap the first value in an array in case of array leniency #40318

Merged
merged 2 commits into from Mar 21, 2019

Conversation

Projects
None yet
5 participants
@astefan
Copy link
Contributor

commented Mar 21, 2019

This bug fix covers an edge case for array leniency option where the array of values is actually an array of objects.

Fixes #40296.

astefan added some commits Mar 21, 2019

@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Mar 21, 2019

@costin

costin approved these changes Mar 21, 2019

Copy link
Member

left a comment

LGTM

@astefan

This comment has been minimized.

Copy link
Contributor Author

commented Mar 21, 2019

@elasticmachine run elasticsearch-ci/2

@matriv

matriv approved these changes Mar 21, 2019

Copy link
Contributor

left a comment

LGTM

@astefan astefan merged commit faf02e0 into elastic:master Mar 21, 2019

8 checks passed

CLA All commits in pull request signed
Details
elasticsearch-ci/1 Build finished.
Details
elasticsearch-ci/2 Build finished.
Details
elasticsearch-ci/bwc Build finished.
Details
elasticsearch-ci/default-distro Build finished.
Details
elasticsearch-ci/docbldesx Build finished.
Details
elasticsearch-ci/oss-distro-docs Build finished.
Details
elasticsearch-ci/packaging-sample Build finished.
Details

@astefan astefan deleted the astefan:40296_fix branch Mar 21, 2019

astefan added a commit that referenced this pull request Mar 22, 2019

astefan added a commit that referenced this pull request Mar 22, 2019

astefan added a commit that referenced this pull request Mar 22, 2019

@michaelbaamonde michaelbaamonde added v7.0.0-rc1 and removed v7.0.0 labels Mar 25, 2019

pgomulka added a commit to pgomulka/elasticsearch that referenced this pull request Mar 25, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.