Skip to content

Conversation

@bpintea
Copy link
Collaborator

@bpintea bpintea commented Mar 28, 2019

This PR fixes (=adds) the advertisement of two supported functions:

  • advertise CURRENT_DATE as DATETIME and SQL92-DATETIME available fn.
  • advertise CURRENT_DATETIME as SQL92 DATETIME available function.

- advertise CURRENT_DATE as DATETIME and SQL92-DATETIME available fn.
- advertise CURRENT_DATETIME as SQL92 DATETIME available function.
Copy link

@droberts195 droberts195 left a comment

Choose a reason for hiding this comment

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

LGTM

@bpintea bpintea merged commit a7c2274 into elastic:master Mar 28, 2019
@bpintea bpintea deleted the fix/advertise_datetime_fn branch March 28, 2019 14:54
bpintea added a commit that referenced this pull request Mar 28, 2019
- advertise CURRENT_DATE as DATETIME and SQL92-DATETIME available fn.
- advertise CURRENT_DATETIME as SQL92 DATETIME available function.

(cherry picked from commit a7c2274)
bpintea added a commit that referenced this pull request Mar 28, 2019
- advertise CURRENT_DATE as DATETIME and SQL92-DATETIME available fn.
- advertise CURRENT_DATETIME as SQL92 DATETIME available function.

(cherry picked from commit a7c2274)
bpintea added a commit that referenced this pull request Mar 28, 2019
- advertise CURRENT_DATE as DATETIME and SQL92-DATETIME available fn.
- advertise CURRENT_DATETIME as SQL92 DATETIME available function.

(cherry picked from commit a7c2274)
@bpintea bpintea added >enhancement and removed fix labels May 3, 2019
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.

2 participants