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

[Backport 2.x] Added new datetime functions and aliases to PPL [Part 2] #1870

Merged
merged 1 commit into from
Jul 27, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 8c8e08c from #1852

* Added new datetime functions, documentation, and tests

Signed-off-by: Matthew Wells <[email protected]>
(cherry picked from commit 8c8e08c)
@Yury-Fridlyand Yury-Fridlyand force-pushed the backport/backport-1852-to-2.x branch from a3d6469 to 76a5874 Compare July 27, 2023 18:35
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #1870 (76a5874) into 2.x (732e431) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##                2.x    #1870   +/-   ##
=========================================
  Coverage     97.41%   97.42%           
- Complexity     4642     4648    +6     
=========================================
  Files           408      408           
  Lines         11506    11528   +22     
  Branches        839      839           
=========================================
+ Hits          11209    11231   +22     
  Misses          290      290           
  Partials          7        7           
Flag Coverage Δ
sql-engine 97.42% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
...pensearch/sql/ppl/parser/AstExpressionBuilder.java 100.00% <100.00%> (ø)

@GumpacG GumpacG merged commit a79601e into 2.x Jul 27, 2023
@github-actions github-actions bot deleted the backport/backport-1852-to-2.x branch July 27, 2023 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants