Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Bug fix, using Local.Root when format the string in DateTimeFunctionIT #794

Merged
merged 1 commit into from Oct 21, 2020
Merged

Bug fix, using Local.Root when format the string in DateTimeFunctionIT #794

merged 1 commit into from Oct 21, 2020

Conversation

penghuo
Copy link
Contributor

@penghuo penghuo commented Oct 21, 2020

Description of changes:
using Local.Root when format the string in DateTimeFunctionIT

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

dai-chen
dai-chen approved these changes Oct 21, 2020
View reviewed changes
Copy link
Member

@dai-chen dai-chen left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

joshuali925
joshuali925 approved these changes Oct 21, 2020
View reviewed changes
Copy link
Contributor

@joshuali925 joshuali925 left a comment

Choose a reason for hiding this comment

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

thanks

@penghuo penghuo merged commit 2614dd2 into opendistro-for-elasticsearch:develop Oct 21, 2020
@penghuo penghuo deleted the sql-local-root branch October 21, 2020 22:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@joshuali925 joshuali925 joshuali925 approved these changes

@dai-chen dai-chen dai-chen approved these changes

@chloe-zh chloe-zh Awaiting requested review from chloe-zh

@abbashus abbashus Awaiting requested review from abbashus

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@penghuo @joshuali925 @dai-chen