Skip to content

Conversation

@matriv
Copy link
Contributor

@matriv matriv commented Sep 26, 2019

To be on the safe side in terms of use cases also add the alias
DATETRUNC to the DATE_TRUNC function.

Follows: #46473

To be on the safe side in terms of use cases also add the alias
DATETRUNC to the DATE_TRUNC function.

Follows: elastic#46473
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

@matriv
Copy link
Contributor Author

matriv commented Sep 26, 2019

Not sure if it should be marked as enhancement (for release notes) or non-issue.
@costin ?

@matriv
Copy link
Contributor Author

matriv commented Sep 27, 2019

@elasticmachine run elasticsearch-ci/bwc

@costin
Copy link
Member

costin commented Sep 27, 2019

You could mark it as a feature since it's a new function.

Copy link
Member

@costin costin left a comment

Choose a reason for hiding this comment

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

LGTM

@matriv matriv merged commit 9ac223c into elastic:master Sep 27, 2019
@matriv matriv deleted the add-alias-datetrunc branch September 27, 2019 12:38
matriv added a commit that referenced this pull request Sep 27, 2019
To be on the safe side in terms of use cases also add the alias
DATETRUNC to the DATE_TRUNC function.

Follows: #46473

(cherry picked from commit 9ac223c)
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.

4 participants